model: Fix hypothesis integration with attr < 21.3.0
ClosedPublic
Actions

Authored by anlambert on Oct 17 2022, 5:30 PM.

Details

Reviewers

vlorentz

Group Reviewers

Reviewers

Commits

rDMODdd3bab81af29: model: Fix hypothesis integration with attr < 21.3.0

Summary

When using attr < 21.3.0, adding field transformer breaks attrs
integration with hypothesis, because attributes transformed with
such function are not casted to generated AttrsClass, but remains
just an list of attributes. This causes error in hypothesis by
raising an AttributeError.

As we use attr 21.2.0 in production and when building debian buster
package, add a workaround for that issue as explained here:
https://github.com/python-attrs/attrs/issues/821.

This fixes debian buster package build for swh-model.

Diff Detail

Repository

rDMOD Data model

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

anlambert created this revision.Oct 17 2022, 5:30 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptOct 17 2022, 5:30 PM

Build is green

Patch application report for D8688 (id=31380)

Rebasing onto 13e7adc3e8...

Current branch diff-target is up to date.

Changes applied before test

commit dd3bab81af296c8ac621213df2345680b53fbd09
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Mon Oct 17 17:22:01 2022 +0200

    model: Fix hypothesis integration with attr < 21.3.0
    
    When using attr < 21.3.0, adding field transformer breaks attrs
    integration with hypothesis, because attributes transformed with
    such function are not casted to generated AttrsClass, but remains
    just an list of attributes. This causes error in hypothesis by
    raising an AttributeError.
    
    As we use attr 21.2.0 in production and when building debian buster
    package, add a workaround for that issue as explained here:
    https://github.com/python-attrs/attrs/issues/821.

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

Harbormaster completed remote builds in B32327: Diff 31380.Oct 17 2022, 5:32 PM

anlambert requested review of this revision.Oct 17 2022, 5:32 PM

vlorentz accepted this revision.Oct 17 2022, 5:56 PM

This revision is now accepted and ready to land.Oct 17 2022, 5:56 PM

Closed by commit rDMODdd3bab81af29: model: Fix hypothesis integration with attr < 21.3.0 (authored by anlambert). · Explain WhyOct 17 2022, 7:05 PM

This revision was automatically updated to reflect the committed changes.

anlambert added a commit: rDMODdd3bab81af29: model: Fix hypothesis integration with attr < 21.3.0.