Merge pull request #778 from rollup/gh-756

add newline after sourcemap comment

bin/src/runRollup.js

@@ -242,7 +242,7 @@ function bundle ( options ) {
 		let { code, map } = bundle.generate( options );
 
 		if ( options.sourceMap === 'inline' ) {
-			code += `\n//# ${SOURCEMAPPING_URL}=${map.toUrl()}`;
+			code += `\n//# ${SOURCEMAPPING_URL}=${map.toUrl()}\n`;
 		}
 
 		process.stdout.write( code );

src/rollup.js

@@ -94,7 +94,7 @@ export function rollup ( options ) {
 						promises.push( writeFile( dest + '.map', map.toString() ) );
 					}
 
-					code += `\n//# ${SOURCEMAPPING_URL}=${url}`;
+					code += `\n//# ${SOURCEMAPPING_URL}=${url}\n`;
 				}
 
 				promises.push( writeFile( dest, code ) );

test/cli/sourcemap-newline/_config.js

@@ -0,0 +1,9 @@
+const assert = require( 'assert' );
+
+module.exports = {
+	description: 'adds a newline after the sourceMappingURL comment (#756)',
+	command: 'rollup -i main.js -m inline',
+	result: code => {
+		assert.equal( code.slice( -1 ), '\n' );
+	}
+};

test/cli/sourcemap-newline/main.js

@@ -0,0 +1 @@
+console.log( 42 );

test/test.js

@@ -439,6 +439,7 @@ describe( 'rollup', function () {
 						else if ( config.result ) {
 							try {
 								config.result( code );
+								done();
 							} catch ( err ) {
 								done( err );
 							}