Test JSON marshal output directly without unmarshal

Compare marshaled JSON string directly to verify correct key names
and omitempty behavior, per reviewer feedback.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: kbukum1

internal/model/job_test.go

@@ -69,31 +69,10 @@ func TestAllowedUpdateTypesJSON(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	// Verify the actual JSON keys are correct by unmarshaling into a raw map
-	var raw map[string]any
-	if err := json.Unmarshal(data, &raw); err != nil {
-		t.Fatal(err)
-	}
-
-	if _, ok := raw["update-types"]; !ok {
-		t.Errorf("expected JSON key \"update-types\" to be present, got keys: %v", raw)
-	}
-	if _, ok := raw["dependency-name"]; !ok {
-		t.Errorf("expected JSON key \"dependency-name\" to be present, got keys: %v", raw)
-	}
-
-	types, ok := raw["update-types"].([]any)
-	if !ok {
-		t.Fatalf("expected update-types to be an array, got %T", raw["update-types"])
-	}
-	expectedTypes := []string{"version-update:semver-minor", "version-update:semver-patch"}
-	if len(types) != len(expectedTypes) {
-		t.Fatalf("expected %d update-types, got %d", len(expectedTypes), len(types))
-	}
-	for i, et := range expectedTypes {
-		if types[i] != et {
-			t.Errorf("update-types[%d]: expected %q, got %q", i, et, types[i])
-		}
+	// Verify marshaled JSON directly without using unmarshal
+	expected := `{"dependency-name":"rails","update-types":["version-update:semver-minor","version-update:semver-patch"]}`
+	if string(data) != expected {
+		t.Errorf("unexpected JSON output:\n  got:  %s\n  want: %s", string(data), expected)
 	}
 
 	// Verify omitempty: UpdateTypes should be absent when nil
@@ -102,12 +81,10 @@ func TestAllowedUpdateTypesJSON(t *testing.T) {
 	if err != nil {
 		t.Fatal(err)
 	}
-	var rawEmpty map[string]any
-	if err := json.Unmarshal(data, &rawEmpty); err != nil {
-		t.Fatal(err)
-	}
-	if _, ok := rawEmpty["update-types"]; ok {
-		t.Errorf("expected \"update-types\" to be omitted when nil, but it was present")
+
+	expectedEmpty := `{"dependency-name":"rails"}`
+	if string(data) != expectedEmpty {
+		t.Errorf("unexpected JSON output for empty UpdateTypes:\n  got:  %s\n  want: %s", string(data), expectedEmpty)
 	}
 }