summaryrefslogtreecommitdiff
path: root/client
diff options
context:
space:
mode:
authorKirill Kolyshkin <kolyshkin@users.noreply.github.com>2019-10-17 11:38:00 -0700
committerGitHub <noreply@github.com>2019-10-17 11:38:00 -0700
commit0a96ee7ff8474c5d62eeded9d54adff7741a32e0 (patch)
tree58ca38bfe6c11389e634e4fae9f63e130da0a360 /client
parentf681590a2533829b88850b7e605a000cb538f102 (diff)
parentde10c7d0136dc054abdb40ca624211c04334bdc0 (diff)
downloaddocker-0a96ee7ff8474c5d62eeded9d54adff7741a32e0.tar.gz
Merge pull request #40080 from thaJeztah/client_string_matching
client: reduce string-matching in tests
Diffstat (limited to 'client')
-rw-r--r--client/checkpoint_create_test.go5
-rw-r--r--client/checkpoint_delete_test.go5
-rw-r--r--client/checkpoint_list_test.go5
-rw-r--r--client/config_create_test.go5
-rw-r--r--client/config_inspect_test.go5
-rw-r--r--client/config_list_test.go5
-rw-r--r--client/config_remove_test.go5
-rw-r--r--client/config_update_test.go5
-rw-r--r--client/container_commit_test.go5
-rw-r--r--client/container_copy_test.go15
-rw-r--r--client/container_create_test.go6
-rw-r--r--client/container_diff_test.go5
-rw-r--r--client/container_exec_test.go15
-rw-r--r--client/container_export_test.go5
-rw-r--r--client/container_inspect_test.go5
-rw-r--r--client/container_kill_test.go5
-rw-r--r--client/container_list_test.go5
-rw-r--r--client/container_logs_test.go5
-rw-r--r--client/container_pause_test.go5
-rw-r--r--client/container_prune_test.go5
-rw-r--r--client/container_remove_test.go5
-rw-r--r--client/container_rename_test.go5
-rw-r--r--client/container_resize_test.go10
-rw-r--r--client/container_restart_test.go5
-rw-r--r--client/container_start_test.go5
-rw-r--r--client/container_stats_test.go5
-rw-r--r--client/container_stop_test.go5
-rw-r--r--client/container_top_test.go5
-rw-r--r--client/container_unpause_test.go5
-rw-r--r--client/container_update_test.go5
-rw-r--r--client/container_wait_test.go5
-rw-r--r--client/disk_usage_test.go5
-rw-r--r--client/events_test.go5
-rw-r--r--client/image_build_test.go5
-rw-r--r--client/image_create_test.go5
-rw-r--r--client/image_history_test.go5
-rw-r--r--client/image_import_test.go5
-rw-r--r--client/image_inspect_test.go5
-rw-r--r--client/image_list_test.go5
-rw-r--r--client/image_load_test.go5
-rw-r--r--client/image_prune_test.go10
-rw-r--r--client/image_pull_test.go13
-rw-r--r--client/image_push_test.go13
-rw-r--r--client/image_remove_test.go5
-rw-r--r--client/image_save_test.go5
-rw-r--r--client/image_search_test.go13
-rw-r--r--client/image_tag_test.go5
-rw-r--r--client/info_test.go5
-rw-r--r--client/network_connect_test.go5
-rw-r--r--client/network_create_test.go5
-rw-r--r--client/network_disconnect_test.go5
-rw-r--r--client/network_inspect_test.go5
-rw-r--r--client/network_list_test.go5
-rw-r--r--client/network_prune_test.go5
-rw-r--r--client/network_remove_test.go5
-rw-r--r--client/node_inspect_test.go5
-rw-r--r--client/node_list_test.go5
-rw-r--r--client/node_remove_test.go5
-rw-r--r--client/node_update_test.go5
-rw-r--r--client/plugin_disable_test.go5
-rw-r--r--client/plugin_enable_test.go5
-rw-r--r--client/plugin_inspect_test.go5
-rw-r--r--client/plugin_list_test.go5
-rw-r--r--client/plugin_push_test.go5
-rw-r--r--client/plugin_remove_test.go5
-rw-r--r--client/plugin_set_test.go5
-rw-r--r--client/request_test.go5
-rw-r--r--client/secret_create_test.go5
-rw-r--r--client/secret_inspect_test.go5
-rw-r--r--client/secret_list_test.go5
-rw-r--r--client/secret_remove_test.go5
-rw-r--r--client/secret_update_test.go5
-rw-r--r--client/service_create_test.go5
-rw-r--r--client/service_inspect_test.go5
-rw-r--r--client/service_list_test.go5
-rw-r--r--client/service_logs_test.go5
-rw-r--r--client/service_remove_test.go5
-rw-r--r--client/service_update_test.go5
-rw-r--r--client/swarm_get_unlock_key_test.go5
-rw-r--r--client/swarm_init_test.go5
-rw-r--r--client/swarm_inspect_test.go5
-rw-r--r--client/swarm_join_test.go5
-rw-r--r--client/swarm_leave_test.go5
-rw-r--r--client/swarm_unlock_test.go5
-rw-r--r--client/swarm_update_test.go5
-rw-r--r--client/task_inspect_test.go5
-rw-r--r--client/task_list_test.go5
-rw-r--r--client/volume_create_test.go5
-rw-r--r--client/volume_inspect_test.go5
-rw-r--r--client/volume_list_test.go5
-rw-r--r--client/volume_remove_test.go5
91 files changed, 139 insertions, 371 deletions
diff --git a/client/checkpoint_create_test.go b/client/checkpoint_create_test.go
index 83ed92cb10..2d48ece150 100644
--- a/client/checkpoint_create_test.go
+++ b/client/checkpoint_create_test.go
@@ -23,11 +23,8 @@ func TestCheckpointCreateError(t *testing.T) {
Exit: true,
})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/checkpoint_delete_test.go b/client/checkpoint_delete_test.go
index 71daa31123..71c261bc2d 100644
--- a/client/checkpoint_delete_test.go
+++ b/client/checkpoint_delete_test.go
@@ -22,11 +22,8 @@ func TestCheckpointDeleteError(t *testing.T) {
CheckpointID: "checkpoint_id",
})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/checkpoint_list_test.go b/client/checkpoint_list_test.go
index 652040d4f8..f444a192ca 100644
--- a/client/checkpoint_list_test.go
+++ b/client/checkpoint_list_test.go
@@ -20,11 +20,8 @@ func TestCheckpointListError(t *testing.T) {
}
_, err := client.CheckpointList(context.Background(), "container_id", types.CheckpointListOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/config_create_test.go b/client/config_create_test.go
index f891cee8d2..1960b1736f 100644
--- a/client/config_create_test.go
+++ b/client/config_create_test.go
@@ -32,11 +32,8 @@ func TestConfigCreateError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ConfigCreate(context.Background(), swarm.ConfigSpec{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/config_inspect_test.go b/client/config_inspect_test.go
index ed247bea56..4dfee3e6a7 100644
--- a/client/config_inspect_test.go
+++ b/client/config_inspect_test.go
@@ -56,11 +56,8 @@ func TestConfigInspectError(t *testing.T) {
}
_, _, err := client.ConfigInspectWithRaw(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/config_list_test.go b/client/config_list_test.go
index 7b26a50ebc..ebaef1c0e2 100644
--- a/client/config_list_test.go
+++ b/client/config_list_test.go
@@ -34,11 +34,8 @@ func TestConfigListError(t *testing.T) {
}
_, err := client.ConfigList(context.Background(), types.ConfigListOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/config_remove_test.go b/client/config_remove_test.go
index 48930ddc25..15262269ce 100644
--- a/client/config_remove_test.go
+++ b/client/config_remove_test.go
@@ -30,11 +30,8 @@ func TestConfigRemoveError(t *testing.T) {
}
err := client.ConfigRemove(context.Background(), "config_id")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/config_update_test.go b/client/config_update_test.go
index d43e717e5a..d5111e6bbc 100644
--- a/client/config_update_test.go
+++ b/client/config_update_test.go
@@ -31,11 +31,8 @@ func TestConfigUpdateError(t *testing.T) {
}
err := client.ConfigUpdate(context.Background(), "config_id", swarm.Version{}, swarm.ConfigSpec{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_commit_test.go b/client/container_commit_test.go
index dd355404aa..e9cc9cbf9d 100644
--- a/client/container_commit_test.go
+++ b/client/container_commit_test.go
@@ -19,11 +19,8 @@ func TestContainerCommitError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerCommit(context.Background(), "nothing", types.ContainerCommitOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_copy_test.go b/client/container_copy_test.go
index 6154914387..1d737cb7e2 100644
--- a/client/container_copy_test.go
+++ b/client/container_copy_test.go
@@ -20,11 +20,8 @@ func TestContainerStatPathError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerStatPath(context.Background(), "container_id", "path")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
@@ -103,11 +100,8 @@ func TestCopyToContainerError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
err := client.CopyToContainer(context.Background(), "container_id", "path/to/file", bytes.NewReader([]byte("")), types.CopyToContainerOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
@@ -183,11 +177,8 @@ func TestCopyFromContainerError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, _, err := client.CopyFromContainer(context.Background(), "container_id", "path/to/file")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_create_test.go b/client/container_create_test.go
index 0a3d7e722f..3922ebc3be 100644
--- a/client/container_create_test.go
+++ b/client/container_create_test.go
@@ -19,9 +19,6 @@ func TestContainerCreateError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerCreate(context.Background(), nil, nil, nil, "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error while testing StatusInternalServerError, got %v", err)
- }
if !errdefs.IsSystem(err) {
t.Fatalf("expected a Server Error while testing StatusInternalServerError, got %T", err)
}
@@ -31,9 +28,6 @@ func TestContainerCreateError(t *testing.T) {
client: newMockClient(errorMock(http.StatusNotFound, "Server error")),
}
_, err = client.ContainerCreate(context.Background(), nil, nil, nil, "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error while testing StatusNotFound, got %v", err)
- }
if err == nil || !IsErrNotFound(err) {
t.Fatalf("expected a Server Error while testing StatusNotFound, got %T", err)
}
diff --git a/client/container_diff_test.go b/client/container_diff_test.go
index 310c4351b9..002267a362 100644
--- a/client/container_diff_test.go
+++ b/client/container_diff_test.go
@@ -19,11 +19,8 @@ func TestContainerDiffError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerDiff(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_exec_test.go b/client/container_exec_test.go
index 80f514371c..c4a19bd126 100644
--- a/client/container_exec_test.go
+++ b/client/container_exec_test.go
@@ -19,11 +19,8 @@ func TestContainerExecCreateError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerExecCreate(context.Background(), "container_id", types.ExecConfig{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
@@ -77,11 +74,8 @@ func TestContainerExecStartError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
err := client.ContainerExecStart(context.Background(), "nothing", types.ExecStartCheck{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
@@ -124,11 +118,8 @@ func TestContainerExecInspectError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerExecInspect(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_export_test.go b/client/container_export_test.go
index 2c054a93ab..e72a97a0b2 100644
--- a/client/container_export_test.go
+++ b/client/container_export_test.go
@@ -17,11 +17,8 @@ func TestContainerExportError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerExport(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_inspect_test.go b/client/container_inspect_test.go
index fd5fc3c6d2..26ffaa660f 100644
--- a/client/container_inspect_test.go
+++ b/client/container_inspect_test.go
@@ -21,11 +21,8 @@ func TestContainerInspectError(t *testing.T) {
}
_, err := client.ContainerInspect(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_kill_test.go b/client/container_kill_test.go
index 8d35a332bf..fcfe8e5153 100644
--- a/client/container_kill_test.go
+++ b/client/container_kill_test.go
@@ -17,11 +17,8 @@ func TestContainerKillError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
err := client.ContainerKill(context.Background(), "nothing", "SIGKILL")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_list_test.go b/client/container_list_test.go
index 6581469a55..f734e78f14 100644
--- a/client/container_list_test.go
+++ b/client/container_list_test.go
@@ -20,11 +20,8 @@ func TestContainerListError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerList(context.Background(), types.ContainerListOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_logs_test.go b/client/container_logs_test.go
index b610eb04f0..a12bbb97dc 100644
--- a/client/container_logs_test.go
+++ b/client/container_logs_test.go
@@ -14,6 +14,7 @@ import (
"time"
"github.com/docker/docker/api/types"
+ "github.com/docker/docker/errdefs"
"gotest.tools/assert"
is "gotest.tools/assert/cmp"
)
@@ -33,7 +34,9 @@ func TestContainerLogsError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerLogs(context.Background(), "container_id", types.ContainerLogsOptions{})
- assert.Check(t, is.Error(err, "Error response from daemon: Server error"))
+ if !errdefs.IsSystem(err) {
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
+ }
_, err = client.ContainerLogs(context.Background(), "container_id", types.ContainerLogsOptions{
Since: "2006-01-02TZ",
})
diff --git a/client/container_pause_test.go b/client/container_pause_test.go
index 33840aaea2..81ada2f3ee 100644
--- a/client/container_pause_test.go
+++ b/client/container_pause_test.go
@@ -17,11 +17,8 @@ func TestContainerPauseError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
err := client.ContainerPause(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_prune_test.go b/client/container_prune_test.go
index 6a830d01dc..147e904764 100644
--- a/client/container_prune_test.go
+++ b/client/container_prune_test.go
@@ -12,6 +12,7 @@ import (
"github.com/docker/docker/api/types"
"github.com/docker/docker/api/types/filters"
+ "github.com/docker/docker/errdefs"
"gotest.tools/assert"
is "gotest.tools/assert/cmp"
)
@@ -25,7 +26,9 @@ func TestContainersPruneError(t *testing.T) {
filters := filters.NewArgs()
_, err := client.ContainersPrune(context.Background(), filters)
- assert.Check(t, is.Error(err, "Error response from daemon: Server error"))
+ if !errdefs.IsSystem(err) {
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
+ }
}
func TestContainersPrune(t *testing.T) {
diff --git a/client/container_remove_test.go b/client/container_remove_test.go
index d94d831304..d144a8e084 100644
--- a/client/container_remove_test.go
+++ b/client/container_remove_test.go
@@ -10,6 +10,7 @@ import (
"testing"
"github.com/docker/docker/api/types"
+ "github.com/docker/docker/errdefs"
"gotest.tools/assert"
is "gotest.tools/assert/cmp"
)
@@ -19,7 +20,9 @@ func TestContainerRemoveError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
err := client.ContainerRemove(context.Background(), "container_id", types.ContainerRemoveOptions{})
- assert.Check(t, is.Error(err, "Error response from daemon: Server error"))
+ if !errdefs.IsSystem(err) {
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
+ }
}
func TestContainerRemoveNotFoundError(t *testing.T) {
diff --git a/client/container_rename_test.go b/client/container_rename_test.go
index cee9a0cee5..5abcd32bdf 100644
--- a/client/container_rename_test.go
+++ b/client/container_rename_test.go
@@ -17,11 +17,8 @@ func TestContainerRenameError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
err := client.ContainerRename(context.Background(), "nothing", "newNothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_resize_test.go b/client/container_resize_test.go
index c763e1ac01..b0aa259a06 100644
--- a/client/container_resize_test.go
+++ b/client/container_resize_test.go
@@ -18,11 +18,8 @@ func TestContainerResizeError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
err := client.ContainerResize(context.Background(), "container_id", types.ResizeOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
@@ -31,11 +28,8 @@ func TestContainerExecResizeError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
err := client.ContainerExecResize(context.Background(), "exec_id", types.ResizeOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_restart_test.go b/client/container_restart_test.go
index 81cac219df..29df115b11 100644
--- a/client/container_restart_test.go
+++ b/client/container_restart_test.go
@@ -19,11 +19,8 @@ func TestContainerRestartError(t *testing.T) {
}
timeout := 0 * time.Second
err := client.ContainerRestart(context.Background(), "nothing", &timeout)
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_start_test.go b/client/container_start_test.go
index 8981cdce1e..cb9376b680 100644
--- a/client/container_start_test.go
+++ b/client/container_start_test.go
@@ -19,11 +19,8 @@ func TestContainerStartError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
err := client.ContainerStart(context.Background(), "nothing", types.ContainerStartOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_stats_test.go b/client/container_stats_test.go
index df10641071..b52385296e 100644
--- a/client/container_stats_test.go
+++ b/client/container_stats_test.go
@@ -17,11 +17,8 @@ func TestContainerStatsError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerStats(context.Background(), "nothing", false)
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_stop_test.go b/client/container_stop_test.go
index 7d860aa575..0bb8c12157 100644
--- a/client/container_stop_test.go
+++ b/client/container_stop_test.go
@@ -19,11 +19,8 @@ func TestContainerStopError(t *testing.T) {
}
timeout := 0 * time.Second
err := client.ContainerStop(context.Background(), "nothing", &timeout)
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_top_test.go b/client/container_top_test.go
index ac0eec17b5..936888dabf 100644
--- a/client/container_top_test.go
+++ b/client/container_top_test.go
@@ -20,11 +20,8 @@ func TestContainerTopError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerTop(context.Background(), "nothing", []string{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_unpause_test.go b/client/container_unpause_test.go
index 7817b50b1e..f53816b714 100644
--- a/client/container_unpause_test.go
+++ b/client/container_unpause_test.go
@@ -17,11 +17,8 @@ func TestContainerUnpauseError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
err := client.ContainerUnpause(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_update_test.go b/client/container_update_test.go
index 8e47e489c0..b7b4423933 100644
--- a/client/container_update_test.go
+++ b/client/container_update_test.go
@@ -19,11 +19,8 @@ func TestContainerUpdateError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerUpdate(context.Background(), "nothing", container.UpdateConfig{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/container_wait_test.go b/client/container_wait_test.go
index 320567f12d..79d1e16d76 100644
--- a/client/container_wait_test.go
+++ b/client/container_wait_test.go
@@ -25,11 +25,8 @@ func TestContainerWaitError(t *testing.T) {
case result := <-resultC:
t.Fatalf("expected to not get a wait result, got %d", result.StatusCode)
case err := <-errC:
- if err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
}
diff --git a/client/disk_usage_test.go b/client/disk_usage_test.go
index 824c689f02..4166e8e2bd 100644
--- a/client/disk_usage_test.go
+++ b/client/disk_usage_test.go
@@ -19,11 +19,8 @@ func TestDiskUsageError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.DiskUsage(context.Background())
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/events_test.go b/client/events_test.go
index 50a885c069..601098fa22 100644
--- a/client/events_test.go
+++ b/client/events_test.go
@@ -53,11 +53,8 @@ func TestEventsErrorFromServer(t *testing.T) {
}
_, errs := client.Events(context.Background(), types.EventsOptions{})
err := <-errs
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_build_test.go b/client/image_build_test.go
index 353bc12eb1..0d4ea85c63 100644
--- a/client/image_build_test.go
+++ b/client/image_build_test.go
@@ -21,11 +21,8 @@ func TestImageBuildError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ImageBuild(context.Background(), nil, types.ImageBuildOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_create_test.go b/client/image_create_test.go
index 947727d462..d05fd80e11 100644
--- a/client/image_create_test.go
+++ b/client/image_create_test.go
@@ -18,11 +18,8 @@ func TestImageCreateError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ImageCreate(context.Background(), "reference", types.ImageCreateOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_history_test.go b/client/image_history_test.go
index ee0e3a35b2..286561f943 100644
--- a/client/image_history_test.go
+++ b/client/image_history_test.go
@@ -19,11 +19,8 @@ func TestImageHistoryError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ImageHistory(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_import_test.go b/client/image_import_test.go
index b3a7d4dea3..7827bddab8 100644
--- a/client/image_import_test.go
+++ b/client/image_import_test.go
@@ -19,11 +19,8 @@ func TestImageImportError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ImageImport(context.Background(), types.ImageImportSource{}, "image:tag", types.ImageImportOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_inspect_test.go b/client/image_inspect_test.go
index f72fae2a0e..bbaaa19010 100644
--- a/client/image_inspect_test.go
+++ b/client/image_inspect_test.go
@@ -22,11 +22,8 @@ func TestImageInspectError(t *testing.T) {
}
_, _, err := client.ImageInspectWithRaw(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_list_test.go b/client/image_list_test.go
index 418213f70d..b1182b9bc6 100644
--- a/client/image_list_test.go
+++ b/client/image_list_test.go
@@ -21,11 +21,8 @@ func TestImageListError(t *testing.T) {
}
_, err := client.ImageList(context.Background(), types.ImageListOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_load_test.go b/client/image_load_test.go
index 22fb1afb82..bdfd0f047f 100644
--- a/client/image_load_test.go
+++ b/client/image_load_test.go
@@ -18,11 +18,8 @@ func TestImageLoadError(t *testing.T) {
}
_, err := client.ImageLoad(context.Background(), nil, true)
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_prune_test.go b/client/image_prune_test.go
index 9b0839bb6c..7123898a86 100644
--- a/client/image_prune_test.go
+++ b/client/image_prune_test.go
@@ -10,6 +10,8 @@ import (
"strings"
"testing"
+ "github.com/docker/docker/errdefs"
+
"github.com/docker/docker/api/types"
"github.com/docker/docker/api/types/filters"
"gotest.tools/assert"
@@ -22,10 +24,10 @@ func TestImagesPruneError(t *testing.T) {
version: "1.25",
}
- filters := filters.NewArgs()
-
- _, err := client.ImagesPrune(context.Background(), filters)
- assert.Check(t, is.Error(err, "Error response from daemon: Server error"))
+ _, err := client.ImagesPrune(context.Background(), filters.NewArgs())
+ if !errdefs.IsSystem(err) {
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
+ }
}
func TestImagesPrune(t *testing.T) {
diff --git a/client/image_pull_test.go b/client/image_pull_test.go
index 0f98a4a128..03720ed456 100644
--- a/client/image_pull_test.go
+++ b/client/image_pull_test.go
@@ -31,11 +31,8 @@ func TestImagePullAnyError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ImagePull(context.Background(), "myimage", types.ImagePullOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
@@ -44,8 +41,8 @@ func TestImagePullStatusUnauthorizedError(t *testing.T) {
client: newMockClient(errorMock(http.StatusUnauthorized, "Unauthorized error")),
}
_, err := client.ImagePull(context.Background(), "myimage", types.ImagePullOptions{})
- if err == nil || err.Error() != "Error response from daemon: Unauthorized error" {
- t.Fatalf("expected an Unauthorized Error, got %v", err)
+ if !errdefs.IsUnauthorized(err) {
+ t.Fatalf("expected a Unauthorized Error, got %[1]T: %[1]v", err)
}
}
@@ -74,8 +71,8 @@ func TestImagePullWithUnauthorizedErrorAndAnotherUnauthorizedError(t *testing.T)
_, err := client.ImagePull(context.Background(), "myimage", types.ImagePullOptions{
PrivilegeFunc: privilegeFunc,
})
- if err == nil || err.Error() != "Error response from daemon: Unauthorized error" {
- t.Fatalf("expected an Unauthorized Error, got %v", err)
+ if !errdefs.IsUnauthorized(err) {
+ t.Fatalf("expected a Unauthorized Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_push_test.go b/client/image_push_test.go
index 48429ef5c9..59d9e0186b 100644
--- a/client/image_push_test.go
+++ b/client/image_push_test.go
@@ -36,11 +36,8 @@ func TestImagePushAnyError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ImagePush(context.Background(), "myimage", types.ImagePushOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
@@ -49,8 +46,8 @@ func TestImagePushStatusUnauthorizedError(t *testing.T) {
client: newMockClient(errorMock(http.StatusUnauthorized, "Unauthorized error")),
}
_, err := client.ImagePush(context.Background(), "myimage", types.ImagePushOptions{})
- if err == nil || err.Error() != "Error response from daemon: Unauthorized error" {
- t.Fatalf("expected an Unauthorized Error, got %v", err)
+ if !errdefs.IsUnauthorized(err) {
+ t.Fatalf("expected a Unauthorized Error, got %[1]T: %[1]v", err)
}
}
@@ -79,8 +76,8 @@ func TestImagePushWithUnauthorizedErrorAndAnotherUnauthorizedError(t *testing.T)
_, err := client.ImagePush(context.Background(), "myimage", types.ImagePushOptions{
PrivilegeFunc: privilegeFunc,
})
- if err == nil || err.Error() != "Error response from daemon: Unauthorized error" {
- t.Fatalf("expected an Unauthorized Error, got %v", err)
+ if !errdefs.IsUnauthorized(err) {
+ t.Fatalf("expected a Unauthorized Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_remove_test.go b/client/image_remove_test.go
index b3b02ebddb..3e64e61ec0 100644
--- a/client/image_remove_test.go
+++ b/client/image_remove_test.go
@@ -11,6 +11,7 @@ import (
"testing"
"github.com/docker/docker/api/types"
+ "github.com/docker/docker/errdefs"
"gotest.tools/assert"
is "gotest.tools/assert/cmp"
)
@@ -21,7 +22,9 @@ func TestImageRemoveError(t *testing.T) {
}
_, err := client.ImageRemove(context.Background(), "image_id", types.ImageRemoveOptions{})
- assert.Check(t, is.Error(err, "Error response from daemon: Server error"))
+ if !errdefs.IsSystem(err) {
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
+ }
}
func TestImageRemoveImageNotFound(t *testing.T) {
diff --git a/client/image_save_test.go b/client/image_save_test.go
index 128a040168..7e887f3870 100644
--- a/client/image_save_test.go
+++ b/client/image_save_test.go
@@ -18,11 +18,8 @@ func TestImageSaveError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ImageSave(context.Background(), []string{"nothing"})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_search_test.go b/client/image_search_test.go
index 184fe024e8..6e3619b5ea 100644
--- a/client/image_search_test.go
+++ b/client/image_search_test.go
@@ -21,11 +21,8 @@ func TestImageSearchAnyError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ImageSearch(context.Background(), "some-image", types.ImageSearchOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
@@ -34,8 +31,8 @@ func TestImageSearchStatusUnauthorizedError(t *testing.T) {
client: newMockClient(errorMock(http.StatusUnauthorized, "Unauthorized error")),
}
_, err := client.ImageSearch(context.Background(), "some-image", types.ImageSearchOptions{})
- if err == nil || err.Error() != "Error response from daemon: Unauthorized error" {
- t.Fatalf("expected an Unauthorized Error, got %v", err)
+ if !errdefs.IsUnauthorized(err) {
+ t.Fatalf("expected a Unauthorized Error, got %[1]T: %[1]v", err)
}
}
@@ -64,8 +61,8 @@ func TestImageSearchWithUnauthorizedErrorAndAnotherUnauthorizedError(t *testing.
_, err := client.ImageSearch(context.Background(), "some-image", types.ImageSearchOptions{
PrivilegeFunc: privilegeFunc,
})
- if err == nil || err.Error() != "Error response from daemon: Unauthorized error" {
- t.Fatalf("expected an Unauthorized Error, got %v", err)
+ if !errdefs.IsUnauthorized(err) {
+ t.Fatalf("expected a Unauthorized Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/image_tag_test.go b/client/image_tag_test.go
index 0ed6355335..2e820efcf6 100644
--- a/client/image_tag_test.go
+++ b/client/image_tag_test.go
@@ -18,11 +18,8 @@ func TestImageTagError(t *testing.T) {
}
err := client.ImageTag(context.Background(), "image_id", "repo:tag")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/info_test.go b/client/info_test.go
index 7bde0f316b..cbaad6e259 100644
--- a/client/info_test.go
+++ b/client/info_test.go
@@ -19,11 +19,8 @@ func TestInfoServerError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.Info(context.Background())
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/network_connect_test.go b/client/network_connect_test.go
index 032d8291be..75141b6a22 100644
--- a/client/network_connect_test.go
+++ b/client/network_connect_test.go
@@ -21,11 +21,8 @@ func TestNetworkConnectError(t *testing.T) {
}
err := client.NetworkConnect(context.Background(), "network_id", "container_id", nil)
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/network_create_test.go b/client/network_create_test.go
index 0fccd085ae..a6bb29bee6 100644
--- a/client/network_create_test.go
+++ b/client/network_create_test.go
@@ -20,11 +20,8 @@ func TestNetworkCreateError(t *testing.T) {
}
_, err := client.NetworkCreate(context.Background(), "mynetwork", types.NetworkCreate{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/network_disconnect_test.go b/client/network_disconnect_test.go
index 3814e4dbbe..19fec8b31b 100644
--- a/client/network_disconnect_test.go
+++ b/client/network_disconnect_test.go
@@ -20,11 +20,8 @@ func TestNetworkDisconnectError(t *testing.T) {
}
err := client.NetworkDisconnect(context.Background(), "network_id", "container_id", false)
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/network_inspect_test.go b/client/network_inspect_test.go
index e804d8dc22..24e732de86 100644
--- a/client/network_inspect_test.go
+++ b/client/network_inspect_test.go
@@ -12,6 +12,7 @@ import (
"github.com/docker/docker/api/types"
"github.com/docker/docker/api/types/network"
+ "github.com/docker/docker/errdefs"
"github.com/pkg/errors"
"gotest.tools/assert"
is "gotest.tools/assert/cmp"
@@ -23,7 +24,9 @@ func TestNetworkInspectError(t *testing.T) {
}
_, err := client.NetworkInspect(context.Background(), "nothing", types.NetworkInspectOptions{})
- assert.Check(t, is.Error(err, "Error response from daemon: Server error"))
+ if !errdefs.IsSystem(err) {
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
+ }
}
func TestNetworkInspectNotFoundError(t *testing.T) {
diff --git a/client/network_list_test.go b/client/network_list_test.go
index 3e3d094fe8..f0b9cc7c46 100644
--- a/client/network_list_test.go
+++ b/client/network_list_test.go
@@ -23,11 +23,8 @@ func TestNetworkListError(t *testing.T) {
_, err := client.NetworkList(context.Background(), types.NetworkListOptions{
Filters: filters.NewArgs(),
})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/network_prune_test.go b/client/network_prune_test.go
index 791ec1bc14..3feba2580f 100644
--- a/client/network_prune_test.go
+++ b/client/network_prune_test.go
@@ -26,11 +26,8 @@ func TestNetworksPruneError(t *testing.T) {
filters := filters.NewArgs()
_, err := client.NetworksPrune(context.Background(), filters)
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/network_remove_test.go b/client/network_remove_test.go
index df2c3b4479..a87e61f5ea 100644
--- a/client/network_remove_test.go
+++ b/client/network_remove_test.go
@@ -18,11 +18,8 @@ func TestNetworkRemoveError(t *testing.T) {
}
err := client.NetworkRemove(context.Background(), "network_id")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/node_inspect_test.go b/client/node_inspect_test.go
index e7164b8e41..a6306f4921 100644
--- a/client/node_inspect_test.go
+++ b/client/node_inspect_test.go
@@ -21,11 +21,8 @@ func TestNodeInspectError(t *testing.T) {
}
_, _, err := client.NodeInspectWithRaw(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/node_list_test.go b/client/node_list_test.go
index fb7b658518..f5295de25f 100644
--- a/client/node_list_test.go
+++ b/client/node_list_test.go
@@ -22,11 +22,8 @@ func TestNodeListError(t *testing.T) {
}
_, err := client.NodeList(context.Background(), types.NodeListOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/node_remove_test.go b/client/node_remove_test.go
index 5570773607..3c641b7009 100644
--- a/client/node_remove_test.go
+++ b/client/node_remove_test.go
@@ -19,11 +19,8 @@ func TestNodeRemoveError(t *testing.T) {
}
err := client.NodeRemove(context.Background(), "node_id", types.NodeRemoveOptions{Force: false})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/node_update_test.go b/client/node_update_test.go
index d51df959a5..864b50b1fb 100644
--- a/client/node_update_test.go
+++ b/client/node_update_test.go
@@ -19,11 +19,8 @@ func TestNodeUpdateError(t *testing.T) {
}
err := client.NodeUpdate(context.Background(), "node_id", swarm.Version{}, swarm.NodeSpec{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/plugin_disable_test.go b/client/plugin_disable_test.go
index 1b1fe1ba33..b7026de8de 100644
--- a/client/plugin_disable_test.go
+++ b/client/plugin_disable_test.go
@@ -19,11 +19,8 @@ func TestPluginDisableError(t *testing.T) {
}
err := client.PluginDisable(context.Background(), "plugin_name", types.PluginDisableOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/plugin_enable_test.go b/client/plugin_enable_test.go
index cd1945d751..6090cc5cc3 100644
--- a/client/plugin_enable_test.go
+++ b/client/plugin_enable_test.go
@@ -19,11 +19,8 @@ func TestPluginEnableError(t *testing.T) {
}
err := client.PluginEnable(context.Background(), "plugin_name", types.PluginEnableOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/plugin_inspect_test.go b/client/plugin_inspect_test.go
index 07aee67473..5914e3c8f1 100644
--- a/client/plugin_inspect_test.go
+++ b/client/plugin_inspect_test.go
@@ -21,11 +21,8 @@ func TestPluginInspectError(t *testing.T) {
}
_, _, err := client.PluginInspectWithRaw(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/plugin_list_test.go b/client/plugin_list_test.go
index 521f5e6d16..665af7d9dc 100644
--- a/client/plugin_list_test.go
+++ b/client/plugin_list_test.go
@@ -21,11 +21,8 @@ func TestPluginListError(t *testing.T) {
}
_, err := client.PluginList(context.Background(), filters.NewArgs())
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/plugin_push_test.go b/client/plugin_push_test.go
index b9292e4785..f7de99b86a 100644
--- a/client/plugin_push_test.go
+++ b/client/plugin_push_test.go
@@ -18,11 +18,8 @@ func TestPluginPushError(t *testing.T) {
}
_, err := client.PluginPush(context.Background(), "plugin_name", "")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/plugin_remove_test.go b/client/plugin_remove_test.go
index 249c5cd7b3..38389fa6a1 100644
--- a/client/plugin_remove_test.go
+++ b/client/plugin_remove_test.go
@@ -19,11 +19,8 @@ func TestPluginRemoveError(t *testing.T) {
}
err := client.PluginRemove(context.Background(), "plugin_name", types.PluginRemoveOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/plugin_set_test.go b/client/plugin_set_test.go
index ad8cdb7db5..c9d4220874 100644
--- a/client/plugin_set_test.go
+++ b/client/plugin_set_test.go
@@ -18,11 +18,8 @@ func TestPluginSetError(t *testing.T) {
}
err := client.PluginSet(context.Background(), "plugin_name", []string{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/request_test.go b/client/request_test.go
index 8656ddef68..8a14d1232e 100644
--- a/client/request_test.go
+++ b/client/request_test.go
@@ -86,11 +86,8 @@ func TestPlainTextError(t *testing.T) {
client: newMockClient(plainTextErrorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ContainerList(context.Background(), types.ContainerListOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/secret_create_test.go b/client/secret_create_test.go
index 09b4653036..e608ba8561 100644
--- a/client/secret_create_test.go
+++ b/client/secret_create_test.go
@@ -32,11 +32,8 @@ func TestSecretCreateError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.SecretCreate(context.Background(), swarm.SecretSpec{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/secret_inspect_test.go b/client/secret_inspect_test.go
index bc5ae38859..127379091e 100644
--- a/client/secret_inspect_test.go
+++ b/client/secret_inspect_test.go
@@ -33,11 +33,8 @@ func TestSecretInspectError(t *testing.T) {
}
_, _, err := client.SecretInspectWithRaw(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/secret_list_test.go b/client/secret_list_test.go
index 0460ceb850..6154405fc5 100644
--- a/client/secret_list_test.go
+++ b/client/secret_list_test.go
@@ -34,11 +34,8 @@ func TestSecretListError(t *testing.T) {
}
_, err := client.SecretList(context.Background(), types.SecretListOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/secret_remove_test.go b/client/secret_remove_test.go
index 3c0d7d27e7..b72a30bc40 100644
--- a/client/secret_remove_test.go
+++ b/client/secret_remove_test.go
@@ -30,11 +30,8 @@ func TestSecretRemoveError(t *testing.T) {
}
err := client.SecretRemove(context.Background(), "secret_id")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/secret_update_test.go b/client/secret_update_test.go
index 6d6a759ce3..7bebbb9996 100644
--- a/client/secret_update_test.go
+++ b/client/secret_update_test.go
@@ -31,11 +31,8 @@ func TestSecretUpdateError(t *testing.T) {
}
err := client.SecretUpdate(context.Background(), "secret_id", swarm.Version{}, swarm.SecretSpec{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/service_create_test.go b/client/service_create_test.go
index 4b80d92722..536b6b63ee 100644
--- a/client/service_create_test.go
+++ b/client/service_create_test.go
@@ -25,11 +25,8 @@ func TestServiceCreateError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ServiceCreate(context.Background(), swarm.ServiceSpec{}, types.ServiceCreateOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/service_inspect_test.go b/client/service_inspect_test.go
index dbf0371e37..f2264b15fd 100644
--- a/client/service_inspect_test.go
+++ b/client/service_inspect_test.go
@@ -22,11 +22,8 @@ func TestServiceInspectError(t *testing.T) {
}
_, _, err := client.ServiceInspectWithRaw(context.Background(), "nothing", types.ServiceInspectOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/service_list_test.go b/client/service_list_test.go
index da239999a8..1ab674d176 100644
--- a/client/service_list_test.go
+++ b/client/service_list_test.go
@@ -22,11 +22,8 @@ func TestServiceListError(t *testing.T) {
}
_, err := client.ServiceList(context.Background(), types.ServiceListOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/service_logs_test.go b/client/service_logs_test.go
index d8779e0603..8af5f31963 100644
--- a/client/service_logs_test.go
+++ b/client/service_logs_test.go
@@ -14,6 +14,7 @@ import (
"time"
"github.com/docker/docker/api/types"
+ "github.com/docker/docker/errdefs"
"gotest.tools/assert"
is "gotest.tools/assert/cmp"
)
@@ -23,7 +24,9 @@ func TestServiceLogsError(t *testing.T) {
client: newMockClient(errorMock(http.StatusInternalServerError, "Server error")),
}
_, err := client.ServiceLogs(context.Background(), "service_id", types.ContainerLogsOptions{})
- assert.Check(t, is.Error(err, "Error response from daemon: Server error"))
+ if !errdefs.IsSystem(err) {
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
+ }
_, err = client.ServiceLogs(context.Background(), "service_id", types.ContainerLogsOptions{
Since: "2006-01-02TZ",
})
diff --git a/client/service_remove_test.go b/client/service_remove_test.go
index f358a7b10c..a57f7bb274 100644
--- a/client/service_remove_test.go
+++ b/client/service_remove_test.go
@@ -9,6 +9,7 @@ import (
"strings"
"testing"
+ "github.com/docker/docker/errdefs"
"gotest.tools/assert"
is "gotest.tools/assert/cmp"
)
@@ -19,7 +20,9 @@ func TestServiceRemoveError(t *testing.T) {
}
err := client.ServiceRemove(context.Background(), "service_id")
- assert.Check(t, is.Error(err, "Error response from daemon: Server error"))
+ if !errdefs.IsSystem(err) {
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
+ }
}
func TestServiceRemoveNotFoundError(t *testing.T) {
diff --git a/client/service_update_test.go b/client/service_update_test.go
index bc6c6f7766..f8a25b5d6a 100644
--- a/client/service_update_test.go
+++ b/client/service_update_test.go
@@ -20,11 +20,8 @@ func TestServiceUpdateError(t *testing.T) {
}
_, err := client.ServiceUpdate(context.Background(), "service_id", swarm.Version{}, swarm.ServiceSpec{}, types.ServiceUpdateOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/swarm_get_unlock_key_test.go b/client/swarm_get_unlock_key_test.go
index 730df7b744..9a5785036a 100644
--- a/client/swarm_get_unlock_key_test.go
+++ b/client/swarm_get_unlock_key_test.go
@@ -11,6 +11,7 @@ import (
"testing"
"github.com/docker/docker/api/types"
+ "github.com/docker/docker/errdefs"
"gotest.tools/assert"
is "gotest.tools/assert/cmp"
)
@@ -21,7 +22,9 @@ func TestSwarmGetUnlockKeyError(t *testing.T) {
}
_, err := client.SwarmGetUnlockKey(context.Background())
- assert.Check(t, is.ErrorContains(err, "Error response from daemon: Server error"))
+ if !errdefs.IsSystem(err) {
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
+ }
}
func TestSwarmGetUnlockKey(t *testing.T) {
diff --git a/client/swarm_init_test.go b/client/swarm_init_test.go
index a1acc69ff2..d929af7d86 100644
--- a/client/swarm_init_test.go
+++ b/client/swarm_init_test.go
@@ -19,11 +19,8 @@ func TestSwarmInitError(t *testing.T) {
}
_, err := client.SwarmInit(context.Background(), swarm.InitRequest{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/swarm_inspect_test.go b/client/swarm_inspect_test.go
index 46996ee10c..91a6a7395b 100644
--- a/client/swarm_inspect_test.go
+++ b/client/swarm_inspect_test.go
@@ -20,11 +20,8 @@ func TestSwarmInspectError(t *testing.T) {
}
_, err := client.SwarmInspect(context.Background())
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/swarm_join_test.go b/client/swarm_join_test.go
index 9cd516c477..e22c852d8f 100644
--- a/client/swarm_join_test.go
+++ b/client/swarm_join_test.go
@@ -19,11 +19,8 @@ func TestSwarmJoinError(t *testing.T) {
}
err := client.SwarmJoin(context.Background(), swarm.JoinRequest{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/swarm_leave_test.go b/client/swarm_leave_test.go
index 36e4fe4003..0417381ff6 100644
--- a/client/swarm_leave_test.go
+++ b/client/swarm_leave_test.go
@@ -18,11 +18,8 @@ func TestSwarmLeaveError(t *testing.T) {
}
err := client.SwarmLeave(context.Background(), false)
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/swarm_unlock_test.go b/client/swarm_unlock_test.go
index b01eb07e18..2d41bbb451 100644
--- a/client/swarm_unlock_test.go
+++ b/client/swarm_unlock_test.go
@@ -19,11 +19,8 @@ func TestSwarmUnlockError(t *testing.T) {
}
err := client.SwarmUnlock(context.Background(), swarm.UnlockRequest{UnlockKey: "SWMKEY-1-y6guTZNTwpQeTL5RhUfOsdBdXoQjiB2GADHSRJvbXeU"})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/swarm_update_test.go b/client/swarm_update_test.go
index 98059f1738..5076f6f0c1 100644
--- a/client/swarm_update_test.go
+++ b/client/swarm_update_test.go
@@ -19,11 +19,8 @@ func TestSwarmUpdateError(t *testing.T) {
}
err := client.SwarmUpdate(context.Background(), swarm.Version{}, swarm.Spec{}, swarm.UpdateFlags{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/task_inspect_test.go b/client/task_inspect_test.go
index 04da8883dd..3252e16267 100644
--- a/client/task_inspect_test.go
+++ b/client/task_inspect_test.go
@@ -21,11 +21,8 @@ func TestTaskInspectError(t *testing.T) {
}
_, _, err := client.TaskInspectWithRaw(context.Background(), "nothing")
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/task_list_test.go b/client/task_list_test.go
index f8fc99fd34..0116f735b4 100644
--- a/client/task_list_test.go
+++ b/client/task_list_test.go
@@ -22,11 +22,8 @@ func TestTaskListError(t *testing.T) {
}
_, err := client.TaskList(context.Background(), types.TaskListOptions{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/volume_create_test.go b/client/volume_create_test.go
index f4078c2f00..f782362406 100644
--- a/client/volume_create_test.go
+++ b/client/volume_create_test.go
@@ -21,11 +21,8 @@ func TestVolumeCreateError(t *testing.T) {
}
_, err := client.VolumeCreate(context.Background(), volumetypes.VolumeCreateBody{})
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/volume_inspect_test.go b/client/volume_inspect_test.go
index 804411cf2d..b78a9e05df 100644
--- a/client/volume_inspect_test.go
+++ b/client/volume_inspect_test.go
@@ -11,6 +11,7 @@ import (
"testing"
"github.com/docker/docker/api/types"
+ "github.com/docker/docker/errdefs"
"github.com/pkg/errors"
"gotest.tools/assert"
is "gotest.tools/assert/cmp"
@@ -22,7 +23,9 @@ func TestVolumeInspectError(t *testing.T) {
}
_, err := client.VolumeInspect(context.Background(), "nothing")
- assert.Check(t, is.ErrorContains(err, "Error response from daemon: Server error"))
+ if !errdefs.IsSystem(err) {
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
+ }
}
func TestVolumeInspectNotFound(t *testing.T) {
diff --git a/client/volume_list_test.go b/client/volume_list_test.go
index bdba3e0846..ccfb845b47 100644
--- a/client/volume_list_test.go
+++ b/client/volume_list_test.go
@@ -22,11 +22,8 @@ func TestVolumeListError(t *testing.T) {
}
_, err := client.VolumeList(context.Background(), filters.NewArgs())
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}
diff --git a/client/volume_remove_test.go b/client/volume_remove_test.go
index 73d5a2ddfd..ccc4fcd45d 100644
--- a/client/volume_remove_test.go
+++ b/client/volume_remove_test.go
@@ -18,11 +18,8 @@ func TestVolumeRemoveError(t *testing.T) {
}
err := client.VolumeRemove(context.Background(), "volume_id", false)
- if err == nil || err.Error() != "Error response from daemon: Server error" {
- t.Fatalf("expected a Server Error, got %v", err)
- }
if !errdefs.IsSystem(err) {
- t.Fatalf("expected a Server Error, got %T", err)
+ t.Fatalf("expected a Server Error, got %[1]T: %[1]v", err)
}
}