fix: avoid panic in WithWorkload when metadata.name is missing

Signed-off-by: Ben Meier <[email protected]>
score-spec · May 9, 2024 · 9d313c7 · 9d313c7
1 parent 5d0b6f5
commit 9d313c7
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/framework/state.go b/framework/state.go
@@ -93,7 +93,11 @@ func (s *State[StateExtras, WorkloadExtras, ResourceExtras]) WithWorkload(spec *
 	} else {
 		out.Workloads = maps.Clone(s.Workloads)
 	}
-	out.Workloads[spec.Metadata["name"].(string)] = ScoreWorkloadState[WorkloadExtras]{
+	name, ok := spec.Metadata["name"].(string)
+	if !ok {
+		return nil, fmt.Errorf("metadata: name: is missing or is not a string")
+	}
+	out.Workloads[name] = ScoreWorkloadState[WorkloadExtras]{
 		Spec:   *spec,
 		File:   filePath,
 		Extras: extras,