Fix test for Go 1.12.x
authorSebastiaan van Stijn <github@gone.nl>
Mon, 4 Mar 2019 17:44:07 +0000 (18:44 +0100)
committerArnaud Rebillout <arnaud.rebillout@collabora.com>
Wed, 4 Sep 2019 07:54:29 +0000 (08:54 +0100)
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 <github@gone.nl>
(cherry picked from commit d4877fb2259feac2c76762ccd3001999cb7f0d58)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Gbp-Pq: Name test--cli-fix-golang12.patch

cli/cli/config/config_test.go

index 11c8dd6de5f290eca20a336ecbf127b03f499a2f..ca28ce800396079d860c9884d3fc2214c788abc5 100644 (file)
@@ -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) {