diff --git a/swh/loader/package/deposit/loader.py b/swh/loader/package/deposit/loader.py --- a/swh/loader/package/deposit/loader.py +++ b/swh/loader/package/deposit/loader.py @@ -85,16 +85,6 @@ date = TimestampWithTimezone.from_dict(revision_data["date"]) # commit_date: codemeta:datePublished if any, deposit completed_date otherwise commit_date = TimestampWithTimezone.from_dict(revision_data["committer_date"]) - metadata = revision_data["metadata"] - metadata.update( - { - "extrinsic": { - "provider": self.client.metadata_url(self.deposit_id), - "when": self.visit_date.isoformat(), - "raw": a_metadata, - }, - } - ) return Revision( type=RevisionType.TAR, @@ -106,7 +96,13 @@ parents=[hash_to_bytes(p) for p in revision_data.get("parents", [])], directory=directory, synthetic=True, - metadata=metadata, + metadata={ + "extrinsic": { + "provider": self.client.metadata_url(self.deposit_id), + "when": self.visit_date.isoformat(), + "raw": a_metadata, + }, + }, ) def load(self) -> Dict: diff --git a/swh/loader/package/deposit/tests/test_deposit.py b/swh/loader/package/deposit/tests/test_deposit.py --- a/swh/loader/package/deposit/tests/test_deposit.py +++ b/swh/loader/package/deposit/tests/test_deposit.py @@ -128,6 +128,9 @@ ], ) + # Only 2 top-level keys now + assert set(revision["metadata"].keys()) == {"extrinsic", "original_artifact"} + for original_artifact in revision["metadata"]["original_artifact"]: check_metadata_paths( original_artifact,