More tests for errors

test/less-test.js

@@ -47,24 +47,33 @@ fs.readdirSync('test/less').forEach(function (file) {
 fs.readdirSync('test/less/errors').forEach(function (file) {
     if (! /\.less/.test(file)) { return }
 
-    toCSS('test/less/errors/' + file, function (err, less) {
+    toCSS('test/less/errors/' + file, function (err, compiledLess) {
         var name = path.basename(file, '.less');
 
         fs.readFile(path.join('test/less/errors', name) + '.txt', 'utf-8', function (e, expectedErr) {
-            sys.print("- error/" + name + ": ")
-            if (err.message === expectedErr) { sys.print(stylize('OK', 'green')) }
-            else if (!err) {
-                sys.print(stylize("No Error", 'red'));
-            } else {
-                sys.print(stylize("FAIL", 'yellow') + '\n');
+            sys.print("- error/" + name + ": ");
+            if (!err) {
+                if (compiledLess) {
+                    sys.print(stylize("No Error", 'red'));
+                } else {
+                    sys.print(stylize("No Error, No Output", 'red'));
+                }
                 
-                require('diff').diffLines(expectedErr, err.message).forEach(function(item) {
-                  if(item.added || item.removed) {
-                    sys.print(stylize(item.value, item.added ? 'green' : 'red'));
-                  } else {
-                    sys.print(item.value);
-                  }
-                })
+            } else {
+                var errMessage = less.formatError(err);
+                if (errMessage === expectedErr) {
+                    sys.print(stylize('OK', 'green'));                    
+                } else {
+                    sys.print(stylize("FAIL", 'yellow') + '\n');
+                
+                    require('diff').diffLines(expectedErr, errMessage).forEach(function(item) {
+                      if(item.added || item.removed) {
+                        sys.print(stylize(item.value, item.added ? 'green' : 'red'));
+                      } else {
+                        sys.print(item.value);
+                      }
+                    })
+                }
             }
             sys.puts("");
         });

test/less/errors/comment-in-selector.txt

@@ -1 +1,2 @@
-Syntax Error on line 1
+ParseError: Syntax Error on line 1
+1 #gaga /* Comment */ span { color: red }

test/less/errors/import-subfolder.txt

@@ -0,0 +1,3 @@
+NameError: .mixin-not-defined is undefined in test\less\errors\mixin-not-defined.less:1:102
+10 
+11 .mixin-not-defined();

test/less/errors/import-subfolder1.less

@@ -0,0 +1 @@
+@import "imports/import-subfolder1.less";

test/less/errors/import-subfolder1.txt

@@ -0,0 +1,3 @@
+NameError: .mixin-not-defined is undefined in test\less\errors\mixin-not-defined.less:1:102
+10 
+11 .mixin-not-defined();

test/less/errors/import-subfolder2.less

@@ -0,0 +1 @@
+@import "imports/import-subfolder2.less";

test/less/errors/import-subfolder2.txt

@@ -0,0 +1,3 @@
+NameError: .mixin-not-defined is undefined in test\less\errors\mixin-not-defined.less:1:102
+10 
+11 .mixin-not-defined();

test/less/errors/imports/import-subfolder1.less

@@ -0,0 +1 @@
+@import "subfolder/mixin-not-defined.less";

test/less/errors/imports/import-subfolder2.less

@@ -0,0 +1 @@
+@import "subfolder/parse-error-curly-bracket.less";

test/less/errors/imports/subfolder/mixin-not-defined.less

@@ -0,0 +1 @@
+@import "../../mixin-not-defined.less";

test/less/errors/imports/subfolder/parse-error-curly-bracket.less

@@ -0,0 +1 @@
+@import "../../parse-error-curly-bracket.less";

test/less/errors/mixin-not-defined.less

@@ -0,0 +1,11 @@
+
+.error-is-further-on() {
+}
+
+.pad-here-to-reproduce-error-in() {
+}
+
+.the-import-subfolder-test() {
+}
+
+.mixin-not-defined();

test/less/errors/mixin-not-defined.txt

@@ -0,0 +1,3 @@
+NameError: .mixin-not-defined is undefined
+10 
+11 .mixin-not-defined();

test/less/errors/parse-error-curly-bracket.less

@@ -0,0 +1 @@
+}}

test/less/errors/parse-error-curly-bracket.txt

@@ -0,0 +1,2 @@
+ParseError: Syntax Error on line 1
+1 }}