dave/BigChef
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 34 Wiki Activity

Use diffs for tests

Browse Source
This commit is contained in:
PhatPhuckDave
2025-08-22 10:10:37 +02:00
parent 5b46ff0efd
commit 969ccae25c
1 changed files with 31 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
processor/surgical_json_test.go
View File

@@ -44,7 +44,7 @@ modified = true
expected: `{ expected: `{
"name": "test", "name": "test",
"value": 42 "value": 42
,"newField":"added"}`, // sjson.Set() adds new fields in compact format ,"newField": "added"}`, // sjson.Set() adds new fields in compact format
}, },
{ {
name: "Modify nested field", name: "Modify nested field",
@@ -94,6 +94,11 @@ modified = true
result = result[:cmd.From] + cmd.With + result[cmd.To:] result = result[:cmd.From] + cmd.With + result[cmd.To:]
} }
diff := cmp.Diff(result, tt.expected)
if diff != "" {
t.Errorf("Differences:\n%s", diff)
}
// Check the actual result matches expected // Check the actual result matches expected
if result != tt.expected { if result != tt.expected {
t.Errorf("Expected:\n%s\n\nGot:\n%s", tt.expected, result) t.Errorf("Expected:\n%s\n\nGot:\n%s", tt.expected, result)
@@ -180,6 +185,11 @@ modified = true
result = result[:cmd.From] + cmd.With + result[cmd.To:] result = result[:cmd.From] + cmd.With + result[cmd.To:]
} }
diff := cmp.Diff(result, expected)
if diff != "" {
t.Errorf("Differences:\n%s", diff)
}
// Check that the result matches expected (preserves formatting and changes weight) // Check that the result matches expected (preserves formatting and changes weight)
if result != expected { if result != expected {
t.Errorf("Expected:\n%s\n\nGot:\n%s", expected, result) t.Errorf("Expected:\n%s\n\nGot:\n%s", expected, result)
@@ -502,6 +512,11 @@ func TestSurgicalJSONPreservesFormatting2(t *testing.T) {
result = result[:cmd.From] + cmd.With + result[cmd.To:] result = result[:cmd.From] + cmd.With + result[cmd.To:]
} }
diff := cmp.Diff(result, expected)
if diff != "" {
t.Errorf("Differences:\n%s", diff)
}
// Check that the result matches expected (preserves formatting and changes weight) // Check that the result matches expected (preserves formatting and changes weight)
if result != expected { if result != expected {
t.Errorf("Expected:\n%s\n\nGot:\n%s", expected, result) t.Errorf("Expected:\n%s\n\nGot:\n%s", expected, result)
@@ -594,6 +609,10 @@ func TestRetardedJSONEditing(t *testing.T) {
result = result[:cmd.From] + cmd.With + result[cmd.To:] result = result[:cmd.From] + cmd.With + result[cmd.To:]
} }
diff := cmp.Diff(result, expected)
if diff != "" {
t.Errorf("Differences:\n%s", diff)
}
// Check that the weight was changed // Check that the weight was changed
if result != expected { if result != expected {
@@ -601,7 +620,6 @@ func TestRetardedJSONEditing(t *testing.T) {
} }
} }
func TestRetardedJSONEditing2(t *testing.T) { func TestRetardedJSONEditing2(t *testing.T) {
original := ` original := `
{ {
@@ -781,7 +799,6 @@ func TestRetardedJSONEditing2(t *testing.T) {
"ParentTags": [] "ParentTags": []
} }
,"AdditionalStats": {"(Value=\"BaseDeepMiningDrillSpeed_+%\")":4000}} ,"AdditionalStats": {"(Value=\"BaseDeepMiningDrillSpeed_+%\")":4000}}
}
] ]
} }
` `
@@ -819,9 +836,12 @@ func TestRetardedJSONEditing2(t *testing.T) {
result = result[:cmd.From] + cmd.With + result[cmd.To:] result = result[:cmd.From] + cmd.With + result[cmd.To:]
} }
diff := cmp.Diff(result, expected) diff := cmp.Diff(result, expected)
if diff != "" { if diff != "" {
t.Errorf("Differences:\n%s", diff) t.Errorf("Differences:\n%s", diff)
} }
if result != expected {
t.Errorf("Expected:\n%s\nGot:\n%s", expected, result)
}
} }