Browse code

Remove build 60 steps

This test is already covered in the individual graph driver tests and it
adds 15s to the test run without adding value. The original idea was to
test max number of layers, this is fulfilled by the graph drivers.

Signed-off-by: Michael Crosby <crosbymichael@gmail.com>

Michael Crosby authored on 2016/08/04 05:16:01
Showing 1 changed files
... ...
@@ -661,27 +661,6 @@ RUN ls -le /file`
661 661
 
662 662
 }
663 663
 
664
-func (s *DockerSuite) TestBuildSixtySteps(c *check.C) {
665
-	testRequires(c, DaemonIsLinux) // TODO Windows: This test passes on Windows,
666
-	// but currently adds a disproportionate amount of time for the value it has.
667
-	// Removing it from Windows CI for now, but this will be revisited in the
668
-	// TP5 timeframe when perf is better.
669
-	name := "foobuildsixtysteps"
670
-
671
-	ctx, err := fakeContext("FROM "+minimalBaseImage()+"\n"+strings.Repeat("ADD foo /\n", 60),
672
-		map[string]string{
673
-			"foo": "test1",
674
-		})
675
-	if err != nil {
676
-		c.Fatal(err)
677
-	}
678
-	defer ctx.Close()
679
-
680
-	if _, err := buildImageFromContext(name, ctx, true); err != nil {
681
-		c.Fatal(err)
682
-	}
683
-}
684
-
685 664
 func (s *DockerSuite) TestBuildAddSingleFileToRoot(c *check.C) {
686 665
 	testRequires(c, DaemonIsLinux) // Linux specific test
687 666
 	name := "testaddimg"