Skip to content

Commit 896ac8d

Browse files
authored
integration: fix TestClustersGet assertion after JSON formatter change (#5227)
Since #5170 the CLI's JSON output uses the standard `: ` separator (via `json.MarshalIndent`) rather than the compact `":"` form produced by the previous `nwidger/jsoncolor` dependency. `TestClustersGet`'s substring assertion still expected the compact form and has been failing in every nightly environment; updating it to match the new output. This pull request and its description were written by Isaac.
1 parent fec675f commit 896ac8d

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

integration/cmd/clusters/clusters_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ func TestClustersGet(t *testing.T) {
3232
clusterId := findValidClusterID(t)
3333
stdout, stderr := testcli.RequireSuccessfulRun(t, ctx, "clusters", "get", clusterId)
3434
outStr := stdout.String()
35-
assert.Contains(t, outStr, fmt.Sprintf(`"cluster_id":"%s"`, clusterId))
35+
assert.Contains(t, outStr, fmt.Sprintf(`"cluster_id": "%s"`, clusterId))
3636
assert.Equal(t, "", stderr.String())
3737
}
3838

0 commit comments

Comments
 (0)