Move check for already registred plugin to utils.

This makes it much easier to test (as it doesn't require any state from
the addon instance).

Author: Robert Jackson

lib/ember-addon-main.js

@@ -135,7 +135,7 @@ dBabelVersion: ${hasValidBabelVersion};`
 
     // add the babel-plugin-htmlbars-inline-precompile to the list of plugins
     // used by `ember-cli-babel` addon
-    if (!this._isInlinePrecompileBabelPluginRegistered(babelPlugins)) {
+    if (!utils.isInlinePrecompileBabelPluginRegistered(babelPlugins)) {
       let pluginInfo = this.astPlugins();
       let templateCompilerPath = this.templateCompilerPath();
 
@@ -194,41 +194,6 @@ dBabelVersion: ${hasValidBabelVersion};`
     }
   },
 
-  /**
-   * This function checks if 'babel-plugin-htmlbars-inline-precompile' is already present in babelPlugins.
-   * The plugin object will be different for non parallel API and parallel API.
-   * For parallel api, check the `baseDir` of a plugin to see if it has current dirname
-   * For non parallel api, check the 'modules' to see if it contains the babel plugin
-   * @param {*} plugins
-   */
-  _isInlinePrecompileBabelPluginRegistered(plugins) {
-    return plugins.some(plugin => {
-      if (Array.isArray(plugin)) {
-        let [pluginPathOrInstance, options] = plugin;
-
-        return (
-          pluginPathOrInstance === require.resolve('babel-plugin-htmlbars-inline-precompile') &&
-          typeof options.modules === 'object' &&
-          options.modules['ember-cli-htmlbars'] === 'hbs'
-        );
-      } else if (
-        plugin !== null &&
-        typeof plugin === 'object' &&
-        plugin._parallelBabel !== undefined
-      ) {
-        return (
-          plugin._parallelBabel.requireFile ===
-            path.resolve(__dirname, 'lib/require-from-worker') &&
-          typeof plugin._parallelBabel.params === 'object' &&
-          typeof plugin._parallelBabel.params.modules === 'object' &&
-          plugin._parallelBabel.params.modules['ember-cli-htmlbars'] === 'hbs'
-        );
-      } else {
-        return false;
-      }
-    });
-  },
-
   projectConfig() {
     return this.project.config(process.env.EMBER_ENV);
   },

lib/utils.js

@@ -7,6 +7,33 @@ const debugGenerator = require('heimdalljs-logger');
 const logger = debugGenerator('ember-cli-htmlbars:utils');
 const addDependencyTracker = require('./addDependencyTracker');
 
+function isInlinePrecompileBabelPluginRegistered(plugins) {
+  return plugins.some(plugin => {
+    if (Array.isArray(plugin)) {
+      let [pluginPathOrInstance, options] = plugin;
+
+      return (
+        pluginPathOrInstance === require.resolve('babel-plugin-htmlbars-inline-precompile') &&
+        typeof options.modules === 'object' &&
+        options.modules['ember-cli-htmlbars'] === 'hbs'
+      );
+    } else if (
+      plugin !== null &&
+      typeof plugin === 'object' &&
+      plugin._parallelBabel !== undefined
+    ) {
+      return (
+        plugin._parallelBabel.requireFile === path.resolve(__dirname, 'lib/require-from-worker') &&
+        typeof plugin._parallelBabel.params === 'object' &&
+        typeof plugin._parallelBabel.params.modules === 'object' &&
+        plugin._parallelBabel.params.modules['ember-cli-htmlbars'] === 'hbs'
+      );
+    } else {
+      return false;
+    }
+  });
+}
+
 function buildOptions(projectConfig, templateCompilerPath, pluginInfo) {
   let EmberENV = projectConfig.EmberENV || {};
 
@@ -221,4 +248,5 @@ module.exports = {
   setup,
   makeCacheKey,
   setupPlugins,
+  isInlinePrecompileBabelPluginRegistered,
 };