From 1a163a39cd0f167e169fc0a271a609c9a20f6a3c Mon Sep 17 00:00:00 2001
From: Jared O'Connell <46976761+jaredoconnell@users.noreply.github.com>
Date: Fri, 13 Sep 2024 11:40:18 -0400
Subject: [PATCH] Skip tests that require pulling container images (#209)

* Skip tests that require pulling container images

* Update skip message and remove comment
---
 engine_test.go | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/engine_test.go b/engine_test.go
index 9f167ac0..363ac7f3 100644
--- a/engine_test.go
+++ b/engine_test.go
@@ -148,6 +148,10 @@ output: []`),
 }
 
 func TestE2E(t *testing.T) {
+	if testing.Short() {
+		t.Skip("skipping e2e test in short mode due to container pulling")
+	}
+
 	content := map[string][]byte{
 		"workflow.yaml": []byte(`version: v0.2.0
 input:
@@ -184,6 +188,10 @@ output:
 }
 
 func TestE2EMultipleOutputs(t *testing.T) {
+	if testing.Short() {
+		t.Skip("skipping e2e test in short mode due to container pulling")
+	}
+
 	content := map[string][]byte{
 		"workflow.yaml": []byte(`version: v0.2.0
 input:
@@ -220,6 +228,10 @@ outputs:
 }
 
 func TestE2EWorkflowDefaultInput(t *testing.T) {
+	if testing.Short() {
+		t.Skip("skipping e2e test in short mode due to container pulling")
+	}
+
 	content := map[string][]byte{
 		"workflow.yaml": []byte(`version: v0.2.0
 input:
@@ -262,6 +274,10 @@ outputs:
 // referenced in the main workflow is incorporated into the
 // workflow's execution.
 func Test_CacheSubworkflows(t *testing.T) {
+	if testing.Short() {
+		t.Skip("skipping e2e test in short mode due to container pulling")
+	}
+
 	fileCache, err := loadfile.NewFileCacheUsingContext(
 		"fixtures/test-subworkflow",
 		map[string]string{