Related to T2494
Details
Details
- Reviewers
vlorentz - Group Reviewers
Reviewers - Maniphest Tasks
- T2494: tests: Use data model objects within tests (drop dicts)
- Commits
- rDSTO87b1070b8f39: test_storage: directory: Use data model object
tox
Diff Detail
Diff Detail
- Repository
- rDSTO Storage manager
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Comment Actions
Build is green
Patch application report for D3563 (id=12552)
Could not rebase; Attempt merge onto 7131dcb9cf...
Updating 7131dcb9..87b1070b Fast-forward swh/storage/pytest_plugin.py | 2 +- swh/storage/tests/conftest.py | 12 +- swh/storage/tests/generate_data_test.py | 49 ----- swh/storage/tests/storage_data.py | 2 +- swh/storage/tests/test_storage.py | 343 ++++++++++++++++---------------- 5 files changed, 180 insertions(+), 228 deletions(-) delete mode 100644 swh/storage/tests/generate_data_test.py
Changes applied before test
commit 87b1070b8f3904bf3eaa8c32e4fbb7f1b8c9f9b6 Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org> Date: Mon Jul 20 11:58:34 2020 +0200 test_storage: directory: Use data model object Related to T2494 commit 1a2b85fa1eeb690d675828d83611cfb87f34ff7f Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org> Date: Mon Jul 20 11:09:08 2020 +0200 test_storage: Make swh_contents fixture generate content model objects Note that this will be improved upon after the storage migration. We can take a look at the swh.model.tests.generate_testdata.gen_contents and make it generate BaseContent objects directly if it's possible. This was not done here so the impacts is limited to storage. Related to T2494 commit 99a28ad30b14ec72b8458ecd9503b22b7898e323 Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org> Date: Mon Jul 20 10:41:02 2020 +0200 tests.generate_data_test: Remove dead code The storage fixtures now uses the swh.model.tests.generate_data_test instead Related to T2494 commit 03e17c3600f340e077d835f19cba05893d114612 Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org> Date: Mon Jul 20 10:12:13 2020 +0200 test_storage: skipped_content/content_missing: Use data model object Related to T2494
See https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/490/ for more details.