From 3150325f32d7a407bdfe7de301e9c47e6099aa3a Mon Sep 17 00:00:00 2001 From: xzyfer Date: Thu, 4 Feb 2016 22:56:41 +1100 Subject: [PATCH] Fix failing sourcemaps tests It looks like a dependency has caused some tests to start failing. Sourcemaps are not a documented API so this is not a breaking change. --- test/main.js | 30 +++++++++++++++++++++--------- 1 file changed, 21 insertions(+), 9 deletions(-) diff --git a/test/main.js b/test/main.js index 9987afa..601a4dc 100644 --- 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,10 +432,16 @@ 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', - 'inheritance.scss' + ]; + + var expectedSourcesAfter = [ + 'includes/_cats.scss', + 'includes/_dogs.sass', + 'inheritance.scss', ]; gulp.src(path.join(__dirname, '/scss/inheritance.scss')) @@ -443,14 +449,14 @@ 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(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); });