Skip to content

tests: Reduce the number of warnings raised

Renaud Boyer requested to merge rboyer/swh-web:fix-warnings into master

TODO

  • pytest:
    • Unknown pytest.mark.inbound_message
    • unused pytest asyncio_mode config option
  • Converter 'swhid' is already registered
  • Vault & Indexer storage DeprecationWarning while init using args
  • docutils:
    • frontend.OptionParser is deprecated
    • use Node.findall instead of Node.traverse
  • UrlField's default scheme -> transitional setting + silence warning
  • Naive datetime value used for an aware field
  • Re-implemented GitLoaderFromDisk to remove the last dependency to the deprecated swh.loader.git.from_disk
  • Identifying contents by sha1 instead of hash dicts {"sha1": b"..."} is deprecated
  • test_message method is not a test

Will be fixed elsewhere

Wontfix here

Results

Down from 41630 warnings in master to 1185, mostly (1136) due to a deprecation warning fixed in swh-loader-git!189 (merged) 🎉

The only change impacting swh-web behavior is setting https the default scheme for UrlFields (hence the change in e2e tests).

Edited by Renaud Boyer

Merge request reports