Fix the new Runner

Author: gitKrystan

bin/cli.js

@@ -1,10 +1,12 @@
 #!/usr/bin/env node
 'use strict';
 
-const Runner = require('../transforms/no-implicit-this/helpers/runner');
+const Runner = require('../transforms/no-implicit-this/helpers/runner').default;
 
-Runner.withArgs(process.argv.slice(2))
-  .run()
+const args = process.argv.slice(2);
+
+Runner.withArgs(args)
+  .run(__dirname, args)
   .catch((error) => {
     console.error(error);
     process.exit(1);

test/fixtures/3.28/input/app/components/gherkyn.hbs

@@ -1,6 +1,6 @@
 {{! template-lint-disable no-implicit-this }}
 {{! template-lint-disable no-curly-component-invocation }}
-<h1>[{{this.property}}]</h1>
+<h1>[{{property}}]</h1>
 
 <h1>[{{this.property}}]</h1>
 

test/fixtures/3.28/output/app/components/.gitkeep

@@ -27,15 +27,11 @@ export class TestRunner {
   }
 
   async runCodemodRuntime() {
-    await execa(
-      '../../../../bin/cli.js',
-      ['http://localhost:4200', 'app', '--telemetry=runtime'],
-      this.execOpts
-    );
+    await execa('../../../../bin/cli.js', ['--root=app', '--telemetry=runtime'], this.execOpts);
   }
 
   async runCodemodEmbroider() {
-    await execa('../../../../bin/cli.js', ['app', '--telemetry=embroider'], this.execOpts);
+    await execa('../../../../bin/cli.js', ['--root=app', '--telemetry=embroider'], this.execOpts);
   }
 
   async runEmbroiderStage2Build(): Promise<void> {

test/helpers/test-runner.ts

@@ -13,12 +13,18 @@ import { assert, isRecord } from '../../../helpers/types';
 const debug = Debug('ember-no-implicit-this-codemod');
 
 export interface Options {
+  root: string;
   config: string | undefined;
   telemetry: 'auto' | 'embroider' | 'runtime';
   url: string;
 }
 
 export const Parser = yargs
+  .option('root', {
+    describe: 'app root FIXME',
+    default: 'app',
+    normalize: true,
+  })
   .option('config', {
     describe: 'Path to config file FIXME',
     normalize: true,
@@ -64,6 +70,7 @@ export default class Runner {
 
   static withOptions(options: Partial<Options> = {}): Runner {
     return new Runner({
+      root: 'app',
       config: undefined,
       telemetry: 'auto',
       url: 'http://localhost:4200',
@@ -113,7 +120,7 @@ export default class Runner {
     }
   }
 
-  async run(): Promise<void> {
+  async run(root: string, args: string[]): Promise<void> {
     const telemetryType = await this.detectTelemetryType();
 
     if (telemetryType === 'runtime') {
@@ -124,7 +131,7 @@ export default class Runner {
       debug('Gathered telemetry on %d modules', Object.keys(telemetry).length);
     }
 
-    runTransform(__dirname, 'no-implicit-this', process.argv, 'hbs');
+    runTransform(root, 'no-implicit-this', [this.options.root, ...args], 'hbs');
   }
 
   private async detect(): Promise<DetectResult> {