Page MenuHomeSoftware Heritage

Add types and tests on configuration checks
ClosedPublic

Authored by ardumont on Fri, Oct 16, 8:05 PM.

Diff Detail

Repository
rDVAU Software Heritage Vault
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Fri, Oct 16, 8:05 PM

Build is green

Patch application report for D4295 (id=15178)

Could not rebase; Attempt merge onto a8fa57718b...

Updating a8fa577..893591b
Fast-forward
 requirements-test.txt          |  1 +
 swh/vault/__init__.py          | 53 +++++++++++++++++++++++++---------------
 swh/vault/api/server.py        | 21 ++++++++--------
 swh/vault/backend.py           | 13 ++++++----
 swh/vault/tests/conftest.py    | 46 +++++++++++++++++------------------
 swh/vault/tests/test_init.py   | 55 ++++++++++++++++++++++++++++++++++++++++++
 swh/vault/tests/test_server.py | 38 ++++++++++++++++++++++++++++-
 7 files changed, 167 insertions(+), 60 deletions(-)
 create mode 100644 swh/vault/tests/test_init.py
Changes applied before test
commit 893591b05d78d1706f48144562382aedb55d6c49
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Oct 16 20:06:00 2020 +0200

    api.server: Add types and tests on configuration checks

commit efd5eb4dbdb3b451997141b661cb9ae53b8c3c34
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Oct 16 19:34:02 2020 +0200

    swh.vault: Unify get_vault factory function with other factories
    
    Related to T1410

commit 4f1b89d01a441e1048c5a276773b37e3f7d2e276
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Oct 16 15:50:06 2020 +0200

    vault.config: Adapt scheduler configuration structure

See https://jenkins.softwareheritage.org/job/DVAU/job/tests-on-diff/23/ for more details.

Build is green

Patch application report for D4295 (id=15194)

Could not rebase; Attempt merge onto b5cecff6df...

Updating b5cecff..7eb906e
Fast-forward
 requirements-test.txt          |  1 +
 swh/vault/__init__.py          | 53 +++++++++++++++++++++++++---------------
 swh/vault/api/server.py        | 16 ++++++------
 swh/vault/backend.py           | 13 ++++++----
 swh/vault/tests/conftest.py    | 46 +++++++++++++++++------------------
 swh/vault/tests/test_init.py   | 55 ++++++++++++++++++++++++++++++++++++++++++
 swh/vault/tests/test_server.py | 38 ++++++++++++++++++++++++++++-
 7 files changed, 166 insertions(+), 56 deletions(-)
 create mode 100644 swh/vault/tests/test_init.py
Changes applied before test
commit 7eb906ea76ad11ceacd0fb153690c2cdf233c742
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Oct 16 20:06:00 2020 +0200

    api.server: Add types and tests on configuration checks

commit 8e4026a5e8c9d43832cd3b038b9175f3f4914407
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Oct 16 19:34:02 2020 +0200

    swh.vault: Unify get_vault factory function with other factories
    
    Related to T1410

See https://jenkins.softwareheritage.org/job/DVAU/job/tests-on-diff/25/ for more details.

vlorentz requested changes to this revision.Mon, Oct 19, 12:43 PM
vlorentz added a subscriber: vlorentz.
vlorentz added inline comments.
swh/vault/tests/test_server.py
62

what is this %?

69

black folded the lines

91

you can remove , "This should not raise", it's redundant

This revision now requires changes to proceed.Mon, Oct 19, 12:43 PM
ardumont added inline comments.Mon, Oct 19, 1:05 PM
swh/vault/tests/test_server.py
62

no idea ;)

ardumont updated this revision to Diff 15215.Mon, Oct 19, 3:13 PM

Adapt according to review:

  • Drop % in configuration key
  • Merge 2 strings in one
  • Drop the redundant message in assertion failure

Build is green

Patch application report for D4295 (id=15215)

Rebasing onto 49d787c185...

First, rewinding head to replay your work on top of it...
Fast-forwarded diff-target to base-revision-1012-D4295.
Changes applied before test

See https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1012/ for more details.

ardumont updated this revision to Diff 15216.Mon, Oct 19, 3:23 PM

Use the right folder to actually sync the diff ¯\_(ツ)_/¯

Build is green

Patch application report for D4295 (id=15216)

Rebasing onto 8e4026a5e8...

Current branch diff-target is up to date.
Changes applied before test
commit a2ffe238d0e59def15ac5dd5a41717f6a5ff63e8
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Oct 16 20:06:00 2020 +0200

    api.server: Add types and tests on configuration checks

See https://jenkins.softwareheritage.org/job/DVAU/job/tests-on-diff/26/ for more details.

ardumont updated this revision to Diff 15259.Tue, Oct 20, 4:15 PM
  • Add tests on current configuration check for cooker instantiation

Build is green

Patch application report for D4295 (id=15259)

Rebasing onto 8e4026a5e8...

Current branch diff-target is up to date.
Changes applied before test
commit 829601a7f50ce2cf9c6d7c236d82a3a78e7e4e2f
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Oct 20 15:15:52 2020 +0200

    Add tests on current configuration check for cooker instantiation

commit a2ffe238d0e59def15ac5dd5a41717f6a5ff63e8
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Oct 16 20:06:00 2020 +0200

    api.server: Add types and tests on configuration checks

See https://jenkins.softwareheritage.org/job/DVAU/job/tests-on-diff/29/ for more details.

ardumont retitled this revision from api.server: Add types and tests on configuration checks to Add types and tests on configuration checks.Wed, Oct 21, 6:24 PM
ardumont edited the summary of this revision. (Show Details)Thu, Oct 22, 11:55 AM

Build is green

Patch application report for D4295 (id=15297)

Rebasing onto 97a860997b...

Current branch diff-target is up to date.
Changes applied before test
commit eb868d0e690e9890172b699184bcffb80696fe89
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Oct 20 15:15:52 2020 +0200

    Add tests on current configuration check for cooker instantiation

commit e5e428481c76966cf253d914ca1276c7c9749e69
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Oct 16 20:06:00 2020 +0200

    api.server: Add types and tests on configuration checks

See https://jenkins.softwareheritage.org/job/DVAU/job/tests-on-diff/32/ for more details.

vlorentz accepted this revision.Thu, Oct 22, 2:34 PM
This revision is now accepted and ready to land.Thu, Oct 22, 2:34 PM