Better ci (#131)

* Remove test folder

* Update Github Actions

* Add `error-on-fail` option

Author: retyui

.github/workflows/test.yml

@@ -6,21 +6,38 @@ on:
   pull_request:
     branches: [main]
 
+env:
+  APP_NAME: MyApp
+  NODEJS_VERSION: 24
+
 jobs:
-  build:
+  test:
+    name: Analyze bundle with React Native ${{ matrix.rn-version }}
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [12.x, 14.x, 16.x]
+        rn-version: [latest, 0.83.4, 0.82.1]
     steps:
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v6
 
-      - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v2
+      - name: Use Node.js ${{ env.NODEJS_VERSION }}
+        uses: actions/setup-node@v6
         with:
-          node-version: ${{ matrix.node-version }}
+          node-version: ${{ env.NODEJS_VERSION }}
           cache: 'yarn'
 
-      - run: yarn install --frozen-lockfile
-      - run: yarn test
+      - name: Install node_modules in ${{ github.workspace }}
+        run: yarn install --frozen-lockfile
+
+      - name: Install React Native ${{ matrix.rn-version }} and run the visualizer
+        working-directory: /tmp
+        shell: bash
+        env:
+          YARN_ENABLE_IMMUTABLE_INSTALLS: false
+        run: |
+          set -x # print all executed commands
+          npx @react-native-community/cli@latest init ${{ env.APP_NAME }} --version ${{ matrix.rn-version }} --skip-git-init --install-pods false --pm yarn
+          cd ${{ env.APP_NAME }}
+          yarn add react-native-bundle-visualizer@portal:${{ github.workspace }}
+          yarn run react-native-bundle-visualizer --verbose --error-on-fail

src/react-native-bundle-visualizer.js

@@ -2,6 +2,7 @@
 const chalk = require('chalk');
 const fs = require('fs-extra');
 const os = require('os');
+const process = require('process');
 const path = require('path');
 const argv = require('minimist')(process.argv.slice(2));
 const execa = require('execa');
@@ -70,6 +71,7 @@ const format = argv.format || 'html';
 const bundleOutputExplorerFile = path.join(outDir, 'explorer.' + format);
 const onlyMapped = !!argv['only-mapped'] || false;
 const borderChecks = argv['border-checks'] === true;
+const errorOnFail = argv['error-on-fail'] || false;
 
 // Make sure the temp dir exists
 fs.ensureDirSync(baseDir);
@@ -184,4 +186,9 @@ bundlePromise
     // Open output file
     return open(bundleOutputExplorerFile);
   })
-  .catch((error) => console.log(chalk.red('=== error ==='), error));
+  .catch((error) => {
+    console.log(chalk.red('=== error ==='), error)
+    if (errorOnFail) {
+      process.exit(1);
+    }
+  });