From: Sebastiaan van Stijn Date: Mon, 4 Mar 2019 17:44:07 +0000 (+0100) Subject: Fix test for Go 1.12.x X-Git-Tag: archive/raspbian/18.09.9+dfsg1-5+rpi1~1^2^2~2 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=099bdd6f818ef8d931ad135f70bf7140f2582996;p=docker.io.git Fix test for Go 1.12.x After switching to Go 1.12, the format-string causes an error; ``` === Errors cli/config/config_test.go:154:3: Fatalf format %q has arg config of wrong type *github.com/docker/cli/cli/config/configfile.ConfigFile cli/config/config_test.go:217:3: Fatalf format %q has arg config of wrong type *github.com/docker/cli/cli/config/configfile.ConfigFile cli/config/config_test.go:253:3: Fatalf format %q has arg config of wrong type *github.com/docker/cli/cli/config/configfile.ConfigFile cli/config/config_test.go:288:3: Fatalf format %q has arg config of wrong type *github.com/docker/cli/cli/config/configfile.ConfigFile cli/config/config_test.go:435:3: Fatalf format %q has arg config of wrong type *github.com/docker/cli/cli/config/configfile.ConfigFile cli/config/config_test.go:448:3: Fatalf format %q has arg config of wrong type *github.com/docker/cli/cli/config/configfile.ConfigFile DONE 1115 tests, 2 skipped, 6 errors in 215.984s make: *** [Makefile:22: test-coverage] Error 2 Exited with code 2 ``` Signed-off-by: Sebastiaan van Stijn (cherry picked from commit d4877fb2259feac2c76762ccd3001999cb7f0d58) Signed-off-by: Sebastiaan van Stijn Gbp-Pq: Name test--cli-fix-golang12.patch --- diff --git a/cli/cli/config/config_test.go b/cli/cli/config/config_test.go index 11c8dd6d..ca28ce80 100644 --- a/cli/cli/config/config_test.go +++ b/cli/cli/config/config_test.go @@ -150,9 +150,8 @@ func TestOldValidAuth(t *testing.T) { // defaultIndexserver is https://index.docker.io/v1/ ac := config.AuthConfigs["https://index.docker.io/v1/"] - if ac.Username != "joejoe" || ac.Password != "hello" { - t.Fatalf("Missing data from parsing:\n%q", config) - } + assert.Equal(t, ac.Username, "joejoe") + assert.Equal(t, ac.Password, "hello") // Now save it and make sure it shows up in new form configStr := saveConfigAndValidateNewFormat(t, config, tmpHome) @@ -213,9 +212,8 @@ func TestOldJSON(t *testing.T) { assert.NilError(t, err) ac := config.AuthConfigs["https://index.docker.io/v1/"] - if ac.Username != "joejoe" || ac.Password != "hello" { - t.Fatalf("Missing data from parsing:\n%q", config) - } + assert.Equal(t, ac.Username, "joejoe") + assert.Equal(t, ac.Password, "hello") // Now save it and make sure it shows up in new form configStr := saveConfigAndValidateNewFormat(t, config, tmpHome) @@ -249,9 +247,8 @@ func TestNewJSON(t *testing.T) { assert.NilError(t, err) ac := config.AuthConfigs["https://index.docker.io/v1/"] - if ac.Username != "joejoe" || ac.Password != "hello" { - t.Fatalf("Missing data from parsing:\n%q", config) - } + assert.Equal(t, ac.Username, "joejoe") + assert.Equal(t, ac.Password, "hello") // Now save it and make sure it shows up in new form configStr := saveConfigAndValidateNewFormat(t, config, tmpHome) @@ -284,9 +281,8 @@ func TestNewJSONNoEmail(t *testing.T) { assert.NilError(t, err) ac := config.AuthConfigs["https://index.docker.io/v1/"] - if ac.Username != "joejoe" || ac.Password != "hello" { - t.Fatalf("Missing data from parsing:\n%q", config) - } + assert.Equal(t, ac.Username, "joejoe") + assert.Equal(t, ac.Password, "hello") // Now save it and make sure it shows up in new form configStr := saveConfigAndValidateNewFormat(t, config, tmpHome) @@ -431,10 +427,8 @@ func TestJSONReaderNoFile(t *testing.T) { assert.NilError(t, err) ac := config.AuthConfigs["https://index.docker.io/v1/"] - if ac.Username != "joejoe" || ac.Password != "hello" { - t.Fatalf("Missing data from parsing:\n%q", config) - } - + assert.Equal(t, ac.Username, "joejoe") + assert.Equal(t, ac.Password, "hello") } func TestOldJSONReaderNoFile(t *testing.T) { @@ -444,9 +438,8 @@ func TestOldJSONReaderNoFile(t *testing.T) { assert.NilError(t, err) ac := config.AuthConfigs["https://index.docker.io/v1/"] - if ac.Username != "joejoe" || ac.Password != "hello" { - t.Fatalf("Missing data from parsing:\n%q", config) - } + assert.Equal(t, ac.Username, "joejoe") + assert.Equal(t, ac.Password, "hello") } func TestJSONWithPsFormatNoFile(t *testing.T) {