Parse only existing sourceMapText

Chadkin · Chadkin · commit 7eaf8c8093a6 · 2016-08-05T16:44:45.000+03:00
diff --git a/src/index.js b/src/index.js
@@ -219,7 +219,7 @@ export default function typescript ( options ) {
 					`\nimport { __assign, __awaiter, __extends, __decorate, __metadata, __param } from 'typescript-helpers';`,
 
 				// Rollup expects `map` to be an object so we must parse the string
-				map: JSON.parse(transformed.sourceMapText)
+				map: transformed.sourceMapText ? JSON.parse(transformed.sourceMapText) : null
 			};
 		}
 	};
diff --git a/test/test.js b/test/test.js
@@ -201,6 +201,13 @@ describe( 'rollup-plugin-typescript', function () {
 			inlineSourceMap: true,
 		});
 	});
+
+	it ( 'should not fail if source maps are off', function () {
+		return bundle( 'sample/overriding-typescript/main.ts', {
+			inlineSourceMap: false,
+			sourceMap: false
+		});
+	});
 });
 
 function fakeTypescript( custom ) {

Original file line number	Diff line number	Diff line change
`@@ -219,7 +219,7 @@ export default function typescript ( options ) {`
`219`	`219`	`\nimport { __assign, __awaiter, __extends, __decorate, __metadata, __param } from 'typescript-helpers';`,
`220`	`220`
`221`	`221`	// Rollup expects `map` to be an object so we must parse the string
`222`		`- map: JSON.parse(transformed.sourceMapText)`
	`222`	`+ map: transformed.sourceMapText ? JSON.parse(transformed.sourceMapText) : null`
`223`	`223`	`};`
`224`	`224`	`}`
`225`	`225`	`};`