diff --git a/integration-cli/docker_cli_by_digest_test.go b/integration-cli/docker_cli_by_digest_test.go index 9cc0073c6c..44466e717c 100644 --- a/integration-cli/docker_cli_by_digest_test.go +++ b/integration-cli/docker_cli_by_digest_test.go @@ -19,7 +19,7 @@ import ( var ( remoteRepoName = "dockercli/busybox-by-dgst" - repoName = fmt.Sprintf("%v/%s", privateRegistryURL, remoteRepoName) + repoName = fmt.Sprintf("%s/%s", privateRegistryURL, remoteRepoName) pushDigestRegex = regexp.MustCompile("[\\S]+: digest: ([\\S]+) size: [0-9]+") digestRegex = regexp.MustCompile("Digest: ([\\S]+)") ) diff --git a/integration-cli/docker_utils.go b/integration-cli/docker_utils.go index 4734c89711..b6177c5831 100644 --- a/integration-cli/docker_utils.go +++ b/integration-cli/docker_utils.go @@ -1567,7 +1567,7 @@ func setupRegistry(c *check.C, schema1 bool) *testRegistryV2 { time.Sleep(100 * time.Millisecond) } - c.Assert(err, check.IsNil, check.Commentf("Timeout waiting for test registry to become available")) + c.Assert(err, check.IsNil, check.Commentf("Timeout waiting for test registry to become available: %v", err)) return reg } diff --git a/integration-cli/registry.go b/integration-cli/registry.go index 26aebdf7b8..d4038d24b6 100644 --- a/integration-cli/registry.go +++ b/integration-cli/registry.go @@ -68,7 +68,7 @@ func (t *testRegistryV2) Ping() error { if err != nil { return err } - if resp.StatusCode != 200 { + if resp.StatusCode != http.StatusOK { return fmt.Errorf("registry ping replied with an unexpected status code %d", resp.StatusCode) } return nil diff --git a/integration-cli/registry_mock.go b/integration-cli/registry_mock.go index 5884eaae6c..300bf46425 100644 --- a/integration-cli/registry_mock.go +++ b/integration-cli/registry_mock.go @@ -37,7 +37,6 @@ func newTestRegistry(c *check.C) (*testRegistry, error) { matched, err = regexp.MatchString(re, url) if err != nil { c.Fatal("Error with handler regexp") - return } if matched { function(w, r)