Skip to content

Commit dfac32d

Browse files
committed
test: clean up schema test by removing debugging code
1 parent 6b5909e commit dfac32d

1 file changed

Lines changed: 1 addition & 12 deletions

File tree

internal/pkg/encoding/json/schema/schema_test.go

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1013,19 +1013,8 @@ func TestNormalizeSchema_OptionsDependencies_MissingDependencyField(t *testing.T
10131013
}
10141014
`)
10151015

1016-
// First, let's see what the normalized schema looks like
1017-
normalized, err := NormalizeSchema(schema)
1018-
require.NoError(t, err)
1019-
1020-
// Pretty print for debugging
1021-
var prettySchema map[string]any
1022-
json.Unmarshal(normalized, &prettySchema)
1023-
prettyBytes, _ := json.MarshalIndent(prettySchema, "", " ")
1024-
t.Logf("Normalized schema:\n%s", string(prettyBytes))
1025-
10261016
// Test 1: triggerActionOnFailure field is missing entirely (fire-and-forget config)
10271017
// This should PASS - actionOnFailureSettings should NOT be required
1028-
// BUG: Currently this fails with "missing property actionOnFailureSettings"
10291018
content1 := orderedmap.FromPairs([]orderedmap.Pair{
10301019
{
10311020
Key: "parameters",
@@ -1038,7 +1027,7 @@ func TestNormalizeSchema_OptionsDependencies_MissingDependencyField(t *testing.T
10381027
}),
10391028
},
10401029
})
1041-
err = ValidateContent(schema, content1)
1030+
err := ValidateContent(schema, content1)
10421031
require.NoError(t, err, "Should pass when triggerActionOnFailure field is missing (undefined)")
10431032

10441033
// Test 2: triggerActionOnFailure is explicitly false

0 commit comments

Comments
 (0)