Skip to content

Add tox.ini and pytest.ini in the MANIFEST.in file

Make tests pass with django 2

For this we:

  • ensure we create the DepositRequest with str as raw_metadata (instead of a bytes), and

  • make atom_dataset return a str instead of a bytes

    Every time this chunk of data is sent on the wire (http req to the django app) this needs to be an str, otherwise we have things like:

    raw_metadata = "b'[...]'"

    especially when testing against django 2, which is the only django version available on debian sid, so this is needed to make it possible to build the deb file on sid with tests enabled.

  • add a py3-django2 environment in tox

  • bump the dependency to django <3 in requirements-server.txt (otherwise it will take precendence on the deps: field of the tox file, thus execute tests in py3-django2 with django 1.11).

Note that no other tests than executing pytest have been done with django2, so do not expect it works flawlessly.

Test Plan

tox should still be happy...


Migrated from D2517 (view on Phabricator)

Merge request reports