Merge branch 'nielsonm-issue/486-jsdoc-pattern-engine' into dev

Author: bmuenzenmeyer

core/lib/pattern_engines.js

@@ -15,8 +15,12 @@ const enginesDirectories = [{
   path: path.join(process.cwd(), 'node_modules')
 }];
 
-// given a path: return the engine name if the path points to a valid engine
-// module directory, or false if it doesn't
+/**
+ * Given a path: return the engine name if the path points to a valid engine
+ * module directory, or false if it doesn't.
+ * @param filePath
+ * @returns Engine name if exists or FALSE
+ */
 function isEngineModule(filePath) {
   const baseName = path.basename(filePath);
   const engineMatch = baseName.match(engineMatcher);
@@ -118,6 +122,11 @@ function findEngineModulesInDirectory(dir) {
 
 const PatternEngines = Object.create({
 
+  /**
+   * Load all pattern engines.
+   * @param patternLabConfig
+   * @memberof PatternEngines
+   */
   loadAllEngines: function (patternLabConfig) {
     var self = this;
 
@@ -165,6 +174,12 @@ const PatternEngines = Object.create({
     logger.debug(`Done loading engines`);
   },
 
+  /**
+   * Get engine name for pattern.
+   * @memberof PatternEngines
+   * @param pattern
+   * @returns engine name matching pattern
+   */
   getEngineNameForPattern: function (pattern) {
     // avoid circular dependency by putting this in here. TODO: is this slow?
     const of = require('./object_factory');
@@ -192,6 +207,12 @@ const PatternEngines = Object.create({
     return 'mustache';
   },
 
+  /**
+   * Get engine for pattern.
+   * @memberof PatternEngines
+   * @param pattern
+   * @returns name of engine for pattern
+   */
   getEngineForPattern: function (pattern) {
     if (pattern.isPseudoPattern) {
       return this.getEngineForPattern(pattern.basePattern);
@@ -201,7 +222,11 @@ const PatternEngines = Object.create({
     }
   },
 
-  // combine all found engines into a single array of supported extensions
+  /**
+   * Combine all found engines into a single array of supported extensions.
+   * @memberof PatternEngines
+   * @returns Array all supported file extensions
+   */
   getSupportedFileExtensions: function () {
     const engineNames = Object.keys(PatternEngines);
     const allEnginesExtensions = engineNames.map((engineName) => {
@@ -210,22 +235,38 @@ const PatternEngines = Object.create({
     return [].concat.apply([], allEnginesExtensions);
   },
 
+  /**
+   * Check if fileExtension is supported.
+   * @memberof PatternEngines
+   * @param fileExtension
+   * @returns Boolean
+   */
   isFileExtensionSupported: function (fileExtension) {
     const supportedExtensions = PatternEngines.getSupportedFileExtensions();
     return (supportedExtensions.lastIndexOf(fileExtension) !== -1);
   },
 
-  // given a filename, return a boolean: whether or not the filename indicates
-  // that the file is pseudopattern JSON
+  /**
+   * Given a filename, return a boolean: whether or not the filename indicates
+   * that the file is pseudopattern JSON
+   * @param filename
+   * @return boolean
+   */
   isPseudoPatternJSON: function (filename) {
     const extension = path.extname(filename);
     return (extension === '.json' && filename.indexOf('~') > -1);
   },
 
-  // takes a filename string, not a full path; a basename (plus extension)
-  // ignore _underscored patterns, dotfiles, and anything not recognized by a
-  // loaded pattern engine. Pseudo-pattern .json files ARE considered to be
-  // pattern files!
+  /**
+   * Takes a filename string, not a full path; a basename (plus extension)
+   * ignore _underscored patterns, dotfiles, and anything not recognized by a
+   * loaded pattern engine. Pseudo-pattern .json files ARE considered to be
+   * pattern files!
+   *
+   * @memberof PatternEngines
+   * @param filename
+   * @returns boolean
+   */
   isPatternFile: function (filename) {
     // skip hidden patterns/files without a second thought
     const extension = path.extname(filename);