Skip to content

Commit 52fba69

Browse files
committed
Merge pull request #420 from jlgeering/error-msg-unformatted
preserve the original error message for notification(s) support
2 parents 847a2d3 + 3075a84 commit 52fba69

File tree

2 files changed

+15
-0
lines changed

2 files changed

+15
-0
lines changed

index.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,7 @@ var gulpSass = function gulpSass(options, sync) {
125125
message += error.formatted;
126126

127127
error.messageFormatted = message;
128+
error.messageOriginal = error.message;
128129
error.message = gutil.colors.stripColor(message);
129130

130131
error.relativePath = relativePath;

test/main.js

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -155,6 +155,20 @@ describe('gulp-sass -- async compile', function() {
155155
stream.write(errorFile);
156156
});
157157

158+
it('should preserve the original sass error message', function(done) {
159+
var errorFile = createVinyl('error.scss');
160+
var stream = sass();
161+
162+
stream.on('error', function(err) {
163+
// Error must include original error message
164+
err.messageOriginal.indexOf('property "font" must be followed by a \':\'').should.not.equal(-1);
165+
// Error must not format or change the original error message
166+
err.messageOriginal.indexOf('on line 2').should.equal(-1);
167+
done();
168+
});
169+
stream.write(errorFile);
170+
});
171+
158172
it('should compile a single sass file if the file name has been changed in the stream', function(done) {
159173
var sassFile = createVinyl('mixins.scss');
160174
var stream;

0 commit comments

Comments
 (0)