add slight delay before test screenshot so p5 has time for first render

shader-park · Sep 5, 2023 · 7242e6c · 7242e6c
1 parent 9e8a69e
commit 7242e6c
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 3 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -69,6 +69,7 @@
     "@rollup/plugin-node-resolve": "^15.2.1",
     "chai": "^4.3.8",
     "mocha": "^10.2.0",
+    "p5": "^1.7.0",
     "png-js": "^1.0.0",
     "puppeteer": "^21.1.0",
     "rollup": "^3.28.1",

diff --git a/test/p5Examples/basicP5Test.html b/test/p5Examples/basicP5Test.html
@@ -4,7 +4,7 @@
         <meta charset="utf-8" />
         <!-- keep the line below for OpenProcessing compatibility -->
         <!-- <script src="https://openprocessing.org/openprocessing_sketch.js"></script> -->
-        <script src="https://cdn.jsdelivr.net/npm/p5@1.7.0/lib/p5.js"></script>
+        <script src="../../node_modules/p5/lib/p5.js"></script>
         <script type="module" src="../../dist/shader-park-p5.js"></script>
         <script>
             let sdf;

diff --git a/test/render.js b/test/render.js
@@ -119,6 +119,8 @@ describe('Compiling, rendering, checking pixels', () => {
             logs.push(msg);
         });
         await page.goto(pagename);
+        // For some reason p5 hasn't rendered yet, so it needs extra time
+        await page.waitForTimeout(50);
         await page.screenshot({ path: outpath, fullPage: true });
         for (const perr of pageErrors) {
             assert.fail(`Page javascript error: ${perr}`);