From 868b3bdaeb9f38c94b9a4db9408b1076a2cc83f5 Mon Sep 17 00:00:00 2001
From: dvoraen <42825596+dvoraen@users.noreply.github.com>
Date: Thu, 1 Dec 2022 06:43:20 -0800
Subject: [PATCH] grouped coverage testing steps together

---
 .../workflows/github_action_test_suite.yml    | 24 +++++++++----------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/.github/workflows/github_action_test_suite.yml b/.github/workflows/github_action_test_suite.yml
index 8b38446fd6..b1343e8d2b 100644
--- a/.github/workflows/github_action_test_suite.yml
+++ b/.github/workflows/github_action_test_suite.yml
@@ -62,6 +62,18 @@ jobs:
           evennia migrate
           evennia collectstatic --noinput
 
+      # For non-coverage tests, run them in parallel.
+      - name: Run test suite
+        if: ! ${{ matrix.coverage-test }}
+        working-directory: testing_mygame
+        run: |
+          evennia test \
+            --settings=settings \
+            --keepdb \
+            --parallel 4 \
+            --timing \
+            evennia
+
       # OBS - it's important to not run the coverage tests with --parallel, it messes up the coverage
       # calculation!
       - name: Run test suite with coverage
@@ -79,18 +91,6 @@ jobs:
           coverage --version
           coverage report | grep TOTAL
 
-      # For other runs, run tests in parallel
-      - name: Run test suite
-        if: ! ${{ matrix.coverage-test }}
-        working-directory: testing_mygame
-        run: |
-          evennia test \
-            --settings=settings \
-            --keepdb \
-            --parallel 4 \
-            --timing \
-            evennia
-
       # we only want to run coverall once, so we only do it for the designated matrix combination(s)
       # it's also not critical if pushing to either service fails (happens for PRs since env is not
       # available outside of the evennia org)