Revert "Disable GPU acceleration in CI tests to mitigate flakiness"

This reverts commit 4690bb4.
marp-team · Jan 21, 2025 · 1c20a3f · 1c20a3f
1 parent a4cc000
commit 1c20a3f
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 13 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -99,7 +99,6 @@ commands:
           name: Jest
           command: npm run test:coverage -- --ci --reporters=default --reporters=jest-junit <<#parameters.runInBand>>-i<</parameters.runInBand>><<^parameters.runInBand>>--maxWorkers=2<</parameters.runInBand>>
           environment:
-            CHROME_DISABLE_GPU: 1 # Disable GPU acceleration to mitigate flaky tests
             JEST_JUNIT_CLASSNAME: '{filepath}'
             JEST_JUNIT_OUTPUT_DIR: tmp/test-results
             MARP_TEST_CI: 1

diff --git a/.github/workflows/test-win.yml b/.github/workflows/test-win.yml
@@ -48,7 +48,6 @@ jobs:
       # https://stackoverflow.com/a/59365905
       - name: Jest
         env:
-          CHROME_DISABLE_GPU: 1 # Disable GPU acceleration to mitigate flaky tests
           JEST_JUNIT_CLASSNAME: '{filepath}'
           JEST_JUNIT_OUTPUT_DIR: tmp/test-results
           MARP_TEST_CI: 1

diff --git a/test/browser/browsers/chrome.ts b/test/browser/browsers/chrome.ts
@@ -132,17 +132,6 @@ describe('ChromeBrowser', () => {
       })
 
       describe('Disabling GPU', () => {
-        it('does not add --disable-gpu argument if CHROME_DISABLE_GPU environment variable is not defined', async () => {
-          delete process.env.CHROME_DISABLE_GPU
-          await new ChromeBrowser({ path: '/path/to/chrome' }).launch()
-
-          expect(puppeteer.launch).toHaveBeenCalledWith(
-            expect.objectContaining({
-              args: expect.not.arrayContaining(['--disable-gpu']),
-            })
-          )
-        })
-
         it('adds --disable-gpu argument if CHROME_DISABLE_GPU environment variable is defined', async () => {
           process.env.CHROME_DISABLE_GPU = '1'
           await new ChromeBrowser({ path: '/path/to/chrome' }).launch()