less 3.11.2 → 3.11.3

package/lib/less/import-manager.js

@@ -26,6 +26,7 @@ export default environment => {
             this.context = context;
             // Deprecated? Unused outside of here, could be useful.
             this.queue = [];        // Files which haven't been imported yet
+            this.files = [];        // List of files imported
         }
 
         /**
@@ -51,6 +52,10 @@ export default environment => {
                     logger.info(`The file ${fullPath} was skipped because it was not found and the import was marked optional.`);
                 }
                 else {
+                    const files = importManager.files
+                    if (files.indexOf(fullPath) === -1) {
+                        files.push(fullPath)
+                    }
                     if (e && !importManager.error) { importManager.error = e; }
                     callback(e, root, importedEqualsRoot, fullPath);
                 }

package/lib/less/index.js

@@ -42,7 +42,7 @@ export default (environment, fileManagers) => {
      * It's not clear what should / must be public and why.
      */
     const initial = {
-        version: [3, 11, 2],
+        version: [3, 11, 3],
         data,
         tree,
         Environment,

package/lib/less/parse-tree.js

@@ -52,12 +52,8 @@ export default SourceMapBuilder => {
                 result.map = sourceMapBuilder.getExternalSourceMap();
             }
 
-            result.imports = [];
-            for (const file in this.imports.files) {
-                if (this.imports.files.hasOwnProperty(file) && file !== this.imports.rootFilename) {
-                    result.imports.push(file);
-                }
-            }
+            const rootFilename = this.imports.rootFilename
+            result.imports = this.imports.files.filter(file => file !== rootFilename);
             return result;
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "less",
-  "version": "3.11.2",
+  "version": "3.11.3",
   "description": "Leaner CSS",
   "homepage": "http://lesscss.org",
   "author": {

package/test/index.js

@@ -48,6 +48,12 @@ var testMap = [
             }
             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],
     [{globalVars: true, banner: '/**\n  * Test\n  */\n'}, 'globalVars/',

package/test/less/import/json/index.json

@@ -0,0 +1,11 @@
+[
+  "{path}/import/import-test-a.less",
+  "{path}/import/import-test-b.less",
+  "{path}/import/deeper/url-import.less",
+  "{path}/import/urls.less",
+  "{path}/import/import-test-c.less",
+  "{path}/import/deeper/deeper-2/url-import.less",
+  "{path}/import/deeper/deeper-2/url-import-2.less",
+  "{path}/import/import-test-f.less",
+  "{path}/import/import-test-e.less"
+]

package/test/less/import/json/index.less

@@ -0,0 +1 @@
+@import "../import-test-a";

package/test/less-test.js

@@ -140,6 +140,42 @@ module.exports = function() {
         }
     }
 
+    function testImports(name, err, compiledLess, doReplacements, sourcemap, baseFolder, imports) {
+        if (err) {
+            fail('ERROR: ' + (err && err.message));
+            return;
+        }
+
+        function stringify(str) {
+            return JSON.stringify(imports, null, '  ')
+        }
+
+        /** Imports are not sorted */
+        const importsString = stringify(imports.sort())
+
+        fs.readFile(path.join('test/less/', name) + '.json', 'utf8', function (e, expectedImports) {
+            if (e) {
+                fail('ERROR: ' + (e && e.message));
+                return;
+            }
+            process.stdout.write('- ' + path.join(baseFolder, name) + ': ');
+            expectedImports = stringify(JSON.parse(expectedImports).sort());
+            expectedImports = globalReplacements(expectedImports, baseFolder);
+
+            if (expectedImports === importsString) {
+                ok('OK');
+            } else if (err) {
+                fail('ERROR: ' + (err && err.message));
+                if (isVerbose) {
+                    process.stdout.write('\n');
+                    process.stdout.write(err.stack + '\n');
+                }
+            } else {
+                difference('FAIL', expectedImports, importsString);
+            }
+        });
+    }
+
     function testErrors(name, err, compiledLess, doReplacements, sourcemap, baseFolder) {
         fs.readFile(path.join(baseFolder, name) + '.txt', 'utf8', function (e, expectedErr) {
             process.stdout.write('- ' + path.join(baseFolder, name) + ': ');
@@ -303,8 +339,13 @@ module.exports = function() {
                     }
                     doubleCallCheck = (new Error()).stack;
 
+                    /**
+                     * @todo - refactor so the result object is sent to the verify function
+                     */
                     if (verifyFunction) {
-                        var verificationResult = verifyFunction(name, err, result && result.css, doReplacements, result && result.map, baseFolder);
+                        var verificationResult = verifyFunction(
+                            name, err, result && result.css, doReplacements, result && result.map, baseFolder, result && result.imports
+                        );
                         release();
                         return verificationResult;
                     }
@@ -362,7 +403,7 @@ module.exports = function() {
         process.stdout.write(stylize(msg, 'yellow') + '\n');
         failedTests++;
 
-        diff(left, right);
+        diff(left || '', right || '');
         endTest();
     }
 
@@ -452,6 +493,7 @@ module.exports = function() {
         testSyncronous: testSyncronous,
         testErrors: testErrors,
         testSourcemap: testSourcemap,
+        testImports: testImports,
         testEmptySourcemap: testEmptySourcemap,
         testNoOptions: testNoOptions,
         prepBomTest: prepBomTest,