Fixes #3504 / organizes tests

Author: matthew-dean

packages/less/lib/less/tree/import.js

@@ -171,11 +171,13 @@ class Import extends Node {
                 throw this.error;
             }
             return newImport;
-        } else {
+        } else if (this.root) {
             ruleset = new Ruleset(null, utils.copyArray(this.root.rules));
             ruleset.evalImports(context);
 
             return this.features ? new Media(ruleset.rules, this.features.value) : ruleset.rules;
+        } else {
+            return [];
         }
     }
 }

packages/less/lib/less/tree/import.js.map

@@ -3,14 +3,14 @@ module.exports = {
     main: {
     // src is used to build list of less files to compile
         src: [
-            "test/less/*.less",
-            "!test/less/plugin-preeval.less", // uses ES6 syntax
+            "test/less/_main/*.less",
+            "!test/less/_main/plugin-preeval.less", // uses ES6 syntax
             // Don't test NPM import, obviously
-            "!test/less/plugin-module.less",
-            "!test/less/import-module.less",
-            "!test/less/javascript.less",
-            "!test/less/urls.less",
-            "!test/less/empty.less"
+            "!test/less/_main/plugin-module.less",
+            "!test/less/_main/import-module.less",
+            "!test/less/_main/javascript.less",
+            "!test/less/_main/urls.less",
+            "!test/less/_main/empty.less"
         ],
         options: {
             helpers: "test/browser/runner-main-options.js",

packages/less/src/less/tree/import.js

@@ -13,8 +13,8 @@ var testFiles = ['charsets', 'colors', 'comments', 'css-3', 'strings', 'media',
 
 for (var i = 0; i < testFiles.length; i++) {
     var file = testFiles[i],
-        lessPath  = '/test/less/' + file + '.less',
-        cssPath   = '/test/css/' + file + '.css',
+        lessPath  = '/test/less/_main/' + file + '.less',
+        cssPath   = '/test/css/_main/' + file + '.css',
         lessStyle = document.createElement('style'),
         cssLink   = document.createElement('link'),
         lessText  = '@import "' + lessPath + '";';

packages/less/test/README.md

@@ -0,0 +1,3 @@
+.a {
+  b: c;
+}

packages/less/test/browser/generator/runner.config.js

@@ -7,80 +7,81 @@ console.log('\n' + stylize('Less', 'underline') + '\n');
 
 lessTester.prepBomTest();
 var testMap = [
-    [{}, 'namespacing/'],
-    [{
-        // TODO: Change this to rewriteUrls: 'all' once the relativeUrls option is removed
-        relativeUrls: true,
-        silent: true,
-        javascriptEnabled: true
-    }],
-    [{
-        math: 'strict-legacy'
-    }, 'math/strict-legacy/'],
-    [{
-        math: 'parens'
-    }, 'math/strict/'],
-    [{
-        math: 'parens-division'
-    }, 'math/parens-division/'],
-    // Use legacy strictMath: true here to demonstrate it still works
-    [{strictMath: true, strictUnits: true, javascriptEnabled: true}, 'errors/',
-        lessTester.testErrors, null],
+    // [{
+    //     // TODO: Change this to rewriteUrls: 'all' once the relativeUrls option is removed
+    //     relativeUrls: true,
+    //     silent: true,
+    //     javascriptEnabled: true
+    // }, '_main/'],
+    // [{}, 'namespacing/'],
+    // [{
+    //     math: 'strict-legacy'
+    // }, 'math/strict-legacy/'],
+    // [{
+    //     math: 'parens'
+    // }, 'math/strict/'],
+    // [{
+    //     math: 'parens-division'
+    // }, 'math/parens-division/'],
+    // // Use legacy strictMath: true here to demonstrate it still works
+    // [{strictMath: true, strictUnits: true, javascriptEnabled: true}, 'errors/',
+    //     lessTester.testErrors, null],
 
-    [{math: 'strict', strictUnits: true, javascriptEnabled: false}, 'no-js-errors/',
-        lessTester.testErrors, null],
-    [{math: 'strict', dumpLineNumbers: 'comments'}, 'debug/', null,
-        function(name) { return name + '-comments'; }],
-    [{math: 'strict', dumpLineNumbers: 'mediaquery'}, 'debug/', null,
-        function(name) { return name + '-mediaquery'; }],
-    [{math: 'strict', dumpLineNumbers: 'all'}, 'debug/', null,
-        function(name) { return name + '-all'; }],
-    // TODO: Change this to rewriteUrls: false once the relativeUrls option is removed
-    [{math: 'strict', relativeUrls: false, rootpath: 'folder (1)/'}, 'static-urls/'],
-    [{math: 'strict', compress: true}, 'compression/'],
-    [{math: 0, strictUnits: true}, 'strict-units/'],
-    [{}, 'legacy/'],
-    [{math: 'strict', strictUnits: true, sourceMap: true, globalVars: true }, 'sourcemaps/',
-        lessTester.testSourcemap, null, null,
-        function(filename, type, baseFolder) {
-            if (type === 'vars') {
-                return path.join(baseFolder, filename) + '.json';
-            }
-            return path.join('test/sourcemaps', filename) + '.json';
-        }],
+    // [{math: 'strict', strictUnits: true, javascriptEnabled: false}, 'no-js-errors/',
+    //     lessTester.testErrors, null],
+    // [{math: 'strict', dumpLineNumbers: 'comments'}, 'debug/', null,
+    //     function(name) { return name + '-comments'; }],
+    // [{math: 'strict', dumpLineNumbers: 'mediaquery'}, 'debug/', null,
+    //     function(name) { return name + '-mediaquery'; }],
+    // [{math: 'strict', dumpLineNumbers: 'all'}, 'debug/', null,
+    //     function(name) { return name + '-all'; }],
+    // // TODO: Change this to rewriteUrls: false once the relativeUrls option is removed
+    // [{math: 'strict', relativeUrls: false, rootpath: 'folder (1)/'}, 'static-urls/'],
+    // [{math: 'strict', compress: true}, 'compression/'],
+    // [{math: 0, strictUnits: true}, 'strict-units/'],
+    // [{}, 'legacy/'],
+    // [{math: 'strict', strictUnits: true, sourceMap: true, globalVars: true }, 'sourcemaps/',
+    //     lessTester.testSourcemap, null, null,
+    //     function(filename, type, baseFolder) {
+    //         if (type === 'vars') {
+    //             return path.join(baseFolder, filename) + '.json';
+    //         }
+    //         return path.join('test/sourcemaps', filename) + '.json';
+    //     }],
 
-    [{math: 'strict', strictUnits: true, globalVars: true }, 'import/json/',
-        lessTester.testImports, null, true,
-        function(filename, type, baseFolder) {
-            return path.join(baseFolder, filename) + '.json';
-        }],
-    [{math: 'strict', strictUnits: true, sourceMap: {sourceMapFileInline: true}},
-        'sourcemaps-empty/', lessTester.testEmptySourcemap],
-    [{math: 'strict', strictUnits: true, sourceMap: {disableSourcemapAnnotation: true}},
-        'sourcemaps-disable-annotation/', lessTester.testSourcemapWithoutUrlAnnotation],
-    [{globalVars: true, banner: '/**\n  * Test\n  */\n'}, 'globalVars/',
-        null, null, null, function(name, type, baseFolder) { return path.join(baseFolder, name) + '.json'; }],
-    [{modifyVars: true}, 'modifyVars/',
-        null, null, null, function(name, type, baseFolder) { return path.join(baseFolder, name) + '.json'; }],
-    [{urlArgs: '424242'}, 'url-args/'],
-    [{rewriteUrls: 'all'}, 'rewrite-urls-all/'],
-    [{rewriteUrls: 'local'}, 'rewrite-urls-local/'],
-    [{rootpath: 'http://example.com/assets/css/', rewriteUrls: 'all'}, 'rootpath-rewrite-urls-all/'],
-    [{rootpath: 'http://example.com/assets/css/', rewriteUrls: 'local'}, 'rootpath-rewrite-urls-local/'],
-    [{paths: ['test/data/', 'test/less/import/']}, 'include-path/'],
-    [{paths: 'test/data/'}, 'include-path-string/'],
-    [{plugin: 'test/plugins/postprocess/'}, 'postProcessorPlugin/'],
-    [{plugin: 'test/plugins/preprocess/'}, 'preProcessorPlugin/'],
-    [{plugin: 'test/plugins/visitor/'}, 'visitorPlugin/'],
-    [{plugin: 'test/plugins/filemanager/'}, 'filemanagerPlugin/'],
-    [{}, 'no-strict-math/'],
-    [{}, '3rd-party/']
+    // [{math: 'strict', strictUnits: true, globalVars: true }, '_main/import/json/',
+    //     lessTester.testImports, null, true,
+    //     function(filename, type, baseFolder) {
+    //         return path.join(baseFolder, filename) + '.json';
+    //     }],
+    // [{math: 'strict', strictUnits: true, sourceMap: {sourceMapFileInline: true}},
+    //     'sourcemaps-empty/', lessTester.testEmptySourcemap],
+    // [{math: 'strict', strictUnits: true, sourceMap: {disableSourcemapAnnotation: true}},
+    //     'sourcemaps-disable-annotation/', lessTester.testSourcemapWithoutUrlAnnotation],
+    // [{globalVars: true, banner: '/**\n  * Test\n  */\n'}, 'globalVars/',
+    //     null, null, null, function(name, type, baseFolder) { return path.join(baseFolder, name) + '.json'; }],
+    // [{modifyVars: true}, 'modifyVars/',
+    //     null, null, null, function(name, type, baseFolder) { return path.join(baseFolder, name) + '.json'; }],
+    // [{urlArgs: '424242'}, 'url-args/'],
+    // [{rewriteUrls: 'all'}, 'rewrite-urls-all/'],
+    // [{rewriteUrls: 'local'}, 'rewrite-urls-local/'],
+    // [{rootpath: 'http://example.com/assets/css/', rewriteUrls: 'all'}, 'rootpath-rewrite-urls-all/'],
+    // [{rootpath: 'http://example.com/assets/css/', rewriteUrls: 'local'}, 'rootpath-rewrite-urls-local/'],
+    // [{paths: ['test/less/data/', 'test/less/_main/import/']}, 'include-path/'],
+    // [{paths: 'test/less/data/'}, 'include-path-string/'],
+    // [{plugin: 'test/plugins/postprocess/'}, 'postProcessorPlugin/'],
+    // [{plugin: 'test/plugins/preprocess/'}, 'preProcessorPlugin/'],
+    // [{plugin: 'test/plugins/visitor/'}, 'visitorPlugin/'],
+    // [{plugin: 'test/plugins/filemanager/'}, 'filemanagerPlugin/'],
+    // [{}, 'no-strict-math/'],
+    // [{}, '3rd-party/'],
+    [{ processImports: false }, 'process-imports/']
 ];
 testMap.forEach(function(args) {
     lessTester.runTestSet.apply(lessTester, args)
 });
-lessTester.testSyncronous({syncImport: true}, 'import');
-lessTester.testSyncronous({syncImport: true}, 'plugin');
+lessTester.testSyncronous({syncImport: true}, '_main/import');
+lessTester.testSyncronous({syncImport: true}, '_main/plugin');
 lessTester.testSyncronous({syncImport: true}, 'math/strict-legacy/css');
 lessTester.testNoOptions();
 lessTester.testJSImport();

packages/less/test/browser/runner-browser-options.js

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-undefined();

packages/less/test/css/calc.css packages/less/test/css/_main/calc.css

@@ -1,3 +1,3 @@
 SyntaxError: Function 'test-undefined' is undefined in {path}functions-1.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-undefined();

packages/less/test/css/charsets.css packages/less/test/css/_main/charsets.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-keyword();

packages/less/test/css/colors.css packages/less/test/css/_main/colors.css

@@ -1,3 +1,3 @@
 SyntaxError: Keyword node returned by a function is not valid here in {path}functions-10-keyword.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-keyword();

packages/less/test/css/comments.css packages/less/test/css/_main/comments.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-operation();

packages/less/test/css/comments2.css packages/less/test/css/_main/comments2.css

@@ -1,3 +1,3 @@
 SyntaxError: Operation node returned by a function is not valid here in {path}functions-11-operation.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-operation();

packages/less/test/css/css-3.css packages/less/test/css/_main/css-3.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-quoted();

packages/less/test/css/css-escapes.css packages/less/test/css/_main/css-escapes.css

@@ -1,3 +1,3 @@
 SyntaxError: Quoted node returned by a function is not valid here in {path}functions-12-quoted.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-quoted();

packages/less/test/css/css-grid.css packages/less/test/css/_main/css-grid.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-selector();

packages/less/test/css/css-guards.css packages/less/test/css/_main/css-guards.css

@@ -1,3 +1,3 @@
 SyntaxError: Selector node returned by a function is not valid here in {path}functions-13-selector.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-selector();

packages/less/test/css/detached-rulesets.css packages/less/test/css/_main/detached-rulesets.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-url();

packages/less/test/css/directives-bubling.css packages/less/test/css/_main/directives-bubling.css

@@ -1,3 +1,3 @@
 SyntaxError: Url node returned by a function is not valid here in {path}functions-14-url.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-url();

packages/less/test/css/empty.css packages/less/test/css/_main/empty.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-value();

packages/less/test/css/extend-chaining.css packages/less/test/css/_main/extend-chaining.css

@@ -1,3 +1,3 @@
 SyntaxError: Value node returned by a function is not valid here in {path}functions-15-value.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-value();

packages/less/test/css/extend-clearfix.css packages/less/test/css/_main/extend-clearfix.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-assignment();

packages/less/test/css/extend-exact.css packages/less/test/css/_main/extend-exact.css

@@ -1,3 +1,3 @@
 SyntaxError: Assignment node returned by a function is not valid here in {path}functions-3-assignment.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-assignment();

packages/less/test/css/extend-media.css packages/less/test/css/_main/extend-media.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-call();

packages/less/test/css/extend-nest.css packages/less/test/css/_main/extend-nest.css

@@ -1,3 +1,3 @@
 SyntaxError: Function 'foo' is undefined in {path}functions-4-call.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-call();

packages/less/test/css/extend-selector.css packages/less/test/css/_main/extend-selector.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-color();

packages/less/test/css/extend.css packages/less/test/css/_main/extend.css

@@ -1,3 +1,3 @@
 SyntaxError: Color node returned by a function is not valid here in {path}functions-5-color.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-color();

packages/less/test/css/extract-and-length.css packages/less/test/css/_main/extract-and-length.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-condition();

packages/less/test/css/functions-each.css packages/less/test/css/_main/functions-each.css

@@ -1,3 +1,3 @@
 SyntaxError: Condition node returned by a function is not valid here in {path}functions-6-condition.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-condition();

packages/less/test/css/functions.css packages/less/test/css/_main/functions.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-dimension();

packages/less/test/css/ie-filters.css packages/less/test/css/_main/ie-filters.css

@@ -1,3 +1,3 @@
 SyntaxError: Dimension node returned by a function is not valid here in {path}functions-7-dimension.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-dimension();

packages/less/test/css/import-inline.css packages/less/test/css/_main/import-inline.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-element();

packages/less/test/css/import-interpolation.css packages/less/test/css/_main/import-interpolation.css

@@ -1,3 +1,3 @@
 SyntaxError: Element node returned by a function is not valid here in {path}functions-8-element.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-element();

packages/less/test/css/import-module.css packages/less/test/css/_main/import-module.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes";
+@plugin "../_main/plugin/plugin-tree-nodes";
 test-expression();

packages/less/test/css/import-once.css packages/less/test/css/_main/import-once.css

@@ -1,3 +1,3 @@
 SyntaxError: Expression node returned by a function is not valid here in {path}functions-9-expression.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes";
+1 @plugin "../_main/plugin/plugin-tree-nodes";
 2 test-expression();

packages/less/test/css/import-reference-issues.css packages/less/test/css/_main/import-reference-issues.css

@@ -1,2 +1,2 @@
-@plugin "../plugin/plugin-tree-nodes.js";
+@plugin "../_main/plugin/plugin-tree-nodes.js";
 test-undefined();

packages/less/test/css/import-reference.css packages/less/test/css/_main/import-reference.css

@@ -1,3 +1,3 @@
 SyntaxError: Function 'test-undefined' is undefined in {path}root-func-undefined-2.less on line 2, column 1:
-1 @plugin "../plugin/plugin-tree-nodes.js";
+1 @plugin "../_main/plugin/plugin-tree-nodes.js";
 2 test-undefined();

packages/less/test/css/import.css packages/less/test/css/_main/import.css

@@ -0,0 +1,3 @@
+@import url('https://fonts.googleapis.com/css?family=Open+Sans:400,700');
+
+.a { b: c; }

packages/less/test/css/javascript.css packages/less/test/css/_main/javascript.css

@@ -30,4 +30,4 @@
     url: url(@a);
 }
 
-@import "../import/import-and-relative-paths-test";
+@import "../_main/import/import-and-relative-paths-test";