Merge pull request #1 from Gaurav0/fix_ci

Fix ci and make compatible with Ember 5

Author: Gaurav0

.github/workflows/ci.yml

@@ -22,8 +22,10 @@ jobs:
   test-all-packages:
     name: Ember Tests
     runs-on: ${{ matrix.os }}
+    timeout-minutes: 30
 
     strategy:
+      fail-fast: false
       matrix:
         node-version: [16.x, 14.x]
         os: [ubuntu-latest, windows-latest]
@@ -51,8 +53,10 @@ jobs:
   integration-tests:
     name: Integration Tests
     runs-on: ${{ matrix.os }}
+    timeout-minutes: 30
 
     strategy:
+      fail-fast: false
       matrix:
         node-version: [16.x, 14.x]
         os: [ubuntu-latest, windows-latest]
@@ -73,8 +77,10 @@ jobs:
   test-packages:
     name: Test Packages
     runs-on: ${{ matrix.os }}
+    timeout-minutes: 30
 
     strategy:
+      fail-fast: false
       matrix:
         node-version: [16.x, 14.x]
         os: [ubuntu-latest, windows-latest]
@@ -99,6 +105,7 @@ jobs:
   test-legacy-mocha:
     name: Legacy Mocha Tests - ${{ matrix.node-version }}
     runs-on: ubuntu-latest
+    timeout-minutes: 30
 
     strategy:
       fail-fast: false
@@ -109,20 +116,23 @@ jobs:
       - name: Checkout Code
         uses: actions/checkout@v2
       - name: Use Node.js ${{ matrix.node-version }}
-        uses: volta-cli/action@v1
+        uses: volta-cli/action@v4
         with:
           node-version: ${{ matrix.node-version }}
       - name: Yarn Install
         run: |
           yarn install --ignore-engines --frozen-lockfile
+      - name: Precook node_modules
+        run: |
+          yarn workspace ember-cli-fastboot pretest:precook
       - name: Run Mocha Tests
         run: |
           yarn workspace ember-cli-fastboot test:mocha
 
   try-scenarios:
-    name: "ember-try for ember-cli-fastboot: ${{ matrix.ember-try-scenario }}"
+    name: "${{ matrix.ember-try-scenario }} Scenario for ember-cli-fastboot"
     runs-on: ubuntu-latest
-    timeout-minutes: 7
+    timeout-minutes: 30
 
     strategy:
       fail-fast: false
@@ -133,6 +143,8 @@ jobs:
           - ember-lts-3.24
           - ember-lts-3.28
           - ember-lts-4.4
+          - ember-lts-4.8
+          - ember-lts-4.12
           - ember-release
           - ember-beta
           - ember-canary

package.json

@@ -5,10 +5,15 @@
     "type": "git",
     "url": "[email protected]:ember-fastboot/ember-cli-fastboot.git"
   },
-  "workspaces": [
-    "packages/*",
-    "test-packages/*"
-  ],
+  "workspaces": {
+    "packages": [
+      "packages/*",
+      "test-packages/*"
+    ],
+    "nohoist": [
+      "**/ember-cli-addon-tests"
+    ]
+  },
   "scripts": {
     "test": "npm-run-all test:*",
     "test:ember-cli-fastboot": "yarn workspace ember-cli-fastboot test:ember",
@@ -24,6 +29,9 @@
     "release-it-lerna-changelog": "^3.1.0",
     "release-it-yarn-workspaces": "^2.0.0"
   },
+  "resolutions": {
+    "fastboot": "4.1.1"
+  },
   "publishConfig": {
     "registry": "https://registry.npmjs.org"
   },

packages/ember-cli-fastboot/.eslintignore

@@ -13,6 +13,7 @@
 # misc
 /coverage/
 !.*
+.*/
 .eslintcache
 
 # ember-try

packages/ember-cli-fastboot/.eslintrc.js

@@ -28,23 +28,15 @@ module.exports = {
     // node files
     {
       files: [
-        '.eslintrc.js',
-        '.prettierrc.js',
-        '.template-lintrc.js',
-        'ember-cli-build.js',
-        'index.js',
-        'testem.js',
-        'blueprints/*/index.js',
-        'config/**/*.js',
-        'tests/dummy/config/**/*.js',
-        'lib/**/*.js'
-      ],
-      excludedFiles: [
-        'app/**',
-        'addon/**',
-        'addon-test-support/**',
-        'app/**',
-        'tests/dummy/app/**',
+        './.eslintrc.js',
+        './.prettierrc.js',
+        './.template-lintrc.js',
+        './ember-cli-build.js',
+        './index.js',
+        './testem.js',
+        './blueprints/*/index.js',
+        './config/**/*.js',
+        './tests/dummy/config/**/*.js',
       ],
       parserOptions: {
         sourceType: 'script',
@@ -56,5 +48,10 @@ module.exports = {
       plugins: ['node'],
       extends: ['plugin:node/recommended'],
     },
+    {
+      // Test files:
+      files: ['tests/**/*-test.{js,ts}'],
+      extends: ['plugin:qunit/recommended'],
+    },
   ],
 };

packages/ember-cli-fastboot/.npmignore

@@ -27,6 +27,7 @@
 /testem.js
 /test/
 /tests/
+/yarn-error.log
 /yarn.lock
 .gitkeep
 

packages/ember-cli-fastboot/.template-lintrc.js

@@ -1,5 +1,5 @@
 'use strict';
 
 module.exports = {
-  extends: 'octane',
+  extends: 'recommended',
 };

packages/ember-cli-fastboot/config/ember-try.js

@@ -51,6 +51,22 @@ module.exports = async function () {
           },
         },
       },
+      {
+        name: 'ember-lts-4.8',
+        npm: {
+          devDependencies: {
+            'ember-source': '~4.8.0',
+          },
+        },
+      },
+      {
+        name: 'ember-lts-4.12',
+        npm: {
+          devDependencies: {
+            'ember-source': '~4.12.0',
+          },
+        },
+      },
       {
         name: 'ember-release',
         npm: {
@@ -116,6 +132,15 @@ module.exports = async function () {
           },
         },
       },
+      {
+        name: 'ember-4.12-no-deprecations',
+        npm: {
+          devDependencies: {
+            'ember-source': '~4.12.0',
+            'ember-deprecation-error': '*',
+          },
+        },
+      },
       {
         name: 'ember-release-no-deprecations',
         npm: {

packages/ember-cli-fastboot/ember-cli-build.js

@@ -15,5 +15,11 @@ module.exports = function (defaults) {
   */
 
   const { maybeEmbroider } = require('@embroider/test-setup');
-  return maybeEmbroider(app);
+  return maybeEmbroider(app, {
+    skipBabel: [
+      {
+        package: 'qunit',
+      },
+    ],
+  });
 };

packages/ember-cli-fastboot/fastboot/instance-initializers/fetch.js

@@ -0,0 +1,10 @@
+function initialize(instance) {
+  let { request } = instance.lookup('service:fastboot');
+  if (fetch) {
+    fetch.__fastbootRequest = request;
+  }
+}
+
+export default {
+  initialize,
+};

packages/ember-cli-fastboot/fix-node-modules.mjs

@@ -12,6 +12,7 @@ const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
 const packagesDir = path.resolve(__dirname, '../../packages');
 const nodeModulesDir = path.resolve(__dirname, 'node_modules');
+const precookedDir = path.resolve(__dirname, 'tmp', 'precooked_node_modules');
 
 // eslint-disable-next-line no-undef
 const shouldRestore = process.argv[2];
@@ -29,18 +30,27 @@ Options:
 function run(shouldRestore) {
   ['fastboot', 'fastboot-express-middleware'].forEach((packageName) => {
     const nodeModulesPackageDir = path.join(nodeModulesDir, packageName);
+    const precookedPackageDir = path.join(precookedDir, packageName);
     const workspacesPackageDir = path.resolve(packagesDir, packageName);
     if (fs.existsSync(nodeModulesPackageDir)) {
       console.log(chalk.blue(`remove ${nodeModulesPackageDir}`));
       fs.removeSync(nodeModulesPackageDir);
     }
+    if (fs.existsSync(precookedPackageDir)) {
+      console.log(chalk.blue(`remove ${precookedPackageDir}`));
+      fs.removeSync(precookedPackageDir);
+    }
     if (!shouldRestore) {
       console.log(
         chalk.green(
           `symlink ${nodeModulesPackageDir} -> ${workspacesPackageDir}`
         )
       );
       fs.symlinkSync(workspacesPackageDir, nodeModulesPackageDir, 'dir');
+      console.log(
+        chalk.green(`symlink ${precookedPackageDir} -> ${workspacesPackageDir}`)
+      );
+      fs.symlinkSync(workspacesPackageDir, precookedPackageDir, 'dir');
     }
   });
 }