#9 Add ability to take sourceMap as simple object (without SouceMapGe…

…nerator wrapper)
enb · Sep 8, 2015 · e8dff3a · e8dff3a
1 parent 96bf82d
commit e8dff3a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/lib/utils.js b/lib/utils.js
@@ -62,9 +62,9 @@ module.exports = {
  * @return {String}
  */
  joinContentAndSourceMap: function (content, sourceMap, opts) {
- if (!(sourceMap instanceof SourceMapGenerator)) {
- throw new Error('sourceMap should be an instance of SourceMapGenerator');
- }
+ sourceMap = sourceMap instanceof SourceMapGenerator
+ ? sourceMap.toString()
+  : JSON.stringify(sourceMap);
 
  opts = opts || {};
  opts.comment = opts.comment || 'inline';

diff --git a/tests/utils.test.js b/tests/utils.test.js
@@ -134,10 +134,10 @@ describe('Utils', function() {
  result.should.be.equal(['line1', 'line2', SOURCE_MAP_LINE].join(os.EOL));
  });
 
- it('should throw if not a SourceMapGenerator passed', function() {
- (function() {
-  return utils.joinContentAndSourceMap('some-content', SOURCE_MAP_CONSUMER);
- }).should.throw();
+ it('should be able to take sourceMap without SouceMapGenerator wrapper', function () {
+ var sourceMap = JSON.parse(SOURCE_MAP_GENERATOR.toString());
+ var result = utils.joinContentAndSourceMap(['line1', 'line2'].join(os.EOL), sourceMap);
+ result.should.be.equal(['line1', 'line2', SOURCE_MAP_LINE].join(os.EOL));
  });
 
  it('should throw if source map not passed', function() {