Skip to content

Commit 1019dcb

Browse files
apostasieapostasie
committed
Add Infinity constant to enable windows testing compat
Signed-off-by: apostasie <[email protected]>
1 parent cabd5bf commit 1019dcb

20 files changed

+58
-50
lines changed

‎cmd/nerdctl/container/container_commit_linux_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ func TestKubeCommitSave(t *testing.T) {
3636
// NOTE: kubectl namespaces are not the same as containerd namespaces.
3737
// We still want kube test objects segregated in their own Kube API namespace.
3838
nerdtest.KubeCtlCommand(helpers, "create", "namespace", "nerdctl-test-k8s").Run(&test.Expected{})
39-
nerdtest.KubeCtlCommand(helpers, "run", "--image", testutil.CommonImage, identifier, "--", "sleep", "Inf").Run(&test.Expected{})
39+
nerdtest.KubeCtlCommand(helpers, "run", "--image", testutil.CommonImage, identifier, "--", "sleep", nerdtest.Infinity).Run(&test.Expected{})
4040
nerdtest.KubeCtlCommand(helpers, "wait", "pod", identifier, "--for=condition=ready", "--timeout=1m").Run(&test.Expected{})
4141
nerdtest.KubeCtlCommand(helpers, "exec", identifier, "--", "mkdir", "-p", "/tmp/whatever").Run(&test.Expected{})
4242
nerdtest.KubeCtlCommand(helpers, "get", "pods", identifier, "-o", "jsonpath={ .status.containerStatuses[0].containerID }").Run(&test.Expected{

‎cmd/nerdctl/container/container_commit_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ func TestCommit(t *testing.T) {
3838
},
3939
Setup: func(data test.Data, helpers test.Helpers) {
4040
identifier := data.Identifier()
41-
helpers.Ensure("run", "-d", "--name", identifier, testutil.CommonImage, "sleep", "infinity")
41+
helpers.Ensure("run", "-d", "--name", identifier, testutil.CommonImage, "sleep", nerdtest.Infinity)
4242
helpers.Ensure("exec", identifier, "sh", "-euxc", `echo hello-test-commit > /foo`)
4343
},
4444
Command: func(data test.Data, helpers test.Helpers) test.TestableCommand {
@@ -63,7 +63,7 @@ func TestCommit(t *testing.T) {
6363
},
6464
Setup: func(data test.Data, helpers test.Helpers) {
6565
identifier := data.Identifier()
66-
helpers.Ensure("run", "-d", "--name", identifier, testutil.CommonImage, "sleep", "infinity")
66+
helpers.Ensure("run", "-d", "--name", identifier, testutil.CommonImage, "sleep", nerdtest.Infinity)
6767
nerdtest.EnsureContainerStarted(helpers, identifier)
6868
helpers.Ensure("exec", identifier, "sh", "-euxc", `echo hello-test-commit > /foo`)
6969
},

‎cmd/nerdctl/container/container_create_linux_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ func TestIssue2993(t *testing.T) {
200200
Setup: func(data test.Data, helpers test.Helpers) {
201201
dataRoot := data.TempDir()
202202

203-
helpers.Ensure("run", "--data-root", dataRoot, "--name", data.Identifier(), "-d", testutil.AlpineImage, "sleep", "infinity")
203+
helpers.Ensure("run", "--data-root", dataRoot, "--name", data.Identifier(), "-d", testutil.AlpineImage, "sleep", nerdtest.Infinity)
204204

205205
h := getAddrHash(defaults.DefaultAddress)
206206
dataStore := filepath.Join(dataRoot, h)
@@ -224,7 +224,7 @@ func TestIssue2993(t *testing.T) {
224224
helpers.Anyhow("rm", "--data-root", data.TempDir(), "-f", data.Identifier())
225225
},
226226
Command: func(data test.Data, helpers test.Helpers) test.TestableCommand {
227-
return helpers.Command("run", "--data-root", data.TempDir(), "--name", data.Identifier(), "-d", testutil.AlpineImage, "sleep", "infinity")
227+
return helpers.Command("run", "--data-root", data.TempDir(), "--name", data.Identifier(), "-d", testutil.AlpineImage, "sleep", nerdtest.Infinity)
228228
},
229229
Expected: func(data test.Data, helpers test.Helpers) *test.Expected {
230230
return &test.Expected{
@@ -247,7 +247,7 @@ func TestIssue2993(t *testing.T) {
247247
Setup: func(data test.Data, helpers test.Helpers) {
248248
dataRoot := data.TempDir()
249249

250-
helpers.Ensure("run", "--data-root", dataRoot, "--name", data.Identifier(), "-d", testutil.AlpineImage, "sleep", "infinity")
250+
helpers.Ensure("run", "--data-root", dataRoot, "--name", data.Identifier(), "-d", testutil.AlpineImage, "sleep", nerdtest.Infinity)
251251

252252
h := getAddrHash(defaults.DefaultAddress)
253253
dataStore := filepath.Join(dataRoot, h)

‎cmd/nerdctl/container/container_exec_linux_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ func TestExecWithUser(t *testing.T) {
2828
testContainer := testutil.Identifier(t)
2929

3030
defer base.Cmd("rm", "-f", testContainer).Run()
31-
base.Cmd("run", "-d", "--name", testContainer, testutil.CommonImage, "sleep", "infinity").AssertOK()
31+
base.Cmd("run", "-d", "--name", testContainer, testutil.CommonImage, "sleep", nerdtest.Infinity).AssertOK()
3232
base.EnsureContainerStarted(testContainer)
3333

3434
testCases := map[string]string{
@@ -59,7 +59,7 @@ func TestExecTTY(t *testing.T) {
5959

6060
testContainer := testutil.Identifier(t)
6161
defer base.Cmd("rm", "-f", testContainer).Run()
62-
base.Cmd("run", "-d", "--name", testContainer, testutil.CommonImage, "sleep", "infinity").AssertOK()
62+
base.Cmd("run", "-d", "--name", testContainer, testutil.CommonImage, "sleep", nerdtest.Infinity).AssertOK()
6363

6464
const sttyPartialOutput = "speed 38400 baud"
6565
// unbuffer(1) emulates tty, which is required by `nerdctl run -t`.

‎cmd/nerdctl/container/container_list_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ func TestContainerListWithFormatLabel(t *testing.T) {
3535
base.Cmd("run", "-d",
3636
"--name", cID,
3737
"--label", labelK+"="+labelV,
38-
testutil.CommonImage, "sleep", "infinity").AssertOK()
38+
testutil.CommonImage, "sleep", nerdtest.Infinity).AssertOK()
3939
defer base.Cmd("rm", "-f", cID).AssertOK()
4040
base.Cmd("ps", "-a",
4141
"--filter", "label="+labelK,
@@ -53,7 +53,7 @@ func TestContainerListWithJsonFormatLabel(t *testing.T) {
5353
base.Cmd("run", "-d",
5454
"--name", cID,
5555
"--label", labelK+"="+labelV,
56-
testutil.CommonImage, "sleep", "infinity").AssertOK()
56+
testutil.CommonImage, "sleep", nerdtest.Infinity).AssertOK()
5757
defer base.Cmd("rm", "-f", cID).AssertOK()
5858
base.Cmd("ps", "-a",
5959
"--filter", "label="+labelK,

‎cmd/nerdctl/container/container_prune_linux_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,9 @@ func TestPruneContainer(t *testing.T) {
3535
}
3636

3737
testCase.Setup = func(data test.Data, helpers test.Helpers) {
38-
helpers.Ensure("run", "-d", "--name", data.Identifier("1"), "-v", "/anonymous", testutil.CommonImage, "sleep", "infinity")
38+
helpers.Ensure("run", "-d", "--name", data.Identifier("1"), "-v", "/anonymous", testutil.CommonImage, "sleep", nerdtest.Infinity)
3939
helpers.Ensure("exec", data.Identifier("1"), "touch", "/anonymous/foo")
40-
helpers.Ensure("create", "--name", data.Identifier("2"), testutil.CommonImage, "sleep", "infinity")
40+
helpers.Ensure("create", "--name", data.Identifier("2"), testutil.CommonImage, "sleep", nerdtest.Infinity)
4141
}
4242

4343
testCase.Command = func(data test.Data, helpers test.Helpers) test.TestableCommand {

‎cmd/nerdctl/container/container_remove_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ func TestRemoveContainer(t *testing.T) {
2828
testCase := nerdtest.Setup()
2929

3030
testCase.Setup = func(data test.Data, helpers test.Helpers) {
31-
helpers.Ensure("run", "-d", "--name", data.Identifier(), testutil.CommonImage, "sleep", "inf")
31+
helpers.Ensure("run", "-d", "--name", data.Identifier(), testutil.CommonImage, "sleep", nerdtest.Infinity)
3232
}
3333

3434
testCase.Cleanup = func(data test.Data, helpers test.Helpers) {

‎cmd/nerdctl/container/container_rename_linux_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ func TestRename(t *testing.T) {
2828
base := testutil.NewBase(t)
2929

3030
defer base.Cmd("rm", "-f", testContainerName).Run()
31-
base.Cmd("run", "-d", "--name", testContainerName, testutil.CommonImage, "sleep", "infinity").AssertOK()
31+
base.Cmd("run", "-d", "--name", testContainerName, testutil.CommonImage, "sleep", nerdtest.Infinity).AssertOK()
3232

3333
defer base.Cmd("rm", "-f", testContainerName+"_new").Run()
3434
base.Cmd("rename", testContainerName, testContainerName+"_new").AssertOK()
@@ -44,11 +44,11 @@ func TestRenameUpdateHosts(t *testing.T) {
4444
base := testutil.NewBase(t)
4545

4646
defer base.Cmd("rm", "-f", testContainerName).Run()
47-
base.Cmd("run", "-d", "--name", testContainerName, testutil.CommonImage, "sleep", "infinity").AssertOK()
47+
base.Cmd("run", "-d", "--name", testContainerName, testutil.CommonImage, "sleep", nerdtest.Infinity).AssertOK()
4848
base.EnsureContainerStarted(testContainerName)
4949

5050
defer base.Cmd("rm", "-f", testContainerName+"_1").Run()
51-
base.Cmd("run", "-d", "--name", testContainerName+"_1", testutil.CommonImage, "sleep", "infinity").AssertOK()
51+
base.Cmd("run", "-d", "--name", testContainerName+"_1", testutil.CommonImage, "sleep", nerdtest.Infinity).AssertOK()
5252
base.EnsureContainerStarted(testContainerName + "_1")
5353

5454
defer base.Cmd("rm", "-f", testContainerName+"_new").Run()

‎cmd/nerdctl/container/container_rename_windows_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ func TestRenameProcessContainer(t *testing.T) {
2727
base := testutil.NewBase(t)
2828

2929
defer base.Cmd("rm", "-f", testContainerName).Run()
30-
base.Cmd("run", "--isolation", "process", "-d", "--name", testContainerName, testutil.CommonImage, "sleep", "infinity").AssertOK()
30+
base.Cmd("run", "--isolation", "process", "-d", "--name", testContainerName, testutil.CommonImage, "sleep", nerdtest.Infinity).AssertOK()
3131

3232
defer base.Cmd("rm", "-f", testContainerName+"_new").Run()
3333
base.Cmd("rename", testContainerName, testContainerName+"_new").AssertOK()
@@ -45,7 +45,7 @@ func TestRenameHyperVContainer(t *testing.T) {
4545
}
4646

4747
defer base.Cmd("rm", "-f", testContainerName).Run()
48-
base.Cmd("run", "--isolation", "hyperv", "-d", "--name", testContainerName, testutil.CommonImage, "sleep", "infinity").AssertOK()
48+
base.Cmd("run", "--isolation", "hyperv", "-d", "--name", testContainerName, testutil.CommonImage, "sleep", nerdtest.Infinity).AssertOK()
4949

5050
defer base.Cmd("rm", "-f", testContainerName+"_new").Run()
5151
base.Cmd("rename", testContainerName, testContainerName+"_new").AssertOK()

‎cmd/nerdctl/container/container_restart_linux_test.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,11 @@ func TestRestartPIDContainer(t *testing.T) {
5252
base := testutil.NewBase(t)
5353

5454
baseContainerName := testutil.Identifier(t)
55-
base.Cmd("run", "-d", "--name", baseContainerName, testutil.AlpineImage, "sleep", "infinity").AssertOK()
55+
base.Cmd("run", "-d", "--name", baseContainerName, testutil.AlpineImage, "sleep", nerdtest.Infinity).AssertOK()
5656
defer base.Cmd("rm", "-f", baseContainerName).Run()
5757

5858
sharedContainerName := fmt.Sprintf("%s-shared", baseContainerName)
59-
base.Cmd("run", "-d", "--name", sharedContainerName, fmt.Sprintf("--pid=container:%s", baseContainerName), testutil.AlpineImage, "sleep", "infinity").AssertOK()
59+
base.Cmd("run", "-d", "--name", sharedContainerName, fmt.Sprintf("--pid=container:%s", baseContainerName), testutil.AlpineImage, "sleep", nerdtest.Infinity).AssertOK()
6060
defer base.Cmd("rm", "-f", sharedContainerName).Run()
6161

6262
base.Cmd("restart", baseContainerName).AssertOK()
@@ -79,11 +79,11 @@ func TestRestartIPCContainer(t *testing.T) {
7979
const shmSize = "32m"
8080
baseContainerName := testutil.Identifier(t)
8181
defer base.Cmd("rm", "-f", baseContainerName).Run()
82-
base.Cmd("run", "-d", "--shm-size", shmSize, "--ipc", "shareable", "--name", baseContainerName, testutil.AlpineImage, "sleep", "infinity").AssertOK()
82+
base.Cmd("run", "-d", "--shm-size", shmSize, "--ipc", "shareable", "--name", baseContainerName, testutil.AlpineImage, "sleep", nerdtest.Infinity).AssertOK()
8383

8484
sharedContainerName := fmt.Sprintf("%s-shared", baseContainerName)
8585
defer base.Cmd("rm", "-f", sharedContainerName).Run()
86-
base.Cmd("run", "-d", "--name", sharedContainerName, fmt.Sprintf("--ipc=container:%s", baseContainerName), testutil.AlpineImage, "sleep", "infinity").AssertOK()
86+
base.Cmd("run", "-d", "--name", sharedContainerName, fmt.Sprintf("--ipc=container:%s", baseContainerName), testutil.AlpineImage, "sleep", nerdtest.Infinity).AssertOK()
8787

8888
base.Cmd("stop", baseContainerName).Run()
8989
base.Cmd("stop", sharedContainerName).Run()
@@ -104,7 +104,7 @@ func TestRestartWithTime(t *testing.T) {
104104
base := testutil.NewBase(t)
105105
tID := testutil.Identifier(t)
106106

107-
base.Cmd("run", "-d", "--name", tID, testutil.AlpineImage, "sleep", "infinity").AssertOK()
107+
base.Cmd("run", "-d", "--name", tID, testutil.AlpineImage, "sleep", nerdtest.Infinity).AssertOK()
108108
defer base.Cmd("rm", "-f", tID).AssertOK()
109109

110110
inspect := base.InspectContainer(tID)

0 commit comments

Comments
 (0)