Skip to content

Commit

Permalink
fixup! Engine: Improve visibility of TestConfigAllJsonResponse failures
Browse files Browse the repository at this point in the history
  • Loading branch information
gbjk committed Dec 7, 2024
1 parent 4d6815b commit c568878
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions engine/apiserver_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -276,10 +276,10 @@ func TestConfigAllJsonResponse(t *testing.T) {
// Using require here makes it much easier to isolate differences per-exchange than below
// We look into pointers separately
for a, p := range e.CurrencyPairs.Pairs {
require.Equalf(t, c.Exchanges[i].CurrencyPairs.Pairs[a], p, "%s exchange Config CurrencyManager Pairs for asset %s should match api response", e.Name, a)
require.Equalf(t, c.Exchanges[i].CurrencyPairs.Pairs[a], p, "%s exchange Config CurrencyManager Pairs for asset %s must match api response", e.Name, a)
}
require.Equalf(t, c.Exchanges[i].CurrencyPairs, e.CurrencyPairs, "%s exchange Config CurrencyManager should match api response", e.Name)
require.Equalf(t, c.Exchanges[i], e, "%s exchange Config should match api response", e.Name) // require here makes it much easier to isolate differences than below
require.Equalf(t, c.Exchanges[i].CurrencyPairs, e.CurrencyPairs, "%s exchange Config CurrencyManager must match api response", e.Name)
require.Equalf(t, c.Exchanges[i], e, "%s exchange Config must match api response", e.Name) // require here makes it much easier to isolate differences than below
}
assert.Equal(t, c, responseConfig, "Config should match api response")
}
Expand Down

0 comments on commit c568878

Please sign in to comment.