[issue-9] added action and end to end tests

Author: snake-py

.github/workflows/github-actions-e2e-test.yml

@@ -0,0 +1,65 @@
+name: End To End Test
+
+on:
+  pull_request:
+    branches:
+      - main
+    types:
+      - opened
+      - synchronize
+      - reopened
+
+jobs:
+  NPM-E2E-Test:
+    runs-on: ubuntu-latest
+    container:
+      image: snakepy/node:node18-latest
+    environment: test
+    steps:
+      - name: Check out repository code
+        uses: actions/checkout@v4
+      - run: npm install
+      - run: npm run build
+      - run: npm link
+      - name: "Setup repository"
+        run: |
+          cd ../
+          git config --global user.email ${{ secrets.GIT_EMAIL }}
+          git config --global user.name ${{ secrets.GIT_USER }}
+          git clone https://github.com/snake-py/eject-test.git 
+          cd eject-test 
+          git checkout npm
+          npm install 
+          git add -A 
+          git stash 
+          npx eject react --verbose
+      - name: "Run E2E Test"
+        run: npm run test:e2e:base --dependency="react" --dpath="../eject-test" --dmanager="npm"
+
+  PNPM-E2E-Test:
+    runs-on: ubuntu-latest
+    container:
+      image: snakepy/node:node18-latest
+    environment: test
+    steps:
+      - name: Check out repository code
+        uses: actions/checkout@v4
+      - run: npm install
+      - run: npm run build
+      - run: npm link
+      - name: "Setup repository"
+        run: |
+          cd ../
+          git config --global user.email ${{ secrets.GIT_EMAIL }}
+          git config --global user.name ${{ secrets.GIT_USER }}
+          git clone https://github.com/snake-py/eject-test.git 
+          cd eject-test 
+          git checkout pnpm
+          pnpm install 
+          git add -A 
+          git stash 
+          npx eject react --verbose
+      - name: "Run E2E Base Test"
+        run: npm run test:e2e:base --dependency="react" --dpath="../eject-test" --dmanager="pnpm"
+      - name: "Run E2E PNPM Test"
+        run: npm run test:e2e:pnpm --dependency="react" --dpath="../eject-test" --dmanager="pnpm"

package.json

@@ -16,7 +16,9 @@
     "release": "release-me patch",
     "release:minor": "release-me minor",
     "release:major": "release-me major",
-    "release:commit": "release-me commit"
+    "release:commit": "release-me commit",
+    "test:e2e:base": "node ./tests/e2e/base.js $npm_config_dependency $npm_config_dpath $npm_config_dmanager",
+    "test:e2e:pnpm": "node ./tests/e2e/pnpm.js $npm_config_dependency $npm_config_dpath $npm_config_dmanager"
   },
   "repository": {
     "type": "git",
@@ -47,4 +49,4 @@
     "eject": "./bin-entry.js",
     "ejectjs": "./bin-entry.js"
   }
-}
+}

src/command.ts

@@ -19,6 +19,14 @@ export function eject(
     dependency: string,
     options: { force: boolean; verbose: boolean },
 ) {
+    if (options.verbose) {
+        console.log({
+            dependency,
+            options,
+            isCI: process.env.GITHUB_ACTIONS,
+        });
+    }
+
     if (!options.force && !isGitInstalled()) {
         warnLog(
             'Git is not installed, please install git or use --force to bypass git check',
@@ -38,21 +46,25 @@ export function eject(
     } catch (error) {
         if (options.verbose) {
             errorLog('Error ejecting dependency:', dependency);
+            console.error(error);
         }
     }
 
     if (successfulEjected) {
         console.log('✅ Ejected dependencies:', chalk.bold(dependency));
     } else {
         console.log('❌ Ejected dependencies: ', chalk.bold(dependency));
-        return;
+        return 150;
     }
     const { packageManager, lockFile } = detectPackageManager();
     updatePackageJson(dependency, packageManager === 'pnpm' ? 'link' : 'file');
     commitEjection(config.COMMIT_MESSAGE);
     const needsToCommit = dependencyManagerAction(packageManager);
     if (needsToCommit) amendCommit();
-    const installCmd = `${packageManager} install`;
+    let installCmd = `${packageManager} install`;
+    if (process.env.GITHUB_ACTIONS && packageManager === 'pnpm') {
+        installCmd = 'pnpm install --no-frozen-lockfile';
+    }
     const installLog = chalk.bold(installCmd);
     console.log(`📦 Running ${installLog} to update ${chalk.bold(lockFile)}`);
     execSync(installCmd, { stdio: 'inherit' });

tests/e2e/base.js

@@ -0,0 +1,31 @@
+// read command line arguments
+import assert from 'assert';
+import fs from 'fs';
+import { resolve } from 'path';
+const args = process.argv.slice(2);
+const dependency = args[0];
+const testProjectPath = args[1];
+const dependencyManager = args[2] ?? 'npm';
+
+assert(fs.existsSync(testProjectPath), 'testProjectPath does not exist');
+assert(
+    fs.existsSync(resolve(testProjectPath, 'ejected')),
+    'No ejected folder found',
+);
+assert(
+    fs.existsSync(resolve(testProjectPath, 'ejected', dependency)),
+    'No ejection found for dependency',
+);
+
+const packageJson = JSON.parse(
+    fs.readFileSync(resolve(testProjectPath, 'package.json'), 'utf8'),
+);
+
+assert(
+    packageJson.dependencies[dependency].includes(
+        dependencyManager === 'pnpm'
+            ? 'link:./ejected/' + dependency
+            : 'file:./ejected/' + dependency,
+    ),
+    'Dependency is not properly ejected',
+);

tests/e2e/pnpm.js

@@ -0,0 +1,19 @@
+// read command line arguments
+import assert from 'assert';
+import fs from 'fs';
+import { resolve } from 'path';
+import { parse } from 'yaml';
+
+const args = process.argv.slice(2);
+const dependency = args[0];
+const testProjectPath = args[1];
+
+assert(
+    fs.existsSync(resolve(testProjectPath, 'pnpm-workspace.yaml')),
+    'pnpm-workspace.yaml does not exist',
+);
+const workspace = parse(
+    fs.readFileSync(resolve(testProjectPath, 'pnpm-workspace.yaml'), 'utf-8'),
+);
+console.log(workspace);
+assert(workspace.packages.includes(`ejected/*`), 'No ejected workspace found');