Page MenuHomeSoftware Heritage

test_serializers: Move to pytest for that specific erratic assertion
ClosedPublic

Authored by ardumont on Jul 8 2020, 1:15 PM.

Details

Summary

Ideally we should refactor those entirely.

Should definitely fix the debian build [1]

[1] https://jenkins.softwareheritage.org/job/debian/job/packages/job/DCORE/job/gbp-buildpackage/118/console

rDCORE84c7319bc37abb5fd97af994a9527ae01218ee44 was not enough.

Got mislead by pytest's better raises function apparently. Although neither tox nor pytest pick it up... i don't get why but meh...
Let's unstuck the build.

Test Plan

tox

Diff Detail

Repository
rDCORE Foundations and core functionalities
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D3460 (id=12246)

Rebasing onto 84c7319bc3...

Current branch diff-target is up to date.
Changes applied before test
commit 737b21d3dd6641ca62f99fec4fd0185df6af72a6
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jul 8 13:13:31 2020 +0200

    test_serializers: Move to pytest for that specific erratic assertion
    
    Ideally we should refactor those entirely.
    
    Should definitely fix the debian build [1]
    
    [1] https://jenkins.softwareheritage.org/job/debian/job/packages/job/DCORE/job/gbp-buildpackage/118/console

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

This revision is now accepted and ready to land.Jul 8 2020, 1:21 PM

Ideally we should refactor those entirely.

See D3461