Skip to content

Commit 3687c91

Browse files
committed
Fix source map generation
1 parent c9fac75 commit 3687c91

File tree

2 files changed

+8
-15
lines changed

2 files changed

+8
-15
lines changed

index.js

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,10 @@ module.exports = function(opts) {
3535
// Add a string containing the list of added dependencies for
3636
// debugging purposes.
3737
newFile.buildResponse = buildResponse.replace('FUNCTION', _fName);
38-
_s.write(newFile);
3938
if (sourceMap) {
40-
_s.write(new File({
41-
path: _fName + '.map',
42-
contents: new Buffer(sourceMap)
43-
}));
39+
newFile.sourceMap = JSON.parse(sourceMap);
4440
}
41+
_s.write(newFile);
4542
_s.resume();
4643
_s.end();
4744
},

test/main.js

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,6 @@ describe('gulp-requirejs', function() {
2828
});
2929

3030
it('should emit source map file when configured', function(done) {
31-
var filesInPipe = [];
32-
3331
var stream = grjs({
3432
out: 'simple_init.js',
3533

@@ -46,15 +44,13 @@ describe('gulp-requirejs', function() {
4644
create: true
4745
});
4846

49-
stream.on('data', (chunk) => {
50-
filesInPipe.push(chunk.path);
51-
});
52-
53-
stream.on('end', function() {
47+
stream.on('data', (output) => {
5448
try {
55-
filesInPipe.length.should.equal(2);
56-
filesInPipe.should.containEql('simple_init.js');
57-
filesInPipe.should.containEql('simple_init.js.map');
49+
output.path.should.equal('simple_init.js');
50+
output.should.have.property('sourceMap');
51+
output.sourceMap.should.be.Object();
52+
output.sourceMap.sources.should.containEql('vendor/simple_amd_file.js');
53+
output.sourceMap.sources.should.containEql('simple_init.js');
5854
done();
5955
} catch(e) {
6056
done(e);

0 commit comments

Comments
 (0)