Page MenuHomeSoftware Heritage

journal: Add support for MetadataAuthority, MetadataFetcher, and RawExtrinsicMetadata.
ClosedPublic

Authored by vlorentz on Jul 30 2020, 5:36 PM.

Diff Detail

Unit TestsFailed

TimeTest
11,208 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.journal.tests.test_kafka_writer::test_kafka_writer
kafka_prefix = 'xxksuzbwwe', kafka_server = '127.0.0.1:41265' consumer = <cimpl.Consumer object at 0x7f0f1768e510> privileged_object_types = {'release', 'revision'}
11,189 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.journal.tests.test_kafka_writer::test_kafka_writer_anonymized
kafka_prefix = 'zrobibolxy', kafka_server = '127.0.0.1:41265' consumer = <cimpl.Consumer object at 0x7f0f1768e598> privileged_object_types = {'release', 'revision'}
2,125 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.journal.tests.test_client::test_client
2,256 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.journal.tests.test_client::test_client_batch_size[100]
2,164 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.journal.tests.test_client::test_client_batch_size[1]
View Full Test Results (2 Failed · 21 Passed)

Event Timeline

Build has FAILED

Patch application report for D3654 (id=12868)

Could not rebase; Attempt merge onto bf35ea79d3...

Updating bf35ea7..df96acf
Fast-forward
 swh/journal/pytest_plugin.py            |  17 +-
 swh/journal/serializers.py              |  48 +++-
 swh/journal/tests/journal_data.py       | 483 +++++++++++++++++---------------
 swh/journal/tests/test_pytest_plugin.py |   3 +
 4 files changed, 305 insertions(+), 246 deletions(-)
Changes applied before test
commit df96acf334b664def773d69fc1bf5e4510985670
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Jul 30 17:36:43 2020 +0200

    Add support for MetadataAuthority, MetadataFetcher, and RawExtrinsicMetadata.

commit 627f3ab8598d5a7cc90fe4b1388e8ee48222c41f
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Jul 30 16:56:47 2020 +0200

    Remove TEST_OBJECT_DICTS, use only TEST_OBJECTS.
    
    TEST_OBJECT_DICTS isn't used in any other package.

Link to build: https://jenkins.softwareheritage.org/job/DJNL/job/tests-on-diff/106/
See console output for more information: https://jenkins.softwareheritage.org/job/DJNL/job/tests-on-diff/106/console

ardumont added a subscriber: ardumont.
ardumont added inline comments.
swh/journal/serializers.py
75

why don't we use the object_.object_type here?

(I assume object_ is a BaseModel object).


Yeah, out of the scope of this new objects adding though.
Just checking if i'm on track with what we wanted to do with that fields in model.

This revision is now accepted and ready to land.Jul 31 2020, 10:28 AM
swh/journal/serializers.py
75

because it didn't exist when this code was written ^^

This revision was landed with ongoing or failed builds.Jul 31 2020, 10:45 AM
This revision was automatically updated to reflect the committed changes.