From 58b1f504d5904516145203e669912e4bbdefae22 Mon Sep 17 00:00:00 2001
From: Iuri Matias <iuri.matias@gmail.com>
Date: Sat, 18 Feb 2017 14:37:07 -0500
Subject: [PATCH] move grunt methods to utils module

---
 lib/config.js | 5 ++---
 lib/plugin.js | 3 +--
 lib/utils.js  | 9 +++++++++
 3 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/lib/config.js b/lib/config.js
index bc3236a2..f56c9903 100644
--- a/lib/config.js
+++ b/lib/config.js
@@ -1,5 +1,4 @@
 var fs = require('fs');
-var grunt = require('grunt');
 var merge = require('merge');
 var Plugins = require('./plugins.js');
 var utils = require('./utils.js');
@@ -149,7 +148,7 @@ Config.prototype.loadChainTrackerFile = function() {
 
 Config.prototype.loadFiles = function(files) {
   var self = this;
-  var originalFiles = grunt.file.expand({nonull: true}, files);
+  var originalFiles = utils.filesMatchinPattern(files);
   var readFiles = [];
 
   // get embark.js object first
@@ -187,7 +186,7 @@ Config.prototype.loadFiles = function(files) {
   }
 
   filesFromPlugins.filter(function(file) {
-    if (grunt.file.isMatch(files, file.intendedPath)) {
+    if (utils.fileMatchesPattern(files, file.intendedPath)) {
       readFiles.push(file);
     }
   });
diff --git a/lib/plugin.js b/lib/plugin.js
index a0e7bab6..57419311 100644
--- a/lib/plugin.js
+++ b/lib/plugin.js
@@ -1,5 +1,4 @@
 /*jshint esversion: 6, loopfunc: true */
-var grunt = require('grunt');
 var fs = require('fs');
 var utils = require('./utils.js');
 
@@ -146,7 +145,7 @@ Plugin.prototype.runFilePipeline = function() {
 Plugin.prototype.runPipeline = function(args) {
   // TODO: should iterate the pipelines
   var pipeline = this.pipeline[0];
-  var shouldRunPipeline = grunt.file.isMatch(pipeline.matcthingFiles, args.targetFile);
+  var shouldRunPipeline = utils.fileMatchesPattern(pipeline.matcthingFiles, args.targetFile);
   if (shouldRunPipeline) {
     return pipeline.cb.call(this, args);
   } else {
diff --git a/lib/utils.js b/lib/utils.js
index 2228fb39..e5143661 100644
--- a/lib/utils.js
+++ b/lib/utils.js
@@ -1,9 +1,18 @@
 var path = require('path');
+var grunt = require('grunt');
 
 function joinPath() {
   return path.join.apply(path.join, arguments);
 }
 
+function filesMatchinPattern(files) {
+  return grunt.file.expand({nonull: true}, files);
+}
+
+function fileMatchesPattern(patterns, intendedPath) {
+  return grunt.file.isMatch(patterns, intendedPath);
+}
+
 module.exports = {
   joinPath: joinPath
 };