Remove use of Ember CLI's ui instance

Author: bertdeblock

lib/commands/config.js

@@ -1,6 +1,7 @@
 'use strict';
 
 const debug = require('debug')('ember-try:commands:config');
+const { log } = require('../utils/console');
 
 module.exports = {
   name: 'try:config',
@@ -16,6 +17,6 @@ module.exports = {
       configPath: commandOptions.configPath,
     });
 
-    this.ui.writeLine(JSON.stringify(config, null, 2));
+    log(JSON.stringify(config, null, 2));
   },
 };

lib/commands/try-each.js

@@ -26,7 +26,6 @@ module.exports = {
     debug('Config: %s', JSON.stringify(config));
 
     let tryEachTask = new this._TryEachTask({
-      ui: this.ui,
       project: this.project,
       config,
     });

lib/commands/try-ember.js

@@ -35,7 +35,6 @@ module.exports = {
     debug('Config: %s', JSON.stringify(config));
 
     let tryEachTask = new this._TryEachTask({
-      ui: this.ui,
       project: this.project,
       config,
     });

lib/commands/try-one.js

@@ -58,7 +58,6 @@ module.exports = {
     }
 
     let tryEachTask = new this._TryEachTask({
-      ui: this.ui,
       project: this.project,
       config,
       commandArgs,

lib/dependency-manager-adapters/base.js

@@ -1,12 +1,12 @@
 'use strict';
 
-const chalk = require('chalk');
 const debug = require('debug');
 const { get, set } = require('es-toolkit/compat');
 const fs = require('fs-extra');
 const path = require('node:path');
 const semver = require('semver');
 const Backup = require('../utils/backup');
+const { warn } = require('../utils/console');
 const { LOCKFILE, PACKAGE_JSON } = require('../utils/package-managers');
 
 class BaseAdapter {
@@ -35,8 +35,8 @@ class BaseAdapter {
     this.debugFunction(...args);
   }
 
-  async setup(options = {}) {
-    this._checkForDifferentLockfiles(options.ui);
+  async setup() {
+    this._checkForDifferentLockfiles();
 
     await this.backup.addFiles([PACKAGE_JSON, this.lockfile]);
   }
@@ -87,7 +87,7 @@ class BaseAdapter {
     }
   }
 
-  _checkForDifferentLockfiles(ui) {
+  _checkForDifferentLockfiles() {
     for (const packageManager in LOCKFILE) {
       const lockfile = LOCKFILE[packageManager];
 
@@ -97,10 +97,8 @@ class BaseAdapter {
 
       try {
         if (fs.statSync(path.join(this.cwd, lockfile)).isFile()) {
-          ui.writeLine(
-            chalk.yellow(
-              `Detected a \`${lockfile}\` file. Add \`packageManager: '${packageManager}'\` to your \`config/ember-try.js\` configuration file if you want to use ${packageManager} to install dependencies.`,
-            ),
+          warn(
+            `Detected a \`${lockfile}\` file. Add \`packageManager: '${packageManager}'\` to your \`config/ember-try.js\` configuration file if you want to use ${packageManager} to install dependencies.`,
           );
         }
       } catch {

lib/dependency-manager-adapters/pnpm.js

@@ -10,9 +10,9 @@ module.exports = class PnpmAdapter extends BaseAdapter {
   name = 'pnpm';
   overridesKey = 'pnpm.overrides';
 
-  async setup(options) {
+  async setup() {
     await this._throwOnResolutionMode();
-    await super.setup(options);
+    await super.setup();
   }
 
   /**

lib/dependency-manager-adapters/workspace.js

@@ -54,8 +54,8 @@ module.exports = class {
     });
   }
 
-  setup(options) {
-    return Promise.all(this._packageAdapters.map((adapter) => adapter.setup(options)));
+  setup() {
+    return Promise.all(this._packageAdapters.map((adapter) => adapter.setup()));
   }
 
   async changeToDependencySet(depSet) {

lib/tasks/try-each.js

@@ -1,8 +1,8 @@
 'use strict';
 
-const chalk = require('chalk');
 const debug = require('debug')('ember-try:task:try-each');
-const runCommand = require('./../utils/run-command');
+const { error, info, log } = require('../utils/console');
+const runCommand = require('../utils/run-command');
 
 module.exports = class TryEachTask {
   constructor(options) {
@@ -11,7 +11,6 @@ module.exports = class TryEachTask {
     this.config = options.config;
     this.dependencyManagerAdapters = options.dependencyManagerAdapters;
     this.project = options.project;
-    this.ui = options.ui;
   }
 
   async run(scenarios, options) {
@@ -30,14 +29,13 @@ module.exports = class TryEachTask {
       }),
     );
     this.ScenarioManager = new ScenarioManager({
-      ui: this.ui,
       dependencyManagerAdapters,
     });
 
     this._canceling = false;
     this._on('SIGINT', () => {
       this._canceling = true;
-      this.ui.writeLine('\nGracefully shutting down from SIGINT (Ctrl-C)');
+      log('\nGracefully shutting down from SIGINT (Ctrl-C)');
       return this.ScenarioManager.cleanup();
     });
 
@@ -57,11 +55,11 @@ module.exports = class TryEachTask {
 
       return this._exitAsAppropriate(results);
     } catch (err) {
-      this.ui.writeLine(chalk.red('Error!'));
+      error('Error!');
 
       if (err) {
-        this.ui.writeLine(chalk.red(err));
-        this.ui.writeLine(chalk.red(err.stack));
+        error(err);
+        error(err.stack);
       }
 
       return 1; // Signifies exit code
@@ -111,12 +109,12 @@ module.exports = class TryEachTask {
   _writeHeader(text) {
     let count = 75 - text.length;
     let separator = new Array(count + 1).join('=');
-    this.ui.writeLine(chalk.blue(`\n=== ${text} ${separator}\n`));
+    info(`\n=== ${text} ${separator}\n`);
   }
 
   _writeFooter(text) {
-    this.ui.writeLine(chalk.blue(`\n${text}`));
-    this.ui.writeLine(chalk.blue('---\n'));
+    info(`\n${text}`);
+    info('---\n');
   }
 
   _determineCommandFor(scenario) {
@@ -152,7 +150,7 @@ module.exports = class TryEachTask {
   }
 
   _printResults(results) {
-    new this.ResultSummary({ ui: this.ui, results }).print();
+    new this.ResultSummary({ results }).print();
   }
 
   _exitAsAppropriate(results) {

lib/utils/config.js

@@ -1,8 +1,8 @@
 'use strict';
 
-const chalk = require('chalk');
 const path = require('path');
 const fs = require('fs');
+const { prefix, warn } = require('./console');
 const findByName = require('./find-by-name');
 const debug = require('debug')('ember-try:utils:config');
 
@@ -70,10 +70,8 @@ async function config(options) {
     const [name, oldOption] = packageManager;
 
     if (typeof configData[oldOption] === 'boolean') {
-      console.warn(
-        chalk.yellow(
-          `${chalk.inverse(' ember-try DEPRECATION ')} The \`${oldOption}\` config option is deprecated. Please use \`packageManager: '${name}'\` instead.`,
-        ),
+      warn(
+        `${prefix('ember-try DEPRECATION')} The \`${oldOption}\` config option is deprecated. Please use \`packageManager: '${name}'\` instead.`,
       );
 
       delete configData[oldOption];

lib/utils/console.js

@@ -0,0 +1,51 @@
+const chalk = require('chalk');
+
+let logFunction = originalLogFunction;
+
+function error(message) {
+  logFunction(message, console.error, chalk.red);
+}
+
+function info(message) {
+  logFunction(message, console.info, chalk.blue);
+}
+
+function log(message) {
+  logFunction(message, console.log);
+}
+
+function prefix(string) {
+  return chalk.inverse(` ${string} `);
+}
+
+function success(message) {
+  logFunction(message, console.log, chalk.green);
+}
+
+function warn(message) {
+  logFunction(message, console.warn, chalk.yellow);
+}
+
+function originalLogFunction(message, consoleLogFunction, chalkColorFunction) {
+  consoleLogFunction(chalkColorFunction ? chalkColorFunction(message) : message);
+}
+
+function mockLog(mockedLogFunction) {
+  logFunction = mockedLogFunction;
+}
+
+function restoreLog() {
+  logFunction = originalLogFunction;
+}
+
+module.exports = {
+  error,
+  info,
+  log,
+  prefix,
+  success,
+  warn,
+
+  _mockLog: mockLog,
+  _restoreLog: restoreLog,
+};