sassy-test - npm Package Compare versions

Comparing version 3.0.1 to 3.0.2

lib/sassy-test.js

@@ -274,11 +274,7 @@ 'use strict';
     value: function assertResult(result) {
-      // We always return a Sass error, so don't run our assertions if there is
-      // a Sass error.
-      if (!result.sassError) {
-        // A missing output.css file is a hard fail.
-        assert.ifError(result.expectedOutputFileError);
+      // A missing output.css file is a hard fail.
+      assert.ifError(result.expectedOutputFileError);
 
-        // Compare the Sass compilation to the expected output file.
-        assert.strictEqual(result.css, result.expectedOutput);
-      }
+      // Compare the Sass compilation to the expected output file.
+      assert.strictEqual(result.css, result.expectedOutput);
     }
@@ -366,3 +362,2 @@
         result: null,
-        sassError: null,
         expectedOutput: null,
@@ -375,3 +370,2 @@ expectedOutputFileError: null
         var result = test.result || {};
-        result.sassError = test.sassError;
         result.expectedOutput = test.expectedOutput;
@@ -388,2 +382,3 @@ result.expectedOutputFileError = test.expectedOutputFileError;
         test.completedReadFile = false;
+        test.sassError = null;
 
@@ -397,3 +392,7 @@ var compareResults = function compareResults() {
           // Give the callback access to the results.
-          callback(test.sassError, handleResult(test));
+          if (test.sassError) {
+            callback(test.sassError, null);
+          } else {
+            callback(null, handleResult(test));
+          }
         };
@@ -428,6 +427,3 @@
         // Do a sass.render() on the input.scss file.
-        this.render(options).catch(function (error) {
-          test.sassError = error;
-          return Promise.resolve(null);
-        }).then(function (result) {
+        this.render(options).then(function (result) {
           test.result = result;
@@ -434,0 +430,0 @@ return Promise.resolve();

package.json

 {
   "name": "sassy-test",
-  "version": "3.0.1",
+  "version": "3.0.2",
   "homepage": "https://github.com/JohnAlbin/sassy-test",
@@ -5,0 +5,0 @@ "author": "John Albin Wilkins <[e-mail]> (http://john.albin.net/)",

README.md

@@ -139,3 +139,3 @@ [![Build Status](https://secure.travis-ci.org/JohnAlbin/sassy-test.png?branch=master)](http://travis-ci.org/JohnAlbin/sassy-test) [![Coverage Status](https://coveralls.io/repos/JohnAlbin/sassy-test/badge.svg?branch=master&service=github)](https://coveralls.io/github/JohnAlbin/sassy-test?branch=master)
         expect(error).to.exist;
-        expect(error.message).to.equal('Some helpful error message from your module.');
+        expect(error.message).to.include('Some helpful error message from your module.');
       });
@@ -142,0 +142,0 @@ });

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

93870

increased by7.58%

Worsened metrics

Lines of code

399

decreased by-0.99%

No dependency changes