Merge pull request #681 from DKvistgaard/master
Declaring logError as function instead of arrow function.master
commit
daca90dd95
2
index.js
2
index.js
|
@ -150,7 +150,7 @@ gulpSass.sync = options => gulpSass(options, true);
|
||||||
//////////////////////////////
|
//////////////////////////////
|
||||||
// Log errors nicely
|
// Log errors nicely
|
||||||
//////////////////////////////
|
//////////////////////////////
|
||||||
gulpSass.logError = (error) => {
|
gulpSass.logError = function logError(error) {
|
||||||
const message = new PluginError('sass', error.messageFormatted).toString();
|
const message = new PluginError('sass', error.messageFormatted).toString();
|
||||||
process.stderr.write(`${message}\n`);
|
process.stderr.write(`${message}\n`);
|
||||||
this.emit('end');
|
this.emit('end');
|
||||||
|
|
18
test/main.js
18
test/main.js
|
@ -140,6 +140,15 @@ describe('gulp-sass -- async compile', () => {
|
||||||
stream.write(sassFile);
|
stream.write(sassFile);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('should emit logError on sass error', (done) => {
|
||||||
|
const errorFile = createVinyl('error.scss');
|
||||||
|
const stream = sass();
|
||||||
|
|
||||||
|
stream.on('error', sass.logError);
|
||||||
|
stream.on('end', done);
|
||||||
|
stream.write(errorFile);
|
||||||
|
});
|
||||||
|
|
||||||
it('should handle sass errors', (done) => {
|
it('should handle sass errors', (done) => {
|
||||||
const errorFile = createVinyl('error.scss');
|
const errorFile = createVinyl('error.scss');
|
||||||
const stream = sass();
|
const stream = sass();
|
||||||
|
@ -397,6 +406,15 @@ describe('gulp-sass -- sync compile', () => {
|
||||||
stream.write(errorFile);
|
stream.write(errorFile);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('should emit logError on sass error', (done) => {
|
||||||
|
const errorFile = createVinyl('error.scss');
|
||||||
|
const stream = sass.sync();
|
||||||
|
|
||||||
|
stream.on('error', sass.logError);
|
||||||
|
stream.on('end', done);
|
||||||
|
stream.write(errorFile);
|
||||||
|
});
|
||||||
|
|
||||||
it('should work with gulp-sourcemaps', (done) => {
|
it('should work with gulp-sourcemaps', (done) => {
|
||||||
const sassFile = createVinyl('inheritance.scss');
|
const sassFile = createVinyl('inheritance.scss');
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue