From a4997cf7a0ae6b8b2c6d75d09f068c13bd5879c0 Mon Sep 17 00:00:00 2001
From: Michael Welles <mlwelles@gmail.com>
Date: Thu, 4 Jun 2026 16:00:04 -0400
Subject: [PATCH] ci: drop redundant Dgraph standalone from -short unit job

The unit-test job runs `go test -short`, which skips every test that needs
a live Dgraph. Standing up a dgraph/standalone container (and setting
MODUSGRAPH_TEST_ADDR) therefore adds setup the job never uses. Remove both;
the integration and load suites keep their own dedicated jobs.
---
 .github/workflows/ci-go-unit-tests.yaml | 17 -----------------
 1 file changed, 17 deletions(-)

diff --git a/.github/workflows/ci-go-unit-tests.yaml b/.github/workflows/ci-go-unit-tests.yaml
index 3623538..31878ab 100644
--- a/.github/workflows/ci-go-unit-tests.yaml
+++ b/.github/workflows/ci-go-unit-tests.yaml
@@ -39,22 +39,5 @@ jobs:
           go-version: 1.25.0
           cache-dependency-path: go.sum
 
-      - name: Set up Dgraph
-        if: matrix.os == 'linux'
-        run: |
-          docker run -d --name dgraph-standalone -p 9080:9080 -p 8080:8080 dgraph/standalone:latest
-          echo "Waiting for Dgraph to be ready..."
-          for i in {1..30}; do
-            if curl -s http://localhost:8080/health > /dev/null; then
-              echo "Dgraph is ready!"
-              break
-            fi
-            echo "Attempt $i: Dgraph not ready, waiting..."
-            sleep 2
-          done
-          sleep 5
-
       - name: Run Unit Tests
-        env:
-          MODUSGRAPH_TEST_ADDR: ${{ matrix.os == 'linux' && 'localhost:9080' || '' }}
         run: go test -short -race -v .