Merge pull request #436 from dlmanning/fix/sourcemap-tests

Fix failing sourcemaps tests
dlmanning · Feb 4, 2016 · 67874cb · 67874cb
2 parents 1888f35 + 3150325
commit 67874cb
Showing 1 changed file with 21 additions and 9 deletions.
diff --git a/test/main.js b/test/main.js
@@ -217,9 +217,9 @@ describe('gulp-sass -- async compile', function() {
 
     // Expected sources are relative to file.base
     var expectedSources = [
+      'inheritance.scss',
       'includes/_cats.scss',
       'includes/_dogs.sass',
-      'inheritance.scss'
     ];
 
     var stream;
@@ -406,9 +406,9 @@ describe('gulp-sass -- sync compile', function() {
 
     // Expected sources are relative to file.base
     var expectedSources = [
+      'inheritance.scss',
       'includes/_cats.scss',
       'includes/_dogs.sass',
-      'inheritance.scss'
     ];
 
     var stream;
@@ -432,25 +432,31 @@ describe('gulp-sass -- sync compile', function() {
   });
 
   it('should work with gulp-sourcemaps and autoprefixer', function(done) {
-    var expectedSources = [
+    var expectedSourcesBefore = [
+      'inheritance.scss',
+      'includes/_cats.scss',
+      'includes/_dogs.sass',
+    ];
+
+    var expectedSourcesAfter = [
       'includes/_cats.scss',
       'includes/_dogs.sass',
-      'inheritance.scss'
+      'inheritance.scss',
     ];
 
     gulp.src(path.join(__dirname, '/scss/inheritance.scss'))
       .pipe(sourcemaps.init())
       .pipe(sass.sync())
       .pipe(tap(function(file) {
         should.exist(file.sourceMap);
-        file.sourceMap.sources.should.eql(expectedSources);
+        file.sourceMap.sources.should.eql(expectedSourcesBefore);
       }))
       .pipe(postcss([autoprefixer()]))
       .pipe(sourcemaps.write())
       .pipe(gulp.dest(path.join(__dirname, '/results/')))
       .pipe(tap(function(file) {
         should.exist(file.sourceMap);
-        file.sourceMap.sources.should.eql(expectedSources);
+        file.sourceMap.sources.should.eql(expectedSourcesAfter);
       }))
       .on('end', done);
   });
@@ -476,7 +482,13 @@ describe('gulp-sass -- sync compile', function() {
   });
 
   it('should work with gulp-sourcemaps and autoprefixer with different file.base', function(done) {
-    var expectedSources = [
+    var expectedSourcesBefore = [
+      'scss/inheritance.scss',
+      'scss/includes/_cats.scss',
+      'scss/includes/_dogs.sass'
+    ];
+
+    var expectedSourcesAfter = [
       'scss/includes/_cats.scss',
       'scss/includes/_dogs.sass',
       'scss/inheritance.scss'
@@ -487,12 +499,12 @@ describe('gulp-sass -- sync compile', function() {
       .pipe(sass.sync())
       .pipe(tap(function(file) {
         should.exist(file.sourceMap);
-        file.sourceMap.sources.should.eql(expectedSources);
+        file.sourceMap.sources.should.eql(expectedSourcesBefore);
       }))
       .pipe(postcss([autoprefixer()]))
       .pipe(tap(function(file) {
         should.exist(file.sourceMap);
-        file.sourceMap.sources.should.eql(expectedSources);
+        file.sourceMap.sources.should.eql(expectedSourcesAfter);
       }))
       .on('end', done);
   });