Merge pull request #510 from dilanx:automated-testing

Added unit and integration testing through Github Actions on push/PR
dilanx · May 14, 2023 · 510a61b · 510a61b
2 parents 42c756c + dc2ed70
commit 510a61b
Show file tree

Hide file tree

Showing 5 changed files with 37 additions and 9 deletions.
diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml
@@ -0,0 +1,27 @@
+name: tests
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [16.x, 18.x]
+
+    steps:
+      - uses: actions/checkout@v2
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v2
+        with:
+          node-version: ${{ matrix.node-version }}
+      - run: npm ci
+      - run: CI=false npm run build
+      - run: npm run test:unit
+      - run: npx playwright install-deps
+      - run: CI=false npm run test:integration
diff --git a/README.md b/README.md
@@ -13,7 +13,7 @@
 
   <br>
 
-[![npm status](https://img.shields.io/npm/v/@craco/craco.svg)](https://www.npmjs.com/package/@craco/craco) [![npm downloads](https://img.shields.io/npm/dm/@craco/craco.svg)](https://www.npmjs.com/package/@craco/craco) [![npm license](https://img.shields.io/npm/l/@craco/craco?color=orange)](https://github.com/dilanx/craco/blob/main/packages/craco/LICENSE) [![GitHub stars](https://img.shields.io/github/stars/dilanx/craco?color=red)](https://github.com/dilanx/craco) [![GitHub contributors](https://img.shields.io/github/contributors/dilanx/craco?color=blueviolet)](https://github.com/dilanx/craco/graphs/contributors) [![PRs Welcome](https://img.shields.io/badge/PRs-welcome-blueviolet.svg)](https://github.com/dilanx/craco/pulls)
+[![npm status](https://img.shields.io/npm/v/@craco/craco.svg)](https://www.npmjs.com/package/@craco/craco) [![npm downloads](https://img.shields.io/npm/dm/@craco/craco.svg)](https://www.npmjs.com/package/@craco/craco) [![npm license](https://img.shields.io/npm/l/@craco/craco?color=orange)](https://github.com/dilanx/craco/blob/main/packages/craco/LICENSE) [![GitHub stars](https://img.shields.io/github/stars/dilanx/craco?color=red)](https://github.com/dilanx/craco) [![GitHub contributors](https://img.shields.io/github/contributors/dilanx/craco?color=blueviolet)](https://github.com/dilanx/craco/graphs/contributors) [![PRs Welcome](https://img.shields.io/badge/PRs-welcome-blueviolet.svg)](https://github.com/dilanx/craco/pulls) ![Node.js CI](https://github.com/dilanx/craco/actions/workflows/run-tests.yml/badge.svg)
 
 </div>
 

diff --git a/test/integration/fixtures/basic-integration-test/index.test.js b/test/integration/fixtures/basic-integration-test/index.test.js
@@ -15,7 +15,7 @@ beforeAll(async () => {
 
   // Leave time for the server to initialize
   await new Promise((resolve) => {
-    setTimeout(resolve, 3000);
+    setTimeout(resolve, 5000);
   });
 });
 
@@ -31,7 +31,7 @@ test('Should have the expected styles', async () => {
 
 afterAll(() => {
   // Stop the local server
-  execSync(`kill $(lsof -t -i:${global.PORT})`, {
+  execSync(`kill $(lsof -t -i:${global.PORT} -a -c node)`, {
     cwd: __dirname,
     stdio: 'ignore',
   });

diff --git a/test/integration/jest.config.js b/test/integration/jest.config.js
@@ -10,13 +10,14 @@ module.exports = {
   moduleNameMapper: {
     '\\.(css|less|scss|sass)$': 'identity-obj-proxy',
   },
-  setupFiles: ['./setup.js'],
+  globalSetup: './setup.js',
   globalTeardown: './teardown.js',
   testEnvironmentOptions: {
     'jest-playwright': {
       browsers: ['firefox'],
       launchOptions: {
         headless: true,
+        timeout: 500000,
       },
     },
   },

diff --git a/test/integration/setup.js b/test/integration/setup.js
@@ -29,18 +29,18 @@ module.exports = async function (globalConfig, projectConfig) {
         `cd ${join(
           rootPath,
           directoryName
-        )}&& npx create-react-app test-project`,
-        { cwd: cwd }
+        )} && npx create-react-app test-project`,
+        { cwd: cwd, stdio: 'inherit' }
       );
       execSync(`cp -r ${testPackageFilesPath}/* ${testProjectPath}`, {
-        cwd: cwd,
+        cwd: cwd
       });
       //install craco
       execSync(`npm install ../../../../../packages/craco`, {
-        cwd: testProjectPath,
+        cwd: testProjectPath, stdio: 'inherit'
       });
       //install other necessary files
-      execSync(`npm install`, { cwd: testProjectPath });
+      execSync(`npm install`, { cwd: testProjectPath, stdio: 'inherit' });
       //build project
       execSync('npm run build', { cwd: testProjectPath, stdio: 'inherit' });
     });