nebius/SWE-rebench-V2-PRs huggingface dataset:
Repository: ember-cli/ember-cli
Pull request #7109 (2017-06-06 21:04:42)

Fix `ember install` for scoped packages
It's not correct to try to strip the scope out of package names. I don't know why the test mock was set up this way, but you can verify this change against a real scoped package by doing:

 * `ember new my-app`
 * `cd my-app`
 * `ember install @cardstack/git`

Without this change it fails, after this change it succeeds.

Stripping off scopes fundamentally can't be correct because it's legal for `@foo/x` and `@bar/x` to want to coexist in a single app. NPM consistently treats the scope as part of the name.


Patch:
diff --git a/lib/utilities/get-package-base-name.js b/lib/utilities/get-package-base-name.js
index 220671880..972826a4c 100644
--- a/lib/utilities/get-package-base-name.js
+++ b/lib/utilities/get-package-base-name.js
@@ -4,18 +4,9 @@ const npa = require('npm-package-arg');
 
 
 module.exports = function(name) {
-  let parsed, parsedName;
-
   if (!name) {
     return null;
   }
 
-  parsed = npa(name);
-  parsedName = parsed.name;
-
-  if (parsed.scope) {
-    parsedName = parsedName.replace(parsed.scope, '').slice(1);
-  }
-
-  return parsedName;
+  return npa(name).name;
 };


Test patch:
diff --git a/tests/unit/commands/install-test.js b/tests/unit/commands/install-test.js
index c0583ef6a..52154b92b 100644
--- a/tests/unit/commands/install-test.js
+++ b/tests/unit/commands/install-test.js
@@ -41,6 +41,11 @@ describe('install command', function() {
             name: 'ember-cli-qunit',
           },
         },
+        {
+          pkg: {
+            name: '@ember-cli/ember-cli-qunit',
+          },
+        },
       ];
     };
 
@@ -249,7 +254,7 @@ describe('install command', function() {
 
         let generateRun = tasks.GenerateFromBlueprint.prototype.run;
         let generateRunArgs = td.explain(generateRun).calls.map(function(call) { return call.args[0].args[0]; });
-        expect(generateRunArgs).to.deep.equal(['ember-cli-qunit']);
+        expect(generateRunArgs).to.deep.equal(['@ember-cli/ember-cli-qunit']);
       });
     });
 
diff --git a/tests/unit/utilities/get-package-base-name-test.js b/tests/unit/utilities/get-package-base-name-test.js
index d7076e838..52ee9e021 100644
--- a/tests/unit/utilities/get-package-base-name-test.js
+++ b/tests/unit/utilities/get-package-base-name-test.js
@@ -8,8 +8,8 @@ describe('getPackageBaseName', function() {
     expect(getPackageBaseName('my-addon')).to.equal('my-addon');
   });
 
-  it('should return the package name without its scope', function() {
-    expect(getPackageBaseName('@scope/my-addon')).to.equal('my-addon');
+  it('should return the full name when scoped', function() {
+    expect(getPackageBaseName('@scope/my-addon')).to.equal('@scope/my-addon');
   });
 
   it('should strip away version numbers', function() {
Repository: ember-cli/ember-cli
Pull request #7137 (2017-06-21 23:24:47)

add in-app testing page reference
Feel free to veto, but it seemed to me that a reference to this section should be called out in the readme.


Patch:
diff --git a/blueprints/app/files/README.md b/blueprints/app/files/README.md
index 0223e5eb5..3e30c30a2 100644
--- a/blueprints/app/files/README.md
+++ b/blueprints/app/files/README.md
@@ -23,6 +23,7 @@ You will need the following things properly installed on your computer.
 
 * `ember serve`
 * Visit your app at [http://localhost:4200](http://localhost:4200).
+* Visit your tests at [http://localhost:4200/tests](http://localhost:4200/tests).
 
 ### Code Generators
 


Test patch:
diff --git a/tests/fixtures/app/npm/README.md b/tests/fixtures/app/npm/README.md
index 38eea8b99..cd91257af 100644
--- a/tests/fixtures/app/npm/README.md
+++ b/tests/fixtures/app/npm/README.md
@@ -22,6 +22,7 @@ You will need the following things properly installed on your computer.
 
 * `ember serve`
 * Visit your app at [http://localhost:4200](http://localhost:4200).
+* Visit your tests at [http://localhost:4200/tests](http://localhost:4200/tests).
 
 ### Code Generators
 
diff --git a/tests/fixtures/app/yarn/README.md b/tests/fixtures/app/yarn/README.md
index 4d2743f13..6e440cbf2 100644
--- a/tests/fixtures/app/yarn/README.md
+++ b/tests/fixtures/app/yarn/README.md
@@ -23,6 +23,7 @@ You will need the following things properly installed on your computer.
 
 * `ember serve`
 * Visit your app at [http://localhost:4200](http://localhost:4200).
+* Visit your tests at [http://localhost:4200/tests](http://localhost:4200/tests).
 
 ### Code Generators
Repository: ember-cli/ember-cli
Pull request #7180 (2017-07-04 03:18:06)

install npm 4 in addon travis using npm
I was using node 4 in some addons, and bundled npm 2 makes it unbearably slow. npm 4 makes it a lot better. npm 5 was not stable enough in my testing.


Patch:
diff --git a/blueprints/addon/files/.travis.yml b/blueprints/addon/files/.travis.yml
index 53d601d93..00a93f630 100644
--- a/blueprints/addon/files/.travis.yml
+++ b/blueprints/addon/files/.travis.yml
@@ -41,6 +41,7 @@ install:
   - yarn install --no-lockfile
 <% } else { %>
   - npm config set spin false
+  - npm install -g npm@4
   - npm --version
 <% } %>
 script:


Test patch:
diff --git a/tests/fixtures/addon/npm/.travis.yml b/tests/fixtures/addon/npm/.travis.yml
index 0801ec37a..8ebd232a8 100644
--- a/tests/fixtures/addon/npm/.travis.yml
+++ b/tests/fixtures/addon/npm/.travis.yml
@@ -32,6 +32,7 @@ matrix:
 
 before_install:
   - npm config set spin false
+  - npm install -g npm@4
   - npm --version
 
 script:
Repository: ember-cli/ember-cli
Pull request #7224 (2017-07-14 03:56:52)

context issue fix

stefanpenner: @bekzod mind adding a test to ensure this doesn't regress
bekzod: @stefanpenner ok, but it won't be quick :)
stefanpenner: @bekzod no worries. Although I suspect you can simply add an assertion here https://github.com/ember-cli/ember-cli/blob/93c4fd5763fa69fcb11eb0bee1b6b38e57ad4d15/tests/unit/models/builder-test.js#L206
bekzod: @stefanpenner done, thanks for hint 👍 
stefanpenner: rebuilding node 4 windows.
stefanpenner: I believe this will need to be a bugfix, @Turbo87 can you confirm?
Turbo87: @stefanpenner if I read https://github.com/ember-cli/ember-cli/commit/6650df901e75050fba5840459f60588d95a80a84 correctly the bug was introduced in the 2.15 branch, so we only need to fix it there. @bekzod can you change the target branch of the PR to `beta`?
bekzod: @Turbo87 what is easiest way changing target ? or better to remake PR ? 
Turbo87: @bekzod what you did above was correct, but you'll also need to adjust your branch. probably the easiest way is:

- copy the sha1 of the bugfix commit (3f2d5618801a48cdf911e3ca0b1328cba6ffe6f9)
- reset your `this-fix` branch to the current `beta` branch (`git reset --hard origin/beta`)
- cherry pick the bugfix commit on top (`git cherry-pick 3f2d5618801a48cdf911e3ca0b1328cba6ffe6f9`)
- force push to update this PR (`git push --force`)

Patch:
diff --git a/lib/models/builder.js b/lib/models/builder.js
index ac862e468..ec375fa96 100644
--- a/lib/models/builder.js
+++ b/lib/models/builder.js
@@ -135,7 +135,7 @@ class Builder extends CoreObject {
       addonPromises = this.project.addons.reduce((sum, addon) => {
         let method = addon[buildStep];
         if (method) {
-          let val = method(results);
+          let val = method.call(addon, results);
           if (val) { sum.push(val); }
         }
         return sum;


Test patch:
diff --git a/tests/unit/models/builder-test.js b/tests/unit/models/builder-test.js
index 5cef7a289..7d67b0d64 100644
--- a/tests/unit/models/builder-test.js
+++ b/tests/unit/models/builder-test.js
@@ -203,6 +203,7 @@ describe('models/builder.js', function() {
         name: 'TestAddon',
         preBuild() {
           hooksCalled.push('preBuild');
+          expect(this).to.equal(addon);
 
           return Promise.resolve();
         },
Repository: ember-cli/ember-cli
Pull request #7272 (2017-07-30 17:23:14)

Fixes `undefined` values in merged aliases
closes https://github.com/ember-cli/ember-cli/issues/7271


Patch:
diff --git a/lib/models/command.js b/lib/models/command.js
index 94536d174..1b45c832a 100644
--- a/lib/models/command.js
+++ b/lib/models/command.js
@@ -524,7 +524,8 @@ let Command = CoreObject.extend({
         let index = this.availableOptions.indexOf(option);
         let aliasIndex = this.availableOptions[index].aliases.indexOf(alias.original);
         if (this.availableOptions[index].aliases[aliasIndex]) {
-          delete this.availableOptions[index].aliases[aliasIndex];
+          // first one wins
+          this.availableOptions[index].aliases = [this.availableOptions[index].aliases[0]];
         }
       }
       return false;


Test patch:
diff --git a/tests/unit/models/command-test.js b/tests/unit/models/command-test.js
index e057455ee..75a73a2a8 100644
--- a/tests/unit/models/command-test.js
+++ b/tests/unit/models/command-test.js
@@ -533,7 +533,6 @@ describe('models/command.js', function() {
         default: true,
         aliases: [
           { 'mild': false },
-          undefined,
         ],
         key: 'spicy',
         required: false,
Repository: ember-cli/ember-cli
Pull request #7436 (2017-11-11 19:44:03)

retire 2.8, introduce 2.16
The time has come on beta (2.17) to retire 2.8 and introduce 2.16 LTS.

I think I got everything.


Patch:
diff --git a/blueprints/addon/files/.travis.yml b/blueprints/addon/files/.travis.yml
index 4c24b6b71..6971fe52f 100644
--- a/blueprints/addon/files/.travis.yml
+++ b/blueprints/addon/files/.travis.yml
@@ -25,8 +25,8 @@ env:
   matrix:
     # we recommend new addons test the current and previous LTS
     # as well as latest stable release (bonus points to beta/canary)
-    - EMBER_TRY_SCENARIO=ember-lts-2.8
     - EMBER_TRY_SCENARIO=ember-lts-2.12
+    - EMBER_TRY_SCENARIO=ember-lts-2.16
     - EMBER_TRY_SCENARIO=ember-release
     - EMBER_TRY_SCENARIO=ember-beta
     - EMBER_TRY_SCENARIO=ember-canary
diff --git a/blueprints/addon/files/addon-config/ember-try.js b/blueprints/addon/files/addon-config/ember-try.js
index 21f607970..3eea72b9a 100644
--- a/blueprints/addon/files/addon-config/ember-try.js
+++ b/blueprints/addon/files/addon-config/ember-try.js
@@ -2,26 +2,18 @@
 module.exports = {
   scenarios: [
     {
-      name: 'ember-lts-2.8',
-      bower: {
-        dependencies: {
-          'ember': 'components/ember#lts-2-8'
-        },
-        resolutions: {
-          'ember': 'lts-2-8'
-        }
-      },
+      name: 'ember-lts-2.12',
       npm: {
         devDependencies: {
-          'ember-source': null
+          'ember-source': '~2.12.0'
         }
       }
     },
     {
-      name: 'ember-lts-2.12',
+      name: 'ember-lts-2.16',
       npm: {
         devDependencies: {
-          'ember-source': '~2.12.0'
+          'ember-source': '~2.16.0'
         }
       }
     },


Test patch:
diff --git a/tests/fixtures/addon/npm/.travis.yml b/tests/fixtures/addon/npm/.travis.yml
index ac366b8b3..38b4be764 100644
--- a/tests/fixtures/addon/npm/.travis.yml
+++ b/tests/fixtures/addon/npm/.travis.yml
@@ -22,8 +22,8 @@ env:
   matrix:
     # we recommend new addons test the current and previous LTS
     # as well as latest stable release (bonus points to beta/canary)
-    - EMBER_TRY_SCENARIO=ember-lts-2.8
     - EMBER_TRY_SCENARIO=ember-lts-2.12
+    - EMBER_TRY_SCENARIO=ember-lts-2.16
     - EMBER_TRY_SCENARIO=ember-release
     - EMBER_TRY_SCENARIO=ember-beta
     - EMBER_TRY_SCENARIO=ember-canary
diff --git a/tests/fixtures/addon/yarn/.travis.yml b/tests/fixtures/addon/yarn/.travis.yml
index defff0fc4..9f30c1837 100644
--- a/tests/fixtures/addon/yarn/.travis.yml
+++ b/tests/fixtures/addon/yarn/.travis.yml
@@ -21,8 +21,8 @@ env:
   matrix:
     # we recommend new addons test the current and previous LTS
     # as well as latest stable release (bonus points to beta/canary)
-    - EMBER_TRY_SCENARIO=ember-lts-2.8
     - EMBER_TRY_SCENARIO=ember-lts-2.12
+    - EMBER_TRY_SCENARIO=ember-lts-2.16
     - EMBER_TRY_SCENARIO=ember-release
     - EMBER_TRY_SCENARIO=ember-beta
     - EMBER_TRY_SCENARIO=ember-canary
Repository: ember-cli/ember-cli
Pull request #7477 (2017-12-01 02:31:41)

Restore `separator: '\n;'` to vendor JS concat
Closes #7470

Retargeted the commit:

Prior to 2.17, Ember CLI used a semicolon to separate files concatenated into vendor.js: https://github.com/ember-cli/ember-cli/blob/v2.16.2/lib/broccoli/bundler.js#L92

Vendor shims are still generated with a leading paren:
```
(function() {
  function vendorModule() {
    'use strict';

    return {
      'default': self['test'],
      __esModule: true,
    };
  }

  define('test', [], vendorModule);
})();
```

This can cause `TypeError: (intermediate value)(...) is not a function` if the code immediately prior does not end in a semicolon.
.


Patch:
diff --git a/lib/broccoli/strategies.js b/lib/broccoli/strategies.js
index a7ed58a87..5e9377ebe 100644
--- a/lib/broccoli/strategies.js
+++ b/lib/broccoli/strategies.js
@@ -119,6 +119,7 @@ module.exports = {
       footerFiles: vendorObject.footerFiles,
       outputFile: options.outputFile,
       annotation: options.annotation,
+      separator: '\n;',
       sourceMapConfig: options.sourceMapConfig,
     });
   },


Test patch:
diff --git a/tests/unit/broccoli/strategies-test.js b/tests/unit/broccoli/strategies-test.js
index cd767746d..3776038a8 100644
--- a/tests/unit/broccoli/strategies-test.js
+++ b/tests/unit/broccoli/strategies-test.js
@@ -69,6 +69,7 @@ describe('assembler strategies', function() {
         ],
         inputFiles: [],
         outputFile: '/assets/vendor.js',
+        separator: '\n;',
         sourceMapConfig: {
           'enabled': true,
         },
Repository: ember-cli/ember-cli
Pull request #7537 (2018-01-02 23:21:03)

[BUGFIX release] Dummy app files follow Ember rules
Addons should use Ember linting rules for their `tests/dummy/app/` directories. In 2.18.0 the default configuration has `index.js` files in the `tests/dummy/app/` directory using CJS rules instead.

For example:

```
> ember --version
ember-cli: 2.18.0
node: 9.3.0
os: darwin x64
> ember g addon demo-eslint-index
> cd demo-eslint-index
> echo "import Controller from '@ember/controller'; export default Controller.extend();" > tests/dunny/app/controllers/index.js
> yarn lint:js
yarn run v1.3.2
$ eslint ./*.js addon addon-test-support app config lib server test-support tests

/Users/mixonic/Projects/demo-eslint-index/tests/dummy/app/controllers/index.js
  1:1  error  Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

✖ 1 problem (1 error, 0 warnings)

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
```

This patch fixes the default configuration to exclude `tests/dummy/app/` from node rules entirely. The ember rules are then permitted to apply on the `index.js` files. It is basically a workaround for https://github.com/eslint/eslint/issues/9740 which would permit the `index.js` rule to be absolute.

rwjblue: Thank you!

Patch:
diff --git a/blueprints/app/files/.eslintrc.js b/blueprints/app/files/.eslintrc.js
index 9a0b8badb..f89e28050 100644
--- a/blueprints/app/files/.eslintrc.js
+++ b/blueprints/app/files/.eslintrc.js
@@ -28,7 +28,8 @@ module.exports = {
       ],<% if (blueprint !== 'app') { %>
       excludedFiles: [
         'app/**',
-        'addon/**'
+        'addon/**',
+        'tests/dummy/app/**'
       ],<% } %>
       parserOptions: {
         sourceType: 'script',


Test patch:
diff --git a/tests/fixtures/addon/npm/.eslintrc.js b/tests/fixtures/addon/npm/.eslintrc.js
index 1c65e6db0..3a07c6bcc 100644
--- a/tests/fixtures/addon/npm/.eslintrc.js
+++ b/tests/fixtures/addon/npm/.eslintrc.js
@@ -28,7 +28,8 @@ module.exports = {
       ],
       excludedFiles: [
         'app/**',
-        'addon/**'
+        'addon/**',
+        'tests/dummy/app/**'
       ],
       parserOptions: {
         sourceType: 'script',
diff --git a/tests/fixtures/addon/yarn/.eslintrc.js b/tests/fixtures/addon/yarn/.eslintrc.js
index 1c65e6db0..3a07c6bcc 100644
--- a/tests/fixtures/addon/yarn/.eslintrc.js
+++ b/tests/fixtures/addon/yarn/.eslintrc.js
@@ -28,7 +28,8 @@ module.exports = {
       ],
       excludedFiles: [
         'app/**',
-        'addon/**'
+        'addon/**',
+        'tests/dummy/app/**'
       ],
       parserOptions: {
         sourceType: 'script',
Repository: ember-cli/ember-cli
Pull request #7541 (2018-01-04 13:38:18)

Add `project.isModuleUnification()`
part of https://github.com/emberjs/ember.js/pull/16043
(which is part of https://github.com/ember-cli/ember-cli/issues/7530)

This PR adds initial support for `ember g *` commands within a module unification project. The existence of a `src` directory in the ember project determines if that project is a module unification project. After this is merged, the next step will be to update the [Ember blueprints](https://github.com/emberjs/ember.js/tree/master/blueprints) to support MU output by checking `project.isModuleUnification()`

/cc @rwjblue 
  
  


Patch:
diff --git a/lib/models/project.js b/lib/models/project.js
index 1c3a0e516..30f639256 100644
--- a/lib/models/project.js
+++ b/lib/models/project.js
@@ -141,6 +141,10 @@ class Project {
       return false;
     };
 
+    NULL_PROJECT.isModuleUnification = function() {
+      return false;
+    };
+
     NULL_PROJECT.name = function() {
       return path.basename(process.cwd());
     };
@@ -185,6 +189,16 @@ class Project {
     return !!this.pkg.keywords && this.pkg.keywords.indexOf('ember-addon') > -1;
   }
 
+  /**
+     Returns whether this is using a module unification format.
+     @private
+     @method isModuleUnification
+     @return {Boolean} Whether this is using a module unification format.
+    */
+  isModuleUnification() {
+    return this.has('src');
+  }
+
   /**
     Returns the path to the configuration.
 


Test patch:
diff --git a/tests/helpers/mock-project.js b/tests/helpers/mock-project.js
index ec8c11678..d9e282eb9 100644
--- a/tests/helpers/mock-project.js
+++ b/tests/helpers/mock-project.js
@@ -63,6 +63,10 @@ class MockProject extends Project {
   isEmberCLIAddon() {
     return false;
   }
+
+  isModuleUnification() {
+    return false;
+  }
 }
 
 module.exports = MockProject;
diff --git a/tests/unit/models/project-test.js b/tests/unit/models/project-test.js
index be334c9b6..5f8f5abe5 100644
--- a/tests/unit/models/project-test.js
+++ b/tests/unit/models/project-test.js
@@ -526,6 +526,25 @@ describe('models/project.js', function() {
     });
   });
 
+  describe('isModuleUnification', function() {
+    beforeEach(function() {
+      projectPath = `${process.cwd()}/tmp/test-app`;
+
+      makeProject();
+    });
+
+    it('returns false when `./src` does not exist', function() {
+      expect(project.isModuleUnification()).to.equal(false);
+    });
+
+    it('returns true when `./src` exists', function() {
+      let srcPath = path.join(projectPath, 'src');
+      fs.ensureDirSync(srcPath);
+
+      expect(project.isModuleUnification()).to.equal(true);
+    });
+  });
+
   describe('findAddonByName', function() {
     beforeEach(function() {
       projectPath = `${process.cwd()}/tmp/test-app`;
Repository: ember-cli/ember-cli
Pull request #7569 (2018-01-22 20:25:59)

mark "lib" folder as node style in eslint for apps
in-repo addons will fail `npm run lint:js` without this.


Patch:
diff --git a/blueprints/app/files/.eslintrc.js b/blueprints/app/files/.eslintrc.js
index f89e28050..8c6214860 100644
--- a/blueprints/app/files/.eslintrc.js
+++ b/blueprints/app/files/.eslintrc.js
@@ -23,7 +23,8 @@ module.exports = {
         'index.js',<% } %>
         'testem.js',
         'ember-cli-build.js',
-        'config/**/*.js'<% if (blueprint !== 'app') { %>,
+        'config/**/*.js'<% if (blueprint === 'app') { %>,
+        'lib/*/index.js'<% } %><% if (blueprint !== 'app') { %>,
         'tests/dummy/config/**/*.js'<% } %>
       ],<% if (blueprint !== 'app') { %>
       excludedFiles: [


Test patch:
diff --git a/tests/fixtures/app/npm/.eslintrc.js b/tests/fixtures/app/npm/.eslintrc.js
index 1f7a6c0bd..e05af46cb 100644
--- a/tests/fixtures/app/npm/.eslintrc.js
+++ b/tests/fixtures/app/npm/.eslintrc.js
@@ -22,7 +22,8 @@ module.exports = {
       files: [
         'testem.js',
         'ember-cli-build.js',
-        'config/**/*.js'
+        'config/**/*.js',
+        'lib/*/index.js'
       ],
       parserOptions: {
         sourceType: 'script',
diff --git a/tests/fixtures/app/yarn/.eslintrc.js b/tests/fixtures/app/yarn/.eslintrc.js
index 1f7a6c0bd..e05af46cb 100644
--- a/tests/fixtures/app/yarn/.eslintrc.js
+++ b/tests/fixtures/app/yarn/.eslintrc.js
@@ -22,7 +22,8 @@ module.exports = {
       files: [
         'testem.js',
         'ember-cli-build.js',
-        'config/**/*.js'
+        'config/**/*.js',
+        'lib/*/index.js'
       ],
       parserOptions: {
         sourceType: 'script',
Repository: ember-cli/ember-cli
Pull request #7605 (2018-02-05 13:48:25)

blueprints/app: Add `qunit-dom` dependency by default
Resolves https://github.com/emberjs/ember-qunit/issues/301

**Reasons:**

- because it makes DOM assertions so much more readable

/cc @ember-cli/core @trentmwillis @cibernox 

rwjblue: I'm definitely in favor, but I think we likely need an RFC first...

Patch:
diff --git a/blueprints/app/files/package.json b/blueprints/app/files/package.json
index ad64a76e6..4e50d04f6 100644
--- a/blueprints/app/files/package.json
+++ b/blueprints/app/files/package.json
@@ -39,7 +39,8 @@
     "ember-source": "~3.1.0-beta.1<% if (welcome) { %>",
     "ember-welcome-page": "^3.0.0<% } %>",
     "eslint-plugin-ember": "^5.0.0",
-    "loader.js": "^4.2.3"
+    "loader.js": "^4.2.3",
+    "qunit-dom": "^0.5.0"
   },
   "engines": {
     "node": "^4.5 || 6.* || >= 7.*"


Test patch:
diff --git a/tests/fixtures/addon/npm/package.json b/tests/fixtures/addon/npm/package.json
index 655a2558f..5d8c31e8d 100644
--- a/tests/fixtures/addon/npm/package.json
+++ b/tests/fixtures/addon/npm/package.json
@@ -45,7 +45,8 @@
     "ember-try": "^0.2.23",
     "eslint-plugin-ember": "^5.0.0",
     "eslint-plugin-node": "^5.2.1",
-    "loader.js": "^4.2.3"
+    "loader.js": "^4.2.3",
+    "qunit-dom": "^0.5.0"
   },
   "engines": {
     "node": "^4.5 || 6.* || >= 7.*"
diff --git a/tests/fixtures/addon/yarn/package.json b/tests/fixtures/addon/yarn/package.json
index e076e9da6..e23cd0ba6 100644
--- a/tests/fixtures/addon/yarn/package.json
+++ b/tests/fixtures/addon/yarn/package.json
@@ -46,7 +46,8 @@
     "ember-welcome-page": "^3.0.0",
     "eslint-plugin-ember": "^5.0.0",
     "eslint-plugin-node": "^5.2.1",
-    "loader.js": "^4.2.3"
+    "loader.js": "^4.2.3",
+    "qunit-dom": "^0.5.0"
   },
   "engines": {
     "node": "^4.5 || 6.* || >= 7.*"
diff --git a/tests/fixtures/app/npm/package.json b/tests/fixtures/app/npm/package.json
index 62a5e5add..d96f914b4 100644
--- a/tests/fixtures/app/npm/package.json
+++ b/tests/fixtures/app/npm/package.json
@@ -38,7 +38,8 @@
     "ember-resolver": "^4.0.0",
     "ember-source": "~3.1.0-beta.1",
     "eslint-plugin-ember": "^5.0.0",
-    "loader.js": "^4.2.3"
+    "loader.js": "^4.2.3",
+    "qunit-dom": "^0.5.0"
   },
   "engines": {
     "node": "^4.5 || 6.* || >= 7.*"
diff --git a/tests/fixtures/app/yarn/package.json b/tests/fixtures/app/yarn/package.json
index 149ecd981..16abec4a5 100644
--- a/tests/fixtures/app/yarn/package.json
+++ b/tests/fixtures/app/yarn/package.json
@@ -39,7 +39,8 @@
     "ember-source": "~3.1.0-beta.1",
     "ember-welcome-page": "^3.0.0",
     "eslint-plugin-ember": "^5.0.0",
-    "loader.js": "^4.2.3"
+    "loader.js": "^4.2.3",
+    "qunit-dom": "^0.5.0"
   },
   "engines": {
     "node": "^4.5 || 6.* || >= 7.*"