Browse code

integ-cli: fix cleanup in build tests

Docker-DCO-1.1-Signed-off-by: Cristian Staretu <cristian.staretu@gmail.com> (github: unclejack)

unclejack authored on 2014/11/21 02:38:41
Showing 1 changed files
... ...
@@ -179,6 +179,7 @@ func TestBuildEnvironmentReplacementAddCopy(t *testing.T) {
179 179
 	if err != nil {
180 180
 		t.Fatal(err)
181 181
 	}
182
+	defer ctx.Close()
182 183
 
183 184
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
184 185
 		t.Fatal(err)
... ...
@@ -632,6 +633,8 @@ func TestBuildSixtySteps(t *testing.T) {
632 632
 	if err != nil {
633 633
 		t.Fatal(err)
634 634
 	}
635
+	defer ctx.Close()
636
+
635 637
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
636 638
 		t.Fatal(err)
637 639
 	}
... ...
@@ -656,6 +659,8 @@ RUN [ $(ls -l /exists | awk '{print $3":"$4}') = 'dockerio:dockerio' ]`,
656 656
 	if err != nil {
657 657
 		t.Fatal(err)
658 658
 	}
659
+	defer ctx.Close()
660
+
659 661
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
660 662
 		t.Fatal(err)
661 663
 	}
... ...
@@ -674,6 +679,8 @@ ADD test_file .`,
674 674
 	if err != nil {
675 675
 		t.Fatal(err)
676 676
 	}
677
+	defer ctx.Close()
678
+
677 679
 	done := make(chan struct{})
678 680
 	go func() {
679 681
 		if _, err := buildImageFromContext(name, ctx, true); err != nil {
... ...
@@ -708,6 +715,8 @@ RUN [ $(ls -l /exists/exists_file | awk '{print $3":"$4}') = 'dockerio:dockerio'
708 708
 	if err != nil {
709 709
 		t.Fatal(err)
710 710
 	}
711
+	defer ctx.Close()
712
+
711 713
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
712 714
 		t.Fatal(err)
713 715
 	}
... ...
@@ -947,6 +956,8 @@ RUN [ $(ls -l /exists | awk '{print $3":"$4}') = 'dockerio:dockerio' ]`,
947 947
 	if err != nil {
948 948
 		t.Fatal(err)
949 949
 	}
950
+	defer ctx.Close()
951
+
950 952
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
951 953
 		t.Fatal(err)
952 954
 	}
... ...
@@ -971,6 +982,8 @@ RUN [ $(ls -l /exists | awk '{print $3":"$4}') = 'dockerio:dockerio' ]`,
971 971
 	if err != nil {
972 972
 		t.Fatal(err)
973 973
 	}
974
+	defer ctx.Close()
975
+
974 976
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
975 977
 		t.Fatal(err)
976 978
 	}
... ...
@@ -996,6 +1009,8 @@ RUN [ $(ls -l /exists/test_file | awk '{print $3":"$4}') = 'root:root' ]`,
996 996
 	if err != nil {
997 997
 		t.Fatal(err)
998 998
 	}
999
+	defer ctx.Close()
1000
+
999 1001
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
1000 1002
 		t.Fatal(err)
1001 1003
 	}
... ...
@@ -1022,6 +1037,8 @@ RUN [ $(ls -l /exists | awk '{print $3":"$4}') = 'dockerio:dockerio' ]`,
1022 1022
 	if err != nil {
1023 1023
 		t.Fatal(err)
1024 1024
 	}
1025
+	defer ctx.Close()
1026
+
1025 1027
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
1026 1028
 		t.Fatal(err)
1027 1029
 	}
... ...
@@ -1040,6 +1057,8 @@ ADD . /`,
1040 1040
 	if err != nil {
1041 1041
 		t.Fatal(err)
1042 1042
 	}
1043
+	defer ctx.Close()
1044
+
1043 1045
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
1044 1046
 		t.Fatal(err)
1045 1047
 	}
... ...
@@ -1064,6 +1083,8 @@ RUN [ $(ls -l /exists | awk '{print $3":"$4}') = 'dockerio:dockerio' ]`,
1064 1064
 	if err != nil {
1065 1065
 		t.Fatal(err)
1066 1066
 	}
1067
+	defer ctx.Close()
1068
+
1067 1069
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
1068 1070
 		t.Fatal(err)
1069 1071
 	}
... ...
@@ -1082,6 +1103,8 @@ COPY test_file .`,
1082 1082
 	if err != nil {
1083 1083
 		t.Fatal(err)
1084 1084
 	}
1085
+	defer ctx.Close()
1086
+
1085 1087
 	done := make(chan struct{})
1086 1088
 	go func() {
1087 1089
 		if _, err := buildImageFromContext(name, ctx, true); err != nil {
... ...
@@ -1116,6 +1139,8 @@ RUN [ $(ls -l /exists/exists_file | awk '{print $3":"$4}') = 'dockerio:dockerio'
1116 1116
 	if err != nil {
1117 1117
 		t.Fatal(err)
1118 1118
 	}
1119
+	defer ctx.Close()
1120
+
1119 1121
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
1120 1122
 		t.Fatal(err)
1121 1123
 	}
... ...
@@ -1140,6 +1165,8 @@ RUN [ $(ls -l /exists | awk '{print $3":"$4}') = 'dockerio:dockerio' ]`,
1140 1140
 	if err != nil {
1141 1141
 		t.Fatal(err)
1142 1142
 	}
1143
+	defer ctx.Close()
1144
+
1143 1145
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
1144 1146
 		t.Fatal(err)
1145 1147
 	}
... ...
@@ -1163,6 +1190,8 @@ RUN [ $(ls -l /exists | awk '{print $3":"$4}') = 'dockerio:dockerio' ]`,
1163 1163
 	if err != nil {
1164 1164
 		t.Fatal(err)
1165 1165
 	}
1166
+	defer ctx.Close()
1167
+
1166 1168
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
1167 1169
 		t.Fatal(err)
1168 1170
 	}
... ...
@@ -1188,6 +1217,8 @@ RUN [ $(ls -l /exists/test_file | awk '{print $3":"$4}') = 'root:root' ]`,
1188 1188
 	if err != nil {
1189 1189
 		t.Fatal(err)
1190 1190
 	}
1191
+	defer ctx.Close()
1192
+
1191 1193
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
1192 1194
 		t.Fatal(err)
1193 1195
 	}
... ...
@@ -1214,6 +1245,8 @@ RUN [ $(ls -l /exists | awk '{print $3":"$4}') = 'dockerio:dockerio' ]`,
1214 1214
 	if err != nil {
1215 1215
 		t.Fatal(err)
1216 1216
 	}
1217
+	defer ctx.Close()
1218
+
1217 1219
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
1218 1220
 		t.Fatal(err)
1219 1221
 	}
... ...
@@ -1231,6 +1264,8 @@ COPY . /`,
1231 1231
 	if err != nil {
1232 1232
 		t.Fatal(err)
1233 1233
 	}
1234
+	defer ctx.Close()
1235
+
1234 1236
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
1235 1237
 		t.Fatal(err)
1236 1238
 	}
... ...
@@ -1858,6 +1893,7 @@ func TestBuildOnBuildLimitedInheritence(t *testing.T) {
1858 1858
 		if err != nil {
1859 1859
 			t.Fatal(err)
1860 1860
 		}
1861
+		defer ctx.Close()
1861 1862
 
1862 1863
 		out1, _, err := dockerCmdInDir(t, ctx.Dir, "build", "-t", name1, ".")
1863 1864
 		if err != nil {
... ...
@@ -1874,6 +1910,7 @@ func TestBuildOnBuildLimitedInheritence(t *testing.T) {
1874 1874
 		if err != nil {
1875 1875
 			t.Fatal(err)
1876 1876
 		}
1877
+		defer ctx.Close()
1877 1878
 
1878 1879
 		out2, _, err = dockerCmdInDir(t, ctx.Dir, "build", "-t", name2, ".")
1879 1880
 		if err != nil {
... ...
@@ -1890,6 +1927,7 @@ func TestBuildOnBuildLimitedInheritence(t *testing.T) {
1890 1890
 		if err != nil {
1891 1891
 			t.Fatal(err)
1892 1892
 		}
1893
+		defer ctx.Close()
1893 1894
 
1894 1895
 		out3, _, err = dockerCmdInDir(t, ctx.Dir, "build", "-t", name3, ".")
1895 1896
 		if err != nil {
... ...
@@ -2984,6 +3022,8 @@ RUN    [ "$(cat $TO)" = "hello" ]
2984 2984
 	if err != nil {
2985 2985
 		t.Fatal(err)
2986 2986
 	}
2987
+	defer ctx.Close()
2988
+
2987 2989
 	_, err = buildImageFromContext(name, ctx, true)
2988 2990
 	if err != nil {
2989 2991
 		t.Fatal(err)
... ...
@@ -3006,6 +3046,8 @@ RUN [ "$(cat /testfile)" = 'test!' ]`
3006 3006
 	if err != nil {
3007 3007
 		t.Fatal(err)
3008 3008
 	}
3009
+	defer ctx.Close()
3010
+
3009 3011
 	_, err = buildImageFromContext(name, ctx, true)
3010 3012
 	if err != nil {
3011 3013
 		t.Fatal(err)
... ...
@@ -3060,6 +3102,7 @@ RUN cat /existing-directory-trailing-slash/test/foo | grep Hi`
3060 3060
 		}
3061 3061
 		return &FakeContext{Dir: tmpDir}
3062 3062
 	}()
3063
+	defer ctx.Close()
3063 3064
 
3064 3065
 	if _, err := buildImageFromContext(name, ctx, true); err != nil {
3065 3066
 		t.Fatalf("build failed to complete for TestBuildAddTar: %v", err)