Merge pull request #218 from abcnews/fix-test-builds

Fix test builds

.github/workflows/tests.yml

@@ -6,7 +6,7 @@ jobs:
   test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - name: Test using Node.js
         uses: actions/setup-node@v4
         with:

package.json

@@ -6,9 +6,10 @@
   "license": "MIT",
   "contributors": [
     "Colin Gourlay <[email protected]>",
-    "Nathan Hoad <[email protected]>",
     "Simon Elvery <[email protected]>",
-    "Joshua Byrd <[email protected]>"
+    "Joshua Byrd <[email protected]>",
+    "Ash Kyd <[email protected]>",
+    "Nathan Hoad"
   ],
   "scripts": {
     "postinstall": "node scripts/postinstall.js",
@@ -92,6 +93,7 @@
     "yeoman-generator": "^5.2.0"
   },
   "devDependencies": {
+    "@types/jest": "^29.5.10",
     "babel-eslint": "^10.1.0",
     "eslint": "^7.7.0",
     "eslint-config-prettier": "^8.2.0",

test/test.js

@@ -1,8 +1,8 @@
-const { _testBuild } = require('../src/cli/build');
 const { _testGenerate } = require('../src/cli/generate');
 const path = require('path');
 const fs = require('fs/promises');
 const mem = require('mem');
+const { execSync } = require('child_process');
 const { getBuildConfig } = require('../src/config/build');
 const { getBabelConfig } = require('../src/config/babel');
 const { getProjectConfig } = require('../src/config/project');
@@ -53,6 +53,10 @@ beforeAll(async () => {
   // Clean everything up
   await rmRecursive(tempRoot);
   await fs.mkdir(tempRoot);
+
+  // Link local aunty to global
+  const linkOutput = execSync('npm link');
+  console.log('Running: npm link', linkOutput.toString());
 });
 
 // Reset mocks
@@ -109,8 +113,17 @@ afterAll(async () => {
             it('should build the generated project', async () => {
               process.chdir(generatedProjectRoot);
 
-              // If the build fails for any reason this will throw.
-              await _testBuild(argv);
+              // execSync will throw on non-zero exit code
+
+              {
+                const output = execSync('npm link @abcnews/aunty');
+                console.log('Running: npm link @abcnews/aunty', output.toString());
+              }
+
+              {
+                const output = execSync('npx aunty build');
+                console.log('Running: npx aunty build', output.toString());
+              }
 
               const fileList = await fs.readdir(path.join(generatedProjectRoot, '.aunty/build'));
 

ts/config.json

@@ -14,7 +14,7 @@
     "esModuleInterop": true,
     "forceConsistentCasingInFileNames": true,
     "allowSyntheticDefaultImports": true,
-    "verbatimModuleSyntax": true,
+    "verbatimModuleSyntax": false,
     "resolveJsonModule": true,
     "isolatedModules": true,
     "strictNullChecks": true