Browse code

improve test

Docker-DCO-1.1-Signed-off-by: Victor Vieux <vieux@docker.com> (github: vieux)

Victor Vieux authored on 2014/06/17 09:59:51
Showing 1 changed files
... ...
@@ -25,6 +25,10 @@ func TestSaveAndLoadRepo(t *testing.T) {
25 25
 	out, _, err = runCommandWithOutput(commitCmd)
26 26
 	errorOut(err, t, fmt.Sprintf("failed to commit container: %v %v", out, err))
27 27
 
28
+	inspectCmd = exec.Command(dockerBinary, "inspect", repoName)
29
+	before, _, err := runCommandWithOutput(inspectCmd)
30
+	errorOut(err, t, fmt.Sprintf("the repo should exist before saving it: %v %v", before, err))
31
+
28 32
 	saveCmdTemplate := `%v save %v > /tmp/foobar-save-load-test.tar`
29 33
 	saveCmdFinal := fmt.Sprintf(saveCmdTemplate, dockerBinary, repoName)
30 34
 	saveCmd := exec.Command("bash", "-c", saveCmdFinal)
... ...
@@ -39,8 +43,12 @@ func TestSaveAndLoadRepo(t *testing.T) {
39 39
 	errorOut(err, t, fmt.Sprintf("failed to load repo: %v %v", out, err))
40 40
 
41 41
 	inspectCmd = exec.Command(dockerBinary, "inspect", repoName)
42
-	out, _, err = runCommandWithOutput(inspectCmd)
43
-	errorOut(err, t, fmt.Sprintf("the repo should exist after loading it: %v %v", out, err))
42
+	after, _, err := runCommandWithOutput(inspectCmd)
43
+	errorOut(err, t, fmt.Sprintf("the repo should exist after loading it: %v %v", after, err))
44
+
45
+	if before != after {
46
+		t.Fatalf("inspect is not the same after a save / load")
47
+	}
44 48
 
45 49
 	deleteContainer(cleanedContainerID)
46 50
 	deleteImages(repoName)