chore(getDefaultConfig): Move call and tests

Attempt to avoid invoking module until needed

Author: bmuenzenmeyer

core/index.js

@@ -22,7 +22,7 @@ const pa = require('./lib/pattern_assembler');
 const pe = require('./lib/pattern_exporter');
 const lh = require('./lib/lineage_hunter');
 
-const PatternLab = require('./lib/patternlab');
+const defaultConfig = require('../patternlab-config.json');
 
 let fs = require('fs-extra'); // eslint-disable-line
 let ui_builder = require('./lib/ui_builder'); // eslint-disable-line
@@ -39,7 +39,18 @@ updateNotifier({
   updateCheckInterval: 1000 * 60 * 60 * 24 // notify at most once a day
 }).notify();
 
+
+/**
+ * Returns the standardized default config
+ *
+ * @return {object} Returns the object representation of the patternlab-config.json
+ */
+const getDefaultConfig = function () {
+  return defaultConfig;
+};
+
 const patternlab_module = function (config) {
+  const PatternLab = require('./lib/patternlab');
   const patternlab = new PatternLab(config);
   const paths = patternlab.config.paths;
 
@@ -380,4 +391,6 @@ const patternlab_module = function (config) {
   };
 };
 
+patternlab_module.getDefaultConfig = getDefaultConfig;
+
 module.exports = patternlab_module;

core/lib/patternlab.js

@@ -8,7 +8,6 @@ const cleanHtml = require('js-beautify').html;
 const inherits = require('util').inherits;
 const pm = require('./plugin_manager');
 const packageInfo = require('../../package.json');
-const defaultConfig = require('../../patternlab-config.json');
 const dataLoader = require('./data_loader')();
 const logger = require('./log');
 const jsonCopy = require('./json_copy');

test/index_tests.js

@@ -0,0 +1,11 @@
+const tap = require('tap');
+const defaultConfig = require('../patternlab-config.json');
+
+const entry = require('../core/index');
+
+tap.test('getDefaultConfig - should return the default config object', function(test) {
+  const requestedConfig = entry.getDefaultConfig();
+  test.type(requestedConfig, 'object');
+  test.equals(requestedConfig, defaultConfig);
+  test.end();
+});

test/patternlab_tests.js

@@ -54,13 +54,6 @@ tap.test('buildPatternData - can load json, yaml, and yml files', function(test)
   test.end();
 });
 
-tap.test('getDefaultConfig - should return the default config object', function(test) {
-  const requestedConfig = plEngineModule.getDefaultConfig();
-  test.type(requestedConfig, 'object');
-  test.equals(requestedConfig, defaultConfig);
-  test.end();
-});
-
 // this test needs to be re-jiggered
 // tap.test('buildPatterns - should replace data link even when pattern parameter present', function(test) {
 //   //arrange