Fix test errors

Signed-off-by: Marcela Melara <marcela.melara@intel.com>
in-toto · Dec 20, 2023 · 7a88a51 · 7a88a51
1 parent fcf6985
commit 7a88a51
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 11 deletions.
diff --git a/in_toto/attestation/predicates/provenance/v1/provenance.go b/in_toto/attestation/predicates/provenance/v1/provenance.go
@@ -6,9 +6,6 @@ import (
 
 	prov1 "github.com/in-toto/attestation/go/predicates/provenance/v1"
 	ita1 "github.com/in-toto/attestation/go/v1"
-	acommon "github.com/in-toto/in-toto-golang/in_toto/attestation/common"
-	alib1 "github.com/in-toto/in-toto-golang/in_toto/attestation/v1"
-	"github.com/in-toto/in-toto-golang/in_toto/slsa_provenance/common"
 
 	"google.golang.org/protobuf/types/known/structpb"
 	"google.golang.org/protobuf/types/known/timestamppb"
@@ -43,7 +40,7 @@ Validation is handled in GenProvenance().
 func GenRunDetails(builder *prov1.Builder, metadata *prov1.BuildMetadata, byproducts []*ita1.ResourceDescriptor) *prov1.RunDetails {
 	runDetails := &prov1.RunDetails{
 		Builder:       builder,
-		BuildMetadata: metadata,
+		Metadata:      metadata,
 		Byproducts:    byproducts,
 	}
 

diff --git a/in_toto/attestation/v1/resource_descriptor.go b/in_toto/attestation/v1/resource_descriptor.go
@@ -40,7 +40,7 @@ func RDListFromRecord(evalArtifacts map[string]map[string]string) ([]*ita1.Resou
 	var rds []*ita1.ResourceDescriptor
 	for name, digestSet := range evalArtifacts {
 
-		rd, err := GenResourceDescriptor(name, "", digestSet, nil, "", "", nil)
+		rd, err := GenerateValidResourceDescriptor(name, "", digestSet, nil, "", "", nil)
 
 		if err != nil {
 			return nil, err

diff --git a/in_toto/runlib.go b/in_toto/runlib.go
@@ -339,12 +339,13 @@ func InTotoRun(name string, runDir string, materialPaths []string, productPaths
 		return nil, err
 	}
 
-	var dssePayload any
+	// FIXME: replace when Attestation Framework support is fully implemented
+	var link Link
 	if attest {
 		// FIXME
 		return nil, errors.New("in-toto attestation support not implemented yet")
 	} else {
-		link := Link{
+		link = Link{
 			Type:        "link",
 			Name:        name,
 			Materials:   materials,
@@ -353,13 +354,11 @@ func InTotoRun(name string, runDir string, materialPaths []string, productPaths
 			Command:     cmdArgs,
 			Environment: map[string]interface{}{},
 		}
-
-		dssePayload = link
 	}
 
 	if useDSSE {
 		env := &Envelope{}
-		if err := env.SetPayload(dssePayload); err != nil {
+		if err := env.SetPayload(link); err != nil {
 			return nil, err
 		}
 		if !reflect.ValueOf(key).IsZero() {

diff --git a/in_toto/verifylib.go b/in_toto/verifylib.go
@@ -52,8 +52,9 @@ func RunInspections(layout Layout, runDir string, lineNormalization bool, useDSS
 			paths = []string{runDir}
 		}
 
+		// FIXME: The `attest` parameter is always false until full Attestation Framework support is added
 		linkEnv, err := InTotoRun(inspection.Name, runDir, paths, paths,
-			inspection.Run, Key{}, []string{"sha256"}, nil, nil, lineNormalization, false, useDSSE)
+			inspection.Run, Key{}, []string{"sha256"}, nil, nil, lineNormalization, false, useDSSE, false)
 
 		if err != nil {
 			return nil, err