From 97d46a14eff5e0dc60bd5f81c8bac8280fe74ec6 Mon Sep 17 00:00:00 2001
From: Randy Fay <randy@randyfay.com>
Date: Mon, 28 Mar 2022 06:38:27 -0600
Subject: [PATCH] Fix problems with ddev e2e test and minor cleanup, add tmate
 (#27)

* Add tmate for debugging
* Use -parallel=1 for standaone tests

Signed-off-by: Randy Fay <randy@randyfay.com>
---
 .github/workflows/ci.yml | 13 +++++++++++++
 Makefile                 |  2 +-
 pkg/e2e/ddev_test.go     | 26 +++++++++++---------------
 3 files changed, 25 insertions(+), 16 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 500aeceb2..72d46e311 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -5,6 +5,12 @@ on:
     branches:
       - v2
   pull_request:
+  workflow_dispatch:
+    inputs:
+      debug_enabled:
+        description: 'To run with tmate enter "debug_enabled"'
+        required: false
+        default: "false"
 
 jobs:
   lint:
@@ -125,5 +131,12 @@ jobs:
           BUILD_TAGS: e2e
         run: make GIT_TAG=e2e-PR-${{ github.event.pull_request.number }}-${{ github.event.pull_request.head.sha }} -f builder.Makefile compose-plugin
 
+      - name: Setup tmate session
+        uses: mxschmitt/action-tmate@v3
+        with:
+          limit-access-to-actor: true
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+        if: ${{ github.event_name == 'workflow_dispatch' && github.event.inputs.debug_enabled }}
+
       - name: E2E Test in standalone mode
         run: make e2e-compose-standalone
diff --git a/Makefile b/Makefile
index feffbe315..be593999f 100644
--- a/Makefile
+++ b/Makefile
@@ -51,7 +51,7 @@ e2e-compose-standalone: ## Run End to end local tests in standalone mode. Set E2
 	rm -f /usr/local/bin/docker-compose
 	cp bin/docker-compose /usr/local/bin
 	docker-compose version
-	go test $(TEST_FLAGS) -count=1 --tags=standalone ./pkg/e2e
+	go test $(TEST_FLAGS) -v -count=1 -parallel=1 --tags=standalone ./pkg/e2e
 
 .PHONY: mocks
 mocks:
diff --git a/pkg/e2e/ddev_test.go b/pkg/e2e/ddev_test.go
index 7f92b74f3..bb1eaeeb0 100644
--- a/pkg/e2e/ddev_test.go
+++ b/pkg/e2e/ddev_test.go
@@ -27,7 +27,7 @@ import (
 	"gotest.tools/v3/assert"
 )
 
-const ddevVersion = "v1.18.2"
+const ddevVersion = "v1.19.1"
 
 func TestComposeRunDdev(t *testing.T) {
 	if !composeStandaloneMode {
@@ -36,10 +36,15 @@ func TestComposeRunDdev(t *testing.T) {
 	if runtime.GOOS == "windows" {
 		t.Skip("Running on Windows. Skipping...")
 	}
+	_ = os.Setenv("DDEV_DEBUG", "true")
+
 	c := NewParallelE2eCLI(t, binDir)
 	dir, err := os.MkdirTemp("", t.Name()+"-")
 	assert.NilError(t, err)
 
+	// ddev needs to be able to find mkcert to figure out where certs are.
+	_ = os.Setenv("PATH", fmt.Sprintf("%s:%s", os.Getenv("PATH"), dir))
+
 	siteName := filepath.Base(dir)
 
 	t.Cleanup(func() {
@@ -60,31 +65,22 @@ func TestComposeRunDdev(t *testing.T) {
 			compressedFilename))
 
 	c.RunCmdInDir(dir, "tar", "-xzf", compressedFilename)
-	c.RunDockerCmd("pull", "drud/ddev-ssh-agent:v1.18.0")
-	c.RunDockerCmd("pull", "busybox:stable")
-	c.RunDockerCmd("pull", "phpmyadmin:5")
-
-	c.RunDockerCmd("pull", tagged("drud/ddev-router"))
-	c.RunDockerCmd("pull", tagged("drud/ddev-dbserver-mariadb-10.3"))
-	c.RunDockerCmd("pull", tagged("drud/ddev-webserver"))
 
 	// Create a simple index.php we can test against.
 	c.RunCmdInDir(dir, "sh", "-c", "echo '<?php\nprint \"ddev is working\";' >index.php")
 
 	c.RunCmdInDir(dir, "./ddev", "config", "--auto")
 	c.RunCmdInDir(dir, "./ddev", "config", "global", "--use-docker-compose-from-path")
+	vRes := c.RunCmdInDir(dir, "./ddev", "version")
+	out := vRes.Stdout()
+	fmt.Printf("ddev version: %s\n", out)
 
 	c.RunCmdInDir(dir, "./ddev", "poweroff")
 
-	startRes := c.RunCmdInDir(dir, "./ddev", "start", "-y")
-	assert.Equal(c.test, startRes.ExitCode, 0, "Could not start project")
+	c.RunCmdInDir(dir, "./ddev", "start", "-y")
 
 	curlRes := c.RunCmdInDir(dir, "curl", "-sSL", fmt.Sprintf("http://%s.ddev.site", siteName))
-	out := curlRes.Stdout()
+	out = curlRes.Stdout()
 	fmt.Println(out)
 	assert.Assert(c.test, strings.Contains(out, "ddev is working"), "Could not start project")
 }
-
-func tagged(img string) string {
-	return fmt.Sprintf("%s:%s", img, ddevVersion)
-}