Return a promise from postcss-import transform function

Author: simonsmith

lib/index.js

@@ -132,5 +132,5 @@ function lintImportedFiles(options, css, filename) {
   .use(bemLinter(options['postcss-bem-linter']))
   .use(reporter(options['postcss-reporter']));
 
-  return processor.process(css, {from: filename}).css;
+  return processor.process(css, {from: filename});
 }

test/error.config.json

@@ -1,6 +1,5 @@
 {
   "postcss-reporter": {
-    "throwError": true,
-    "plugins": ["postcss-bem-linter"]
+    "throwError": true
   }
 }

test/test.js

@@ -172,7 +172,8 @@ describe('suitcss', function() {
       var lintImportedFilesStub, beforeLintStub, revert;
 
       beforeEach(function() {
-        lintImportedFilesStub = sinon.stub().returns('/*linting done*/');
+        var postcssPromise = sinon.stub().resolves('/*linting done*/')();
+        lintImportedFilesStub = sinon.stub().returns(postcssPromise);
         beforeLintStub = sinon.stub().returns('/*before lint*/');
         revert = suitcss.__set__('lintImportedFiles', lintImportedFilesStub);
       });
@@ -185,35 +186,38 @@ describe('suitcss', function() {
         suitcss(read('fixtures/component'), {
           root: 'test/fixtures',
           beforeLint: beforeLintStub
-        }).catch(done);
-
-        expect(lintImportedFilesStub).to.be.calledOnce;
-        expect(beforeLintStub).to.be.calledOnce;
-        expect(beforeLintStub).to.have.been.calledBefore(lintImportedFilesStub);
-
-        done();
+        })
+        .then(function() {
+          expect(beforeLintStub).to.be.calledOnce;
+          expect(lintImportedFilesStub).to.be.calledOnce;
+          expect(beforeLintStub).to.have.been.calledBefore(lintImportedFilesStub);
+          done();
+        })
+        .catch(done);
       });
 
       it('should pass the result of `beforeLint` to `lintImportedFiles`', function(done) {
         suitcss(read('fixtures/component'), {
           root: 'test/fixtures',
           beforeLint: beforeLintStub
-        }).catch(done);
-
-        expect(lintImportedFilesStub.args[0][1]).to.equal('/*before lint*/');
-
-        done();
+        })
+        .then(function() {
+          expect(lintImportedFilesStub.args[0][1]).to.equal('/*before lint*/');
+          done();
+        })
+        .catch(done);
       });
 
       it('should pass the options object to the beforeLint function as the third parameter', function(done) {
         suitcss(read('fixtures/component'), {
           root: 'test/fixtures',
           beforeLint: beforeLintStub
-        }).catch(done);
-
-        expect(beforeLintStub.args[0][2]).to.contain({root: 'test/fixtures'});
-
-        done();
+        })
+        .then(function() {
+          expect(beforeLintStub.args[0][2]).to.contain({root: 'test/fixtures'});
+          done();
+        })
+        .catch(done);
       });
     });
   });