@abcnews/aunty 14.0.1 → 14.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abcnews/aunty",
-  "version": "14.0.1",
+  "version": "14.0.2",
   "description": "A toolkit for working with ABC News projects",
   "repository": "abcnews/aunty",
   "license": "MIT",
@@ -12,7 +12,8 @@
   ],
   "scripts": {
     "postinstall": "node scripts/postinstall.js",
-    "release": "release-it"
+    "release": "release-it",
+    "test": "jest test/test.js"
   },
   "files": [
     "assets",
@@ -94,6 +95,7 @@
     "babel-eslint": "^10.1.0",
     "eslint": "^7.7.0",
     "eslint-config-prettier": "^8.2.0",
-    "release-it": "^15.1.4"
+    "release-it": "^15.1.4",
+    "yeoman-automation-adapter": "^2.0.0"
   }
 }

package/src/cli/build/index.js

@@ -19,77 +19,82 @@ const { command } = require('../');
 const cleanCommand = require('../clean');
 const { MESSAGES, OPTIONS } = require('./constants');
 
-module.exports = command(
-  {
-    name: 'build',
-    nodeEnv: 'production',
-    options: OPTIONS,
-    usage: MESSAGES.usage
-  },
-  async argv => {
-    const { root } = getProjectConfig();
-    const webpackConfig = getWebpackConfig();
-    let deployConfig;
-
-    if (!argv.local) {
-      deployConfig = getDeployConfig({ id: argv.id });
-      webpackConfig.forEach(config => {
-        config.output.publicPath = deployConfig.targets[0].publicPath;
-      });
-    }
-
-    if (argv.dry) {
-      return dry(
-        combine(
-          {
-            'Webpack config': webpackConfig
-          },
-          deployConfig
-            ? {
-                'Deploy config': deployConfig
-              }
-            : {}
-        )
-      );
-    }
+const build = async argv => {
+  const { root } = getProjectConfig();
+  const webpackConfig = getWebpackConfig();
+  let deployConfig;
 
-    throws(await cleanCommand(['--quiet']));
+  if (!argv.local) {
+    deployConfig = getDeployConfig({ id: argv.id });
+    webpackConfig.forEach(config => {
+      config.output.publicPath = deployConfig.targets[0].publicPath;
+    });
+  }
 
-    info(
-      MESSAGES.build({
-        id: deployConfig ? deployConfig.id : null,
-        publicPaths: deployConfig ? deployConfig.targets.map(x => x.publicPath) : [webpackConfig[0].output.publicPath]
-      })
+  if (argv.dry) {
+    return dry(
+      combine(
+        {
+          'Webpack config': webpackConfig
+        },
+        deployConfig
+          ? {
+              'Deploy config': deployConfig
+            }
+          : {}
+      )
     );
+  }
 
-    let spinner = spin('Building');
-    const startTime = Date.now();
-    const compiler = webpack(webpackConfig);
-    const stats = unpack(await packs(pify(compiler.run.bind(compiler)))());
+  throws(await cleanCommand(['--quiet']));
 
-    if (stats.hasErrors()) {
-      const errors = stats.toJson({}, true).errors;
+  info(
+    MESSAGES.build({
+      id: deployConfig ? deployConfig.id : null,
+      publicPaths: deployConfig ? deployConfig.targets.map(x => x.publicPath) : [webpackConfig[0].output.publicPath]
+    })
+  );
 
-      spinner.fail();
+  let spinner = spin('Building');
+  const startTime = Date.now();
+  const compiler = webpack(webpackConfig);
+  const stats = unpack(await packs(pify(compiler.run.bind(compiler)))());
 
-      if (errors.length > 1) {
-        throw MESSAGES.multipleErrors(errors.map(error => error.message));
-      }
+  if (stats.hasErrors()) {
+    const errors = stats.toJson({}, true).errors;
 
-      throw errors[0];
-    }
+    spinner.fail();
 
-    if (stats.hasWarnings()) {
-      spinner.warn();
-      stats.toJson({}, true).warnings.forEach(warning => warn(warning.message));
-    } else {
-      spinner.succeed(`Built in ${((Date.now() - startTime) / 1000).toFixed(2)}s`);
+    if (errors.length > 1) {
+      throw MESSAGES.multipleErrors(errors.map(error => error.message));
     }
 
-    if (deployConfig) {
-      spinner = spin('Creating deploy configuration');
-      writeJsonFile.sync(join(root, OUTPUT_DIRECTORY_NAME, DEPLOY_FILE_NAME), deployConfig);
-      spinner.succeed('Created deploy configuration');
-    }
+    throw errors[0];
   }
+
+  if (stats.hasWarnings()) {
+    spinner.warn();
+    stats.toJson({}, true).warnings.forEach(warning => warn(warning.message));
+  } else {
+    spinner.succeed(`Built in ${((Date.now() - startTime) / 1000).toFixed(2)}s`);
+  }
+
+  if (deployConfig) {
+    spinner = spin('Creating deploy configuration');
+    writeJsonFile.sync(join(root, OUTPUT_DIRECTORY_NAME, DEPLOY_FILE_NAME), deployConfig);
+    spinner.succeed('Created deploy configuration');
+  }
+};
+
+module.exports = command(
+  {
+    name: 'build',
+    nodeEnv: 'production',
+    options: OPTIONS,
+    usage: MESSAGES.usage
+  },
+  build
 );
+
+// Expose this for the tests
+module.exports._testBuild = build;

package/src/cli/constants.js

@@ -1,7 +1,6 @@
 // Ours
 const { createLogo } = require('../utils/branding');
 const { cmd, dim, hvy, opt, req, sec } = require('../utils/color');
-const { setOfValues } = require('../utils/structures');
 
 const COMMAND_ALIASES = (module.exports.COMMAND_ALIASES = {
   b: 'build',
@@ -14,7 +13,7 @@ const COMMAND_ALIASES = (module.exports.COMMAND_ALIASES = {
   t: 'test'
 });
 
-module.exports.COMMANDS = setOfValues(COMMAND_ALIASES);
+module.exports.COMMANDS = new Set(Object.values(COMMAND_ALIASES));
 
 module.exports.DEFAULTS = {
   name: '__command__',

package/src/cli/generate/constants.js

@@ -1,6 +1,5 @@
 // Ours
 const { cmd, hvy, opt, req, sec } = require('../../utils/color');
-const { setOfValues } = require('../../utils/structures');
 
 module.exports.OPTIONS = {
   boolean: ['announce'],
@@ -14,7 +13,7 @@ const GENERATOR_ALIASES = (module.exports.GENERATOR_ALIASES = {
   p: 'project'
 });
 
-module.exports.GENERATORS = setOfValues(GENERATOR_ALIASES);
+module.exports.GENERATORS = new Set(Object.values(GENERATOR_ALIASES));
 
 module.exports.MESSAGES = {
   generatorDoesNotExist: name => `The generator '${name}' does not exist.`,

package/src/cli/generate/index.js

@@ -10,55 +10,63 @@ const { log } = require('../../utils/logging');
 const { command } = require('../');
 const { GENERATOR_ALIASES, GENERATORS, MESSAGES, OPTIONS } = require('./constants');
 
-module.exports = command(
-  {
-    name: 'generate',
-    hasSubcommands: true,
-    options: OPTIONS
-  },
-  async argv => {
-    if (argv.dry) {
-      throw MESSAGES.noDryRuns;
-    }
-
-    const generatorName = GENERATOR_ALIASES[argv._[0]] || argv._[0];
+const generate = async argv => {
+  if (argv.dry) {
+    throw MESSAGES.noDryRuns;
+  }
 
-    // If we didn't supply a known generator name, blow up, unless we wanted
-    // the generate command's usage message (in which case, print it, then exit).
+  const generatorName = GENERATOR_ALIASES[argv._[0]] || argv._[0];
 
-    if (!GENERATORS.has(generatorName)) {
-      if (!generatorName || generatorName.indexOf('-') === 0 || argv.help) {
-        return log(MESSAGES.usage);
-      }
+  // If we didn't supply a known generator name, blow up, unless we wanted
+  // the generate command's usage message (in which case, print it, then exit).
 
-      throw MESSAGES.generatorDoesNotExist(generatorName);
+  if (!GENERATORS.has(generatorName)) {
+    if (!generatorName || generatorName.indexOf('-') === 0 || argv.help) {
+      return log(MESSAGES.usage);
     }
 
-    log(createCommandLogo(`generate ${generatorName}`));
+    throw MESSAGES.generatorDoesNotExist(generatorName);
+  }
 
-    const env = yeoman.createEnv();
+  log(createCommandLogo(`generate ${generatorName}`));
 
-    // Register the generator
+  // If we're in a test environment, insert the auto adapter with the provided answers
+  const env = global.auntyYeomanAnswers
+    ? yeoman.createEnv([], {}, new (require('yeoman-automation-adapter').AutoAdapter)(global.auntyYeomanAnswers, true))
+    : yeoman.createEnv();
 
-    const generatorPath = require.resolve(`../../generators/${generatorName}`);
+  // Register the generator
 
-    env.register(generatorPath, generatorName);
+  const generatorPath = require.resolve(`../../generators/${generatorName}`);
 
-    // Run the generator, including known arguments
+  env.register(generatorPath, generatorName);
 
-    const runArgs = [generatorName]
-      .concat(argv['--'])
-      .concat(argv.help ? ['--help'] : [])
-      .join(' ');
+  // Run the generator, including known arguments
 
-    const [err] = await pack(env.run(runArgs));
+  const runArgs = [generatorName]
+    .concat(argv['--'])
+    .concat(argv.help ? ['--help'] : [])
+    .join(' ');
 
-    if (err) {
-      throw err;
-    }
+  const [err] = await pack(env.run(runArgs));
 
-    if (argv.announce) {
-      announce();
-    }
+  if (err) {
+    throw err;
+  }
+
+  if (argv.announce) {
+    announce();
   }
+};
+
+module.exports = command(
+  {
+    name: 'generate',
+    hasSubcommands: true,
+    options: OPTIONS
+  },
+  generate
 );
+
+// Expose this for the tests
+module.exports._testGenerate = generate;

package/src/config/serve.js

@@ -86,6 +86,16 @@ const findPort = async (port, max = port + 100, host = '0.0.0.0') => {
       // If the connection is refused, it's assumed nothing is listening and the port is available.
       if (e.code === 'ECONNREFUSED') {
         found();
+      } else if (e.code === 'ENOTFOUND' && e.syscall === 'getaddrinfo' && e.hostname?.includes(INTERNAL_SUFFIX)) {
+        console.error(
+          [
+            'Could not resolve hostname ' + e.hostname,
+            'You appear to be on the ABC network without a hostname attached to your computer.',
+            "Aunty can't continue. Consider reconnecting, or add your hostname to your hosts file.",
+            ''
+          ].join('\n')
+        );
+        process.exit(1);
       } else {
         // Not sure what to do with other errors. Log the code & keep seeking a free port.
         next(`error code ${e.code}`);

package/src/generators/project/templates/preact/src/index.tsx

@@ -26,7 +26,7 @@ if (module.hot) {
   module.hot.accept('./components/App', () => {
     try {
       renderApp();
-    } catch (err) {
+    } catch (err<% if (isTS) { %>: any<% } %>) {
       import('./components/ErrorBox').then(({ default: ErrorBox }) => {
         if (appMountEl) {
           render(<ErrorBox error={err} />, appMountEl);

package/src/utils/structures.js

@@ -31,5 +31,3 @@ module.exports.combine = (...sources) => sources.reduce(combine, {});
  * @see combine
  */
 module.exports.merge = (...sources) => sources.reduce(combineDeep, {});
-
-module.exports.setOfValues = source => new Set(Object.keys(source).map(key => source[key]));