Merge pull request #13 from billybonks/bug-fix/css-syntax

config.syntax should be '' if css
billybonks · Sep 26, 2016 · f5ef9cd · f5ef9cd
2 parents d0da425 + cdafd27
commit f5ef9cd
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/index.js b/index.js
@@ -61,7 +61,7 @@ function StyleLinter(inputNodes, options) {
   }
   this.linterConfig.files = null;
 
-  this.setSyntax(this.linterConfig.syntax);
+  this.setSyntax(this.linterConfig);
 
   Filter.call(this, inputNodes, options);
 }
@@ -70,7 +70,8 @@ function StyleLinter(inputNodes, options) {
  * Sets the, file extensions that the broccoli plugin must parse
  * @param {string} syntax sass|css|less|sugarss
  */
-StyleLinter.prototype.setSyntax = function(syntax) {
+StyleLinter.prototype.setSyntax = function(config) {
+  var syntax = config.syntax;
   var extensions = [];
   var targetExtension;
   if(!syntax)
@@ -81,6 +82,9 @@ StyleLinter.prototype.setSyntax = function(syntax) {
   } else {
     targetExtension = syntax;
   }
+  if(syntax === 'css'){
+    config.syntax = "";
+  }
   extensions.push(targetExtension);
   if(this.testPassingFiles || this.testFailingFiles)
     targetExtension = 'stylelint-test.js';

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "broccoli-stylelint",
-  "version": "0.8.2",
+  "version": "0.8.3",
   "description": "Broccoli stylelint plugin",
   "keywords": [
     "broccoli",

diff --git a/tests/test.js b/tests/test.js
@@ -108,7 +108,7 @@ describe('Broccoli StyleLint Plugin', function() {
       })
 
       it('should not log when log=false', function(){
-        return buildAndLint('tests/fixtures/test-plugin', {log: false,console: fakeConsole}).then(function(results){
+        return buildAndLint('tests/fixtures//has-errors', {log: false,console: fakeConsole}).then(function(results){
           expect(fakeConsole.log).to.not.have.been.called();
         });
       })