Enrich assertDecodeInfoPackage with (PackageId, Ast.Package) in the return type (#18165)

* needed to avoid double decoding of archives

run-all-tests: true
This commit is contained in:
tudor-da 2024-01-15 16:03:22 +01:00 committed by GitHub
parent 7be5899f5c
commit eda8187e7b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 7 additions and 5 deletions

View File

@ -43,7 +43,7 @@ object PackageMetadata {
}
def from(archive: DamlLf.Archive, priority: Priority): PackageMetadata = {
val packageInfo = Decode.assertDecodeInfoPackage(archive)
val packageInfo = Decode.assertDecodeInfoPackage(archive)._2
PackageMetadata(
templates = createVersionedTemplatesMap(packageInfo.definedTemplates, priority),
interfaces = packageInfo.definedInterfaces,

View File

@ -43,7 +43,7 @@ object PackageMetadata {
}
def from(archive: DamlLf.Archive, priority: Priority): PackageMetadata = {
val packageInfo = Decode.assertDecodeInfoPackage(archive)
val packageInfo = Decode.assertDecodeInfoPackage(archive)._2
PackageMetadata(
templates = createVersionedTemplatesMap(packageInfo.definedTemplates, priority),
interfaces = packageInfo.definedInterfaces,

View File

@ -58,11 +58,13 @@ object Decode {
): (PackageId, Ast.Package) =
assertRight(decodeArchive(archive, onlySerializableDataDefs))
def decodeInfoPackage(archive: DamlLf.Archive): Either[Error, PackageInfo] =
def decodeInfoPackage(
archive: DamlLf.Archive
): Either[Error, ((PackageId, Ast.Package), PackageInfo)] =
decodeArchive(archive, onlySerializableDataDefs = true)
.map(entry => new PackageInfo(Map(entry)))
.map(entry => entry -> new PackageInfo(Map(entry)))
def assertDecodeInfoPackage(archive: DamlLf.Archive): PackageInfo =
def assertDecodeInfoPackage(archive: DamlLf.Archive): ((PackageId, Ast.Package), PackageInfo) =
assertRight(decodeInfoPackage(archive: DamlLf.Archive))
}