nullstack · Mortaro · Jan 30, 2022 · Jan 20, 2022 · Jan 20, 2022 · Jan 20, 2022
diff --git a/.github/workflows/pr-tests.yml b/.github/workflows/pr-tests.yml
@@ -0,0 +1,51 @@
+# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
+
+name: Nullstack Tests
+
+on:
+  pull_request:
+    branches: [ master, next ]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+    container: lironavon/docker-puppeteer-container:14.16.0
+    env:
+      CI: true
+
+    strategy:
+      matrix:
+        node-version: [12.20.0, 14.x]
+        # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
+
+    steps:
+      - uses: actions/checkout@v2
+      # cache the dependencies from any node_modules directory
+      - name: Cache dependencies
+        uses: actions/cache@v2
+        with:
+          path: |
+            **/node_modules
+            **/package-lock.json
+          key: node_modules-${{ matrix.node-version }}-${{ hashFiles('**/package.json') }}
+
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v2
+        with:
+          node-version: ${{ matrix.node-version }}
+
+      - name: Install and link main deps
+        run: |
+          npm install
+          npm link
+
+      - name: Install deps at tests folder
+        working-directory: ./tests
+        run: |
+          npm link nullstack
+          npm install
+
+      - name: Run tests
+        working-directory: ./tests
+        run: npm test
diff --git a/tests/jest-puppeteer.config.js b/tests/jest-puppeteer.config.js
@@ -0,0 +1,28 @@
+// get environment variable
+const CI = !!process.env.CI;
+
+const baseOptions = {
+  server: {
+    command: 'npm run start',
+    port: 6969,
+    launchTimeout: 25000
+  },
+  browserContext: 'incognito'
+}
+
+const ciPipelineOptions = {
+  launch: {
+    executablePath: '/usr/bin/google-chrome-stable',
+    headless: true,
+    args: [
+      '--ignore-certificate-errors',
+      '--no-sandbox',
+      '--disable-setuid-sandbox',
+      '--disable-accelerated-2d-canvas',
+      '--disable-gpu'
+    ]
+  },
+  ...baseOptions
+}
+
+module.exports = CI ? ciPipelineOptions : baseOptions;
diff --git a/tests/jest.config.js b/tests/jest.config.js
@@ -0,0 +1,7 @@
+const CI = !!process.env.CI;
+
+module.exports = {
+  preset: "jest-puppeteer",
+  forceExit: CI,
+  testTimeout: CI ? 5000 : 20000
+}
diff --git a/tests/package.json b/tests/package.json
@@ -8,12 +8,13 @@
     "jest": "^26.6.3",
     "nullstack": "*",
     "puppeteer": "^5.5.0",
+    "jest-puppeteer": "^6.0.3",
     "webpack-cli": "^3.3.12"
   },
   "scripts": {
     "start": "npx nullstack start --input=./tests --port=6969",
     "build": "npx nullstack build --input=./tests --mode=ssr",
-    "test": "npm run build && jest --testTimeout=20000",
+    "test": "npm run build && jest",
     "script": "node src/scripts/run.js"
   },
   "dependencies": {

diff --git a/tests/src/Application.test.js b/tests/src/Application.test.js
@@ -1,13 +1,3 @@
-const puppeteer = require('puppeteer');
-
-let browser;
-let page;
-
-beforeAll(async () => {
-  browser = await puppeteer.launch();
-  page = await browser.newPage();
-});
-
 describe('Application', () => {
 
   test('the application is running', async () => {
@@ -32,8 +22,4 @@ describe('Application', () => {
     expect(element).toBeTruthy();
   });
 
-});
-
-afterAll(async () => {
-  browser.close();
 });
diff --git a/tests/src/ChildComponent.test.js b/tests/src/ChildComponent.test.js
@@ -1,11 +1,4 @@
-const puppeteer = require('puppeteer');
-
-let browser;
-let page;
-
 beforeAll(async () => {
-  browser = await puppeteer.launch();
-  page = await browser.newPage();
   await page.goto('http://localhost:6969/child-component');
 });
 
@@ -43,9 +36,4 @@ describe('ChildComponent', () => {
     expect(element).toBeTruthy();
   });
 
-
-});
-
-afterAll(async () => {
-  browser.close();
 });
diff --git a/tests/src/Context.test.js b/tests/src/Context.test.js
@@ -1,11 +1,4 @@
-const puppeteer = require('puppeteer');
-
-let browser;
-let page;
-
 beforeAll(async () => {
-  browser = await puppeteer.launch();
-  page = await browser.newPage();
   await page.goto('http://localhost:6969/context');
 });
 
@@ -36,8 +29,4 @@ describe('Context', () => {
     expect(element).toBeTruthy();
   });
 
-});
-
-afterAll(async () => {
-  browser.close();
 });
diff --git a/tests/src/ContextData.test.js b/tests/src/ContextData.test.js
@@ -1,11 +1,4 @@
-const puppeteer = require('puppeteer');
-
-let browser;
-let page;
-
 beforeAll(async () => {
-  browser = await puppeteer.launch();
-  page = await browser.newPage();
   await page.goto('http://localhost:6969/context-data');
 });
 
@@ -28,8 +21,4 @@ describe('ContextData', () => {
     expect(element).toBeTruthy();
   });
 
-});
-
-afterAll(async () => {
-  browser.close();
 });
diff --git a/tests/src/ContextEnvironment.test.js b/tests/src/ContextEnvironment.test.js
@@ -1,11 +1,4 @@
-const puppeteer = require('puppeteer');
-
-let browser;
-let page;
-
 beforeAll(async () => {
-  browser = await puppeteer.launch();
-  page = await browser.newPage();
   await page.goto('http://localhost:6969/context-environment');
 });
 
@@ -48,8 +41,4 @@ describe('ContextEnvironment', () => {
     expect(element).toBeTruthy();
   });
 
-});
-
-afterAll(async () => {
-  browser.close();
 });
diff --git a/tests/src/ContextPage.test.js b/tests/src/ContextPage.test.js
@@ -1,11 +1,4 @@
-const puppeteer = require('puppeteer');
-
-let browser;
-let page;
-
 beforeAll(async () => {
-  browser = await puppeteer.launch();
-  page = await browser.newPage();
   await page.goto('http://localhost:6969/context-page');
 });
 
@@ -95,9 +88,4 @@ describe('ContextPage', () => {
     expect(element).toBeTruthy();  
   });
 
-
-});
-
-afterAll(async () => {
-  browser.close();
 });
diff --git a/tests/src/ContextProject.test.js b/tests/src/ContextProject.test.js
@@ -1,18 +1,4 @@
-const puppeteer = require('puppeteer');
-
-let browser;
-
-beforeAll(async () => {
-  browser = await puppeteer.launch();
-});
-
 describe('ContextProject', () => {
-
-  let page;
-
-  beforeAll(async () => {
-    page = await browser.newPage();
-  });
 
   test('cors can be enabled', async () => {
     const response = await page.goto('http://localhost:6969/context-project');
@@ -23,11 +9,8 @@ describe('ContextProject', () => {
 });
 
 describe('ContextProject', () => {
-
-  let page;
 
   beforeAll(async () => {
-    page = await browser.newPage();
     await page.goto('http://localhost:6969/context-project');
     await page.waitForSelector('[data-project]');
   });
@@ -147,16 +130,13 @@ describe('ContextProject', () => {
     expect(text).toMatch('');
   });
 
-
 });
 
 describe('robots.txt', () => {
-
-  let page;
+
   let text;
-  
+
   beforeAll(async () => {
-    page = await browser.newPage();
     await page.goto('http://localhost:6969/robots.txt');
     text = await page.evaluate(() => document.body.innerHTML);
   });
@@ -181,8 +161,4 @@ describe('robots.txt', () => {
     expect(index).toBeGreaterThan(-1);
   });
 
-});
-
-afterAll(async () => {
-  browser.close();
 });
diff --git a/tests/src/ContextSecrets.test.js b/tests/src/ContextSecrets.test.js
@@ -1,11 +1,4 @@
-const puppeteer = require('puppeteer');
-
-let browser;
-let page;
-
 beforeAll(async () => {
-  browser = await puppeteer.launch();
-  page = await browser.newPage();
   await page.goto('http://localhost:6969/context-secrets');
 });
 
@@ -31,8 +24,4 @@ describe('ContextSecrets', () => {
     expect(element).toBeTruthy();
   });
 
-});
-
-afterAll(async () => {
-  browser.close();
 });
diff --git a/tests/src/ContextSettings.test.js b/tests/src/ContextSettings.test.js
@@ -1,11 +1,4 @@
-const puppeteer = require('puppeteer');
-
-let browser;
-let page;
-
 beforeAll(async () => {
-  browser = await puppeteer.launch();
-  page = await browser.newPage();
   await page.goto('http://localhost:6969/context-settings');
 });
 
@@ -31,8 +24,4 @@ describe('ContextSettings', () => {
     expect(element).toBeTruthy();
   });
 
-});
-
-afterAll(async () => {
-  browser.close();
 });
diff --git a/tests/src/ContextWorker.test.js b/tests/src/ContextWorker.test.js
@@ -1,17 +1,6 @@
-const puppeteer = require('puppeteer');
-
-let browser;
-
-beforeAll(async () => {
-  browser = await puppeteer.launch();
-});
-
 describe('ContextWorker', () => {
 
-  let page;
-
   beforeAll(async () => {
-    page = await browser.newPage();
     await page.goto('http://localhost:6969/context-worker');
   });
 
@@ -84,10 +73,7 @@ describe('ContextWorker', () => {
 
 describe('ContextWorker', () => {
 
-  let page;
-
   beforeAll(async () => {
-    page = await browser.newPage();
     await page.goto('http://localhost:6969/context-worker');
   });
 
@@ -103,10 +89,7 @@ describe('ContextWorker', () => {
 
 describe('ContextWorker', () => {
 
-  let page;
-
   beforeAll(async () => {
-    page = await browser.newPage();
     await page.goto('http://localhost:6969/context-worker');
   });
 
@@ -135,8 +118,4 @@ describe('ContextWorker', () => {
     expect(element).toBeTruthy();
   });
 
-});
-
-afterAll(async () => {
-  browser.close();
 });