support NullVoxPopuli/ember-cli-update-codemods-manifest/add-no-implicit-this

Author: Kelly Selden

src/index.js

@@ -194,7 +194,7 @@ All blueprints are up-to-date!`;
     statsOnly,
     listCodemods,
     runCodemods,
-    codemodsUrl: 'https://raw.githubusercontent.com/ember-cli/ember-cli-update-codemods-manifest/v3/manifest.json',
+    codemodsUrl: 'https://raw.githubusercontent.com/ember-cli/ember-cli-update-codemods-manifest/ember-no-implicit-this-codemod/manifest.json',
     createCustomDiff,
     ignoredFiles: [await getBlueprintFilePath(cwd)],
     wasRunAsExecutable

test/acceptance/ember-cli-update-test.js

@@ -146,9 +146,14 @@ describe(function() {
     } = await merge({
       fixturesPath: 'test/fixtures/codemod/local',
       commitMessage: 'my-app',
-      runCodemods: true
+      runCodemods: true,
+      async beforeMerge() {
+        await run('npm install', { cwd: tmpPath });
+      }
     });
 
+    ps.stdout.pipe(process.stdout);
+
     ps.stdout.on('data', data => {
       let str = data.toString();
       if (str.includes('These codemods apply to your project.')) {
@@ -160,9 +165,17 @@ describe(function() {
       status
     } = await promise;
 
+    assertNoUnstaged(status);
+    assertCodemodRan(status);
+
+    await fs.remove(path.join(tmpPath, 'package-lock.json'));
+
     // file is indeterminent between OS's, so ignore
     await fs.remove(path.join(tmpPath, 'MODULE_REPORT.md'));
 
+    // remove dist and node_modules before fixture compare
+    await run('git clean -fdX', { cwd: tmpPath });
+
     let mergeFixtures = 'test/fixtures/codemod/latest-node/my-app';
     if (process.env.NODE_LTS) {
       mergeFixtures = 'test/fixtures/codemod/min-node/my-app';
@@ -171,9 +184,6 @@ describe(function() {
     fixtureCompare({
       mergeFixtures
     });
-
-    assertNoUnstaged(status);
-    assertCodemodRan(status);
   });
 
   it('scopes to sub dir if run from there', async function() {

test/fixtures/codemod/latest-node/my-app/app/controllers/application.js

@@ -4,6 +4,8 @@ import { assign } from '@ember/polyfills';
 import { map } from '@ember/object/computed';
 
 export default Controller.extend({
+  controllerProperty: 'test',
+
   fullName: computed('firstName', 'lastName', function() {
     return `${this.firstName} ${this.lastName}`;
   }),

test/fixtures/codemod/latest-node/my-app/app/templates/application.hbs

@@ -2,4 +2,6 @@
 <WelcomePage />
 {{!-- Feel free to remove this! --}}
 
+{{this.controllerProperty}}
+
 {{outlet}}

test/fixtures/codemod/latest-node/my-app/codemods.log

@@ -3,6 +3,8 @@ import { merge } from '@ember/polyfills';
 import { map } from '@ember/object/computed';
 
 export default Ember.Controller.extend({
+  controllerProperty: 'test',
+
   fullName: computed(function() {
     return `${this.firstName} ${this.lastName}`;
   }).property('firstName', 'lastName'),

test/fixtures/codemod/local/my-app/app/controllers/application.js

@@ -2,4 +2,6 @@
 <WelcomePage />
 {{!-- Feel free to remove this! --}}
 
+{{controllerProperty}}
+
 {{outlet}}

test/fixtures/codemod/local/my-app/app/templates/application.hbs

@@ -4,6 +4,8 @@ import { assign } from '@ember/polyfills';
 import { map } from '@ember/object/computed';
 
 export default Controller.extend({
+  controllerProperty: 'test',
+
   fullName: computed('firstName', 'lastName', function() {
     return `${this.firstName} ${this.lastName}`;
   }),

test/fixtures/codemod/min-node/my-app/app/controllers/application.js

@@ -2,4 +2,6 @@
 <WelcomePage />
 {{!-- Feel free to remove this! --}}
 
+{{this.controllerProperty}}
+
 {{outlet}}