From 376b38c88a421a842503009fef14d578e81751b4 Mon Sep 17 00:00:00 2001
From: Erik Onarheim <erik.onarheim@gmail.com>
Date: Thu, 15 Feb 2024 21:15:44 -0600
Subject: [PATCH] fix: Flakey tests caused by Debug static

---
 karma.conf.js                   |  2 +-
 src/engine/Debug/DebugSystem.ts | 12 +++++++-----
 src/spec/DebugSpec.ts           |  3 +++
 src/spec/util/TestUtils.ts      |  1 +
 4 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/karma.conf.js b/karma.conf.js
index 3d77f5789..28e4a589e 100644
--- a/karma.conf.js
+++ b/karma.conf.js
@@ -182,7 +182,7 @@ module.exports = (config) => {
             '--disable-gpu',
             '--no-sandbox',
             '--enable-precise-memory-info',
-            '--js-flags="--max_old_space_size=8192 --expose-gc"'
+            '--js-flags="--max_old_space_size=8192"'
           ]
       },
       ChromiumHeadless_with_debug: {
diff --git a/src/engine/Debug/DebugSystem.ts b/src/engine/Debug/DebugSystem.ts
index 2462f595a..c61642a36 100644
--- a/src/engine/Debug/DebugSystem.ts
+++ b/src/engine/Debug/DebugSystem.ts
@@ -295,12 +295,14 @@ export class DebugSystem extends System {
   }
 
   postupdate(engine: Scene<unknown>, elapsedMs: number): void {
-    this._graphicsContext.save();
-    if (this._camera) {
-      this._camera.draw(this._graphicsContext);
+    if (this._engine.isDebug) {
+      this._graphicsContext.save();
+      if (this._camera) {
+        this._camera.draw(this._graphicsContext);
+      }
+      Debug.flush(this._graphicsContext);
+      this._graphicsContext.restore();
     }
-    Debug.flush(this._graphicsContext);
-    this._graphicsContext.restore();
   }
 
   /**
diff --git a/src/spec/DebugSpec.ts b/src/spec/DebugSpec.ts
index 9ba8f35c8..5ab98c91c 100644
--- a/src/spec/DebugSpec.ts
+++ b/src/spec/DebugSpec.ts
@@ -9,6 +9,9 @@ describe('Debug draw static', () => {
   it('exists', () => {
     expect(ex.Debug).toBeDefined();
   });
+  beforeEach(() => {
+    ex.Debug.clear();
+  });
 
   afterEach(() => {
     ex.Debug.clear();
diff --git a/src/spec/util/TestUtils.ts b/src/spec/util/TestUtils.ts
index e9d2092fc..968895bbd 100644
--- a/src/spec/util/TestUtils.ts
+++ b/src/spec/util/TestUtils.ts
@@ -20,6 +20,7 @@ export namespace TestUtils {
       displayMode: ex.DisplayMode.Fixed,
       ...options
     };
+    ex.Debug.clear();
     ex.Flags._reset();
     ex.Flags.enable('suppress-obsolete-message');
     flags.forEach(f => ex.Flags.enable(f));