Merge pull request #2623 from suyanhanx/remove-incompatible-in-compos…

…e-test chore(test): remove some incompatible in compose test
containerd · Nov 14, 2023 · 171d761 · 171d761
2 parents 462355d + d9d684b
commit 171d761
Show file tree

Hide file tree

Showing 8 changed files with 1 addition and 43 deletions.
diff --git a/cmd/nerdctl/compose_create_linux_test.go b/cmd/nerdctl/compose_create_linux_test.go
@@ -48,10 +48,6 @@ services:
 }
 
 func TestComposeCreateDependency(t *testing.T) {
-	// docker-compose v1 depecreated this command
-	// docker-compose v2 reimplemented this command
-	testutil.DockerIncompatible(t)
-
 	base := testutil.NewBase(t)
 	var dockerComposeYAML = fmt.Sprintf(`
 version: '3.1'
@@ -79,9 +75,6 @@ services:
 }
 
 func TestComposeCreatePull(t *testing.T) {
-	// docker-compose v1 depecreated this command
-	// docker-compose v2 reimplemented this command
-	testutil.DockerIncompatible(t)
 
 	base := testutil.NewBase(t)
 	var dockerComposeYAML = fmt.Sprintf(`
@@ -111,10 +104,6 @@ services:
 }
 
 func TestComposeCreateBuild(t *testing.T) {
-	// docker-compose v1 depecreated this command
-	// docker-compose v2 reimplemented this command
-	testutil.DockerIncompatible(t)
-
 	const imageSvc0 = "composebuild_svc0"
 
 	dockerComposeYAML := fmt.Sprintf(`

diff --git a/cmd/nerdctl/compose_down_linux_test.go b/cmd/nerdctl/compose_down_linux_test.go
@@ -28,9 +28,6 @@ import (
 func TestComposeDownRemoveUsedNetwork(t *testing.T) {
 	base := testutil.NewBase(t)
 
-	// The error output is different with docker
-	testutil.DockerIncompatible(t)
-
 	var (
 		dockerComposeYAMLOrphan = fmt.Sprintf(`
 version: '3.1'
@@ -60,7 +57,7 @@ services:
 	base.ComposeCmd("-p", projectName, "-f", compFull.YAMLFullPath(), "up", "-d").AssertOK()
 	defer base.ComposeCmd("-p", projectName, "-f", compFull.YAMLFullPath(), "down", "--remove-orphans").AssertOK()
 
-	base.ComposeCmd("-p", projectName, "-f", compOrphan.YAMLFullPath(), "down", "-v").AssertCombinedOutContains("is in use")
+	base.ComposeCmd("-p", projectName, "-f", compOrphan.YAMLFullPath(), "down", "-v").AssertCombinedOutContains("in use")
 
 }
 

diff --git a/cmd/nerdctl/compose_exec_linux_test.go b/cmd/nerdctl/compose_exec_linux_test.go
@@ -30,9 +30,6 @@ import (
 )
 
 func TestComposeExec(t *testing.T) {
-	// disabling `-it` in `compose exec` is only supported in compose v2.
-	// Currently CI is using compose v1.
-	testutil.DockerIncompatible(t)
 	base := testutil.NewBase(t)
 	var dockerComposeYAML = fmt.Sprintf(`
 version: '3.1'
@@ -62,9 +59,6 @@ services:
 }
 
 func TestComposeExecWithEnv(t *testing.T) {
-	// disabling `-it` in `compose exec` is only supported in compose v2.
-	// Currently CI is using compose v1.
-	testutil.DockerIncompatible(t)
 	base := testutil.NewBase(t)
 	var dockerComposeYAML = fmt.Sprintf(`
 version: '3.1'
@@ -131,9 +125,6 @@ services:
 }
 
 func TestComposeExecWithUser(t *testing.T) {
-	// disabling `-it` in `compose exec` is only supported in compose v2.
-	// Currently CI is using compose v1.
-	testutil.DockerIncompatible(t)
 	base := testutil.NewBase(t)
 	var dockerComposeYAML = fmt.Sprintf(`
 version: '3.1'

diff --git a/cmd/nerdctl/compose_images_linux_test.go b/cmd/nerdctl/compose_images_linux_test.go
@@ -78,10 +78,6 @@ volumes:
 }
 
 func TestComposeImagesJson(t *testing.T) {
-	// `--format` is only supported in docker compose v2.
-	// Currently, CI is using docker compose v1.
-	testutil.DockerIncompatible(t)
-
 	base := testutil.NewBase(t)
 	var dockerComposeYAML = fmt.Sprintf(`
 version: '3.1'

diff --git a/cmd/nerdctl/compose_kill_linux_test.go b/cmd/nerdctl/compose_kill_linux_test.go
@@ -25,10 +25,6 @@ import (
 )
 
 func TestComposeKill(t *testing.T) {
-	// docker-compose v2 hides exited/killed containers in `compose ps`, and shows
-	// them if `-a` is passed, which is not supported yet by `nerdctl compose`.
-	testutil.DockerIncompatible(t)
-
 	base := testutil.NewBase(t)
 	var dockerComposeYAML = fmt.Sprintf(`
 version: '3.1'

diff --git a/cmd/nerdctl/compose_port_linux_test.go b/cmd/nerdctl/compose_port_linux_test.go
@@ -52,10 +52,6 @@ services:
 }
 
 func TestComposePortFailure(t *testing.T) {
-	// when no port mapping is found, docker compose v1 prints `\n` while v2 prints `:0\n`
-	// both v1 and v2 have exit code 0 (succeess)
-	// nerdctl compose will fail with error (no public port found).
-	testutil.DockerIncompatible(t)
 	base := testutil.NewBase(t)
 
 	var dockerComposeYAML = fmt.Sprintf(`

diff --git a/cmd/nerdctl/compose_start_linux_test.go b/cmd/nerdctl/compose_start_linux_test.go
@@ -57,10 +57,6 @@ services:
 }
 
 func TestComposeStartFailWhenServicePause(t *testing.T) {
-	// Incompatible with docker compose v1. Currently CI is using compose v1.
-	// Starting a paused container triggers an error in v2 but is ignored in v1.
-	testutil.DockerIncompatible(t)
-
 	base := testutil.NewBase(t)
 	switch base.Info().CgroupDriver {
 	case "none", "":

diff --git a/cmd/nerdctl/compose_stop_linux_test.go b/cmd/nerdctl/compose_stop_linux_test.go
@@ -24,9 +24,6 @@ import (
 )
 
 func TestComposeStop(t *testing.T) {
-	// docker-compose v2 hides exited/killed containers in `compose ps`, and shows
-	// them if `-a` is passed, which is not supported yet by `nerdctl compose`.
-	testutil.DockerIncompatible(t)
 	base := testutil.NewBase(t)
 	var dockerComposeYAML = fmt.Sprintf(`
 version: '3.1'