add test case: build only

satotake · web-flow · commit 68a212511118 · 2021-06-21T12:25:58.000Z
Signed-off-by: GitHub &lt;noreply@github.com&gt;
diff --git a/local/e2e/compose/fixtures/dry-run-test/pull/compose.yaml b/local/e2e/compose/fixtures/dry-run-test/pull/compose.yaml
@@ -8,4 +8,7 @@ services:
   expectedFail:
     # this image is not expected to be registered on any registries
     image: expected-not-to-be-registered
+    command: top
+  expectedSkip-build:
+    build: Dockerfile
     command: top
diff --git a/pkg/compose/pull.go b/pkg/compose/pull.go
@@ -191,7 +191,10 @@ func getPullPlan(service types.ServiceConfig, localDigests []string, dstrDigest
 		}
 	}
 
-	if dstrDigest == "" {
+	if service.Image == "" {
+		// build only service
+		plan = pullPlanSkip
+	} else if dstrDigest == "" {
 		plan = pullPlanFail
 	} else if canSkip {
 		plan = pullPlanSkip

Original file line number	Diff line number	Diff line change
`@@ -191,7 +191,10 @@ func getPullPlan(service types.ServiceConfig, localDigests []string, dstrDigest`
`191`	`191`	`}`
`192`	`192`	`}`
`193`	`193`
`194`		`- if dstrDigest == "" {`
	`194`	`+ if service.Image == "" {`
	`195`	`+ // build only service`
	`196`	`+ plan = pullPlanSkip`
	`197`	`+ } else if dstrDigest == "" {`
`195`	`198`	`plan = pullPlanFail`
`196`	`199`	`} else if canSkip {`
`197`	`200`	`plan = pullPlanSkip`