Commit 9d83b9ab by Marcin Cieslak

includedFiles returns an array in no particular order

Fixes: https://github.com/sass/node-sass/issues/1123
parent 2571f9e8
...@@ -7,6 +7,7 @@ var assert = require('assert'), ...@@ -7,6 +7,7 @@ var assert = require('assert'),
resolveFixture = path.resolve.bind(null, __dirname, 'fixtures'); resolveFixture = path.resolve.bind(null, __dirname, 'fixtures');
describe('api', function() { describe('api', function() {
describe('.render(options, callback)', function() { describe('.render(options, callback)', function() {
it('should compile sass to css with file', function(done) { it('should compile sass to css with file', function(done) {
var expected = read(fixture('simple/expected.css'), 'utf8').trim(); var expected = read(fixture('simple/expected.css'), 'utf8').trim();
...@@ -980,7 +981,7 @@ describe('api', function() { ...@@ -980,7 +981,7 @@ describe('api', function() {
file: fixture('include-files/index.scss') file: fixture('include-files/index.scss')
}, function(error, result) { }, function(error, result) {
assert(!error); assert(!error);
assert.deepEqual(result.stats.includedFiles, expected); assert.deepEqual(result.stats.includedFiles.sort(), expected.sort());
done(); done();
}); });
}); });
...@@ -1441,11 +1442,12 @@ describe('api', function() { ...@@ -1441,11 +1442,12 @@ describe('api', function() {
fixture('include-files/bar.scss').replace(/\\/g, '/'), fixture('include-files/bar.scss').replace(/\\/g, '/'),
fixture('include-files/foo.scss').replace(/\\/g, '/'), fixture('include-files/foo.scss').replace(/\\/g, '/'),
fixture('include-files/index.scss').replace(/\\/g, '/') fixture('include-files/index.scss').replace(/\\/g, '/')
]; ].sort();
var actual = result.stats.includedFiles.sort();
assert.equal(result.stats.includedFiles[0], expected[0]); assert.equal(actual[0], expected[0]);
assert.equal(result.stats.includedFiles[1], expected[1]); assert.equal(actual[1], expected[1]);
assert.equal(result.stats.includedFiles[2], expected[2]); assert.equal(actual[2], expected[2]);
done(); done();
}); });
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment