(#25) Create defer functions the soonest

mdelapenya · Apr 16, 2020 · a07447a · a07447a
1 parent 3c26102
commit a07447a
Showing 1 changed file with 10 additions and 12 deletions.
diff --git a/compose_test.go b/compose_test.go
@@ -15,6 +15,11 @@ func TestLocalDockerCompose(t *testing.T) {
 	identifier := strings.ToLower(uuid.New().String())
 
 	compose := NewLocalDockerCompose([]string{path}, identifier)
+	destroyFn := func() {
+		err := compose.Down()
+		checkIfError(t, err)
+	}
+	defer destroyFn()
 
 	err := compose.
 		WithCommand([]string{"up", "-d"}).
@@ -26,12 +31,6 @@ func TestLocalDockerCompose(t *testing.T) {
 	}
 	absent := map[string]string{}
 	assertContainerEnvironmentVariables(t, compose.Identifier+"_nginx_1", present, absent)
-
-	destroyFn := func() {
-		err := compose.Down()
-		checkIfError(t, err)
-	}
-	defer destroyFn()
 }
 
 func TestLocalDockerComposeComplex(t *testing.T) {
@@ -40,17 +39,16 @@ func TestLocalDockerComposeComplex(t *testing.T) {
 	identifier := strings.ToLower(uuid.New().String())
 
 	compose := NewLocalDockerCompose([]string{path}, identifier)
-
-	err := compose.
-		WithCommand([]string{"up", "-d"}).
-		Invoke()
-	checkIfError(t, err)
-
 	destroyFn := func() {
 		err := compose.Down()
 		checkIfError(t, err)
 	}
 	defer destroyFn()
+
+	err := compose.
+		WithCommand([]string{"up", "-d"}).
+		Invoke()
+	checkIfError(t, err)
 }
 
 func TestLocalDockerComposeWithEnvironment(t *testing.T) {