Fix CI

ember-cli-addon-tests defaults to use ember-data 3.8 which is not
compatible any more
https://github.com/tomdale/ember-cli-addon-tests/blob/6f0beb154853d7e755a8007a30f01eb2b4f4a6dc/lib/utilities/pristine.js#L98

Also disable a test which throws:

Error: Cannot find module 'abortcontroller-polyfill/dist/cjs-ponyfill'

Author: xg-wang

.github/workflows/ci.yml

@@ -119,7 +119,6 @@ jobs:
         run: |
           yarn install --ignore-engines --frozen-lockfile
       - name: Run Mocha Tests
-        working-directory: ./packages/ember-cli-fastboot
         run: |
           npm --version
-          yarn test:mocha
+          yarn workspace ember-cli-fastboot test:mocha

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

@@ -6,37 +6,42 @@ const request = RSVP.denodeify(require('request'));
 
 const AddonTestApp = require('ember-cli-addon-tests').AddonTestApp;
 
-describe('FastBoot config', function() {
+describe('FastBoot config', function () {
   this.timeout(400000);
 
   let app;
 
-  before(function() {
+  before(function () {
     app = new AddonTestApp();
 
-    return app.create('fastboot-config', { emberVersion: 'latest'})
-      .then(function() {
+    return app
+      .create('fastboot-config', {
+        emberVersion: 'latest',
+        emberDataVersion: '~3.19.0',
+      })
+      .then(function () {
         return app.startServer({
-          command: 'serve'
+          command: 'serve',
         });
       });
   });
 
-  after(function() {
+  after(function () {
     return app.stopServer();
   });
 
-  it('provides sandbox globals', function() {
+  it('provides sandbox globals', function () {
     return request({
       url: 'http://localhost:49741/',
       headers: {
-        'Accept': 'text/html'
-      }
-    })
-      .then(function(response) {
-        expect(response.statusCode).to.equal(200);
-        expect(response.headers['content-type']).to.equalIgnoreCase('text/html; charset=utf-8');
-        expect(response.body).to.contain('<h1>My Global</h1>');
-      });
+        Accept: 'text/html',
+      },
+    }).then(function (response) {
+      expect(response.statusCode).to.equal(200);
+      expect(response.headers['content-type']).to.equalIgnoreCase(
+        'text/html; charset=utf-8'
+      );
+      expect(response.body).to.contain('<h1>My Global</h1>');
+    });
   });
 });

packages/ember-cli-fastboot/test/fastboot-location-config-test.js

@@ -6,49 +6,55 @@ const request = RSVP.denodeify(require('request'));
 
 const AddonTestApp = require('ember-cli-addon-tests').AddonTestApp;
 
-describe('FastBootLocation Configuration', function() {
+describe('FastBootLocation Configuration', function () {
   this.timeout(300000);
 
   let app;
 
-  before(function() {
+  before(function () {
     app = new AddonTestApp();
 
-    return app.create('fastboot-location-config', { emberVersion: 'latest'})
-      .then(function() {
+    return app
+      .create('fastboot-location-config', {
+        emberVersion: 'latest',
+        emberDataVersion: '~3.19.0',
+      })
+      .then(function () {
         return app.startServer({
-          command: 'serve'
+          command: 'serve',
         });
       });
   });
 
-  after(function() {
+  after(function () {
     return app.stopServer();
   });
 
-  it('should use the redirect code provided by the EmberApp', function() {
+  it('should use the redirect code provided by the EmberApp', function () {
     return request({
       url: 'http://localhost:49741/redirect-on-transition-to',
       followRedirect: false,
       headers: {
-        Accept: 'text/html'
-      }
-    }).then(function(response) {
+        Accept: 'text/html',
+      },
+    }).then(function (response) {
       if (response.statusCode === 500) throw new Error(response.body);
       expect(response.statusCode).to.equal(302);
-      expect(response.headers.location).to.equal('//localhost:49741/test-passed');
+      expect(response.headers.location).to.equal(
+        '//localhost:49741/test-passed'
+      );
     });
   });
 
-  describe('when fastboot.fastbootHeaders is false', function() {
-    it('should not send the "x-fastboot-path" header on a redirect', function() {
+  describe('when fastboot.fastbootHeaders is false', function () {
+    it('should not send the "x-fastboot-path" header on a redirect', function () {
       return request({
         url: 'http://localhost:49741/redirect-on-transition-to',
         followRedirect: false,
         headers: {
-          Accept: 'text/html'
-        }
-      }).then(function(response) {
+          Accept: 'text/html',
+        },
+      }).then(function (response) {
         if (response.statusCode === 500) throw new Error(response.body);
         expect(response.headers).to.not.include.keys(['x-fastboot-path']);
       });

packages/ember-cli-fastboot/test/fastboot-location-test.js

@@ -6,33 +6,37 @@ const request = RSVP.denodeify(require('request'));
 
 const AddonTestApp = require('ember-cli-addon-tests').AddonTestApp;
 
-describe('FastBootLocation', function() {
+describe('FastBootLocation', function () {
   this.timeout(300000);
 
   let app;
-  before(function() {
+  before(function () {
     app = new AddonTestApp();
 
-    return app.create('fastboot-location', { emberVersion: 'latest'})
-      .then(function() {
+    return app
+      .create('fastboot-location', {
+        emberVersion: 'latest',
+        emberDataVersion: '~3.19.0',
+      })
+      .then(function () {
         return app.startServer({
-          command: 'serve'
+          command: 'serve',
         });
       });
   });
 
-  after(function() {
+  after(function () {
     return app.stopServer();
   });
 
-  it('should NOT redirect when no transition is called', function() {
+  it('should NOT redirect when no transition is called', function () {
     return request({
       url: 'http://localhost:49741/my-root/test-passed',
       followRedirect: false,
       headers: {
-        Accept: 'text/html'
-      }
-    }).then(function(response) {
+        Accept: 'text/html',
+      },
+    }).then(function (response) {
       if (response.statusCode === 500) throw new Error(response.body);
       expect(response.statusCode).to.equal(200);
 
@@ -46,15 +50,14 @@ describe('FastBootLocation', function() {
     });
   });
 
-  it('should NOT redirect when intermediateTransitionTo is called', function() {
+  it('should NOT redirect when intermediateTransitionTo is called', function () {
     return request({
-      url:
-        'http://localhost:49741/my-root/redirect-on-intermediate-transition-to',
+      url: 'http://localhost:49741/my-root/redirect-on-intermediate-transition-to',
       followRedirect: false,
       headers: {
-        Accept: 'text/html'
-      }
-    }).then(function(response) {
+        Accept: 'text/html',
+      },
+    }).then(function (response) {
       if (response.statusCode === 500) throw new Error(response.body);
       expect(response.statusCode).to.equal(200);
 
@@ -69,14 +72,14 @@ describe('FastBootLocation', function() {
     });
   });
 
-  it('should redirect when transitionTo is called', function() {
+  it('should redirect when transitionTo is called', function () {
     return request({
       url: 'http://localhost:49741/my-root/redirect-on-transition-to',
       followRedirect: false,
       headers: {
-        Accept: 'text/html'
-      }
-    }).then(function(response) {
+        Accept: 'text/html',
+      },
+    }).then(function (response) {
       if (response.statusCode === 500) throw new Error(response.body);
       expect(response.statusCode).to.equal(307);
 
@@ -92,14 +95,14 @@ describe('FastBootLocation', function() {
     });
   });
 
-  it('should redirect when replaceWith is called', function() {
+  it('should redirect when replaceWith is called', function () {
     return request({
       url: 'http://localhost:49741/my-root/redirect-on-replace-with',
       followRedirect: false,
       headers: {
-        Accept: 'text/html'
-      }
-    }).then(function(response) {
+        Accept: 'text/html',
+      },
+    }).then(function (response) {
       if (response.statusCode === 500) throw new Error(response.body);
       expect(response.statusCode).to.equal(307);
 
@@ -115,14 +118,14 @@ describe('FastBootLocation', function() {
     });
   });
 
-  it('should NOT redirect when transitionTo is called with identical route name', function() {
+  it('should NOT redirect when transitionTo is called with identical route name', function () {
     return request({
       url: 'http://localhost:49741/my-root/noop-transition-to',
       followRedirect: false,
       headers: {
-        Accept: 'text/html'
-      }
-    }).then(function(response) {
+        Accept: 'text/html',
+      },
+    }).then(function (response) {
       if (response.statusCode === 500) throw new Error(response.body);
       expect(response.statusCode).to.equal(200);
 
@@ -136,14 +139,14 @@ describe('FastBootLocation', function() {
     });
   });
 
-  it('should NOT redirect when replaceWith is called with identical route name', function() {
+  it('should NOT redirect when replaceWith is called with identical route name', function () {
     return request({
       url: 'http://localhost:49741/my-root/noop-replace-with',
       followRedirect: false,
       headers: {
-        Accept: 'text/html'
-      }
-    }).then(function(response) {
+        Accept: 'text/html',
+      },
+    }).then(function (response) {
       if (response.statusCode === 500) throw new Error(response.body);
       expect(response.statusCode).to.equal(200);
 

packages/ember-cli-fastboot/test/new-package-json-test.js

@@ -5,7 +5,6 @@ const expect = require('chai').expect;
 const helpers = require('broccoli-test-helper');
 const createBuilder = helpers.createBuilder;
 const createTempDir = helpers.createTempDir;
-const co = require('co');
 const FastbootConfig = require('../lib/broccoli/fastboot-config');
 
 describe('FastbootConfig', function() {

packages/ember-cli-fastboot/test/package-json-test.js

@@ -1,28 +1,32 @@
 'use strict';
 
-const chai         = require('chai');
-const expect       = chai.expect;
-const fs           = require('fs-extra');
-const path         = require('path');
+const chai = require('chai');
+const expect = chai.expect;
+const fs = require('fs-extra');
+const path = require('path');
 const AddonTestApp = require('ember-cli-addon-tests').AddonTestApp;
 
 chai.use(require('chai-fs'));
 
-describe('generating package.json', function() {
+describe('generating package.json', function () {
   this.timeout(300000);
 
-  describe('with customized outputPaths options', function() {
+  describe('with customized outputPaths options', function () {
     // Tests an app with a custom `outputPaths` set
     let customApp = new AddonTestApp();
 
-    before(function() {
-      return customApp.create('customized-outputpaths')
-        .then(function() {
+    before(function () {
+      return customApp
+        .create('customized-outputpaths', {
+          emberVersion: 'latest',
+          emberDataVersion: '~3.19.0',
+        })
+        .then(function () {
           return customApp.runEmberCommand('build');
         });
     });
 
-    it('respects custom output paths and maps to them in the manifest', function() {
+    it('respects custom output paths and maps to them in the manifest', function () {
       function p(filePath) {
         return customApp.filePath(path.join('dist', filePath));
       }
@@ -31,16 +35,15 @@ describe('generating package.json', function() {
       let manifest = pkg.fastboot.manifest;
 
       expect(manifest.appFiles).to.include('some-assets/path/app-file.js');
-      manifest.appFiles.forEach(function(file) {
+      manifest.appFiles.forEach(function (file) {
         expect(p(file)).to.be.a.file();
       });
       expect(p(manifest.htmlFile)).to.be.a.file();
 
       expect(manifest.vendorFiles).to.include('some-assets/path/lib.js');
-      manifest.vendorFiles.forEach(function(file) {
+      manifest.vendorFiles.forEach(function (file) {
         expect(p(file)).to.be.a.file();
       });
     });
   });
 });
-