Merge pull request #26 from rollup/handle-abstract-classes

Handle abstract classes

Author: Victorystick

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## master
 * Do not duplicate TypeScript's helpers ([#24](https://github.com/rollup/rollup-plugin-typescript/issues/24)
+* Handle `export abstract class` ([#23](https://github.com/rollup/rollup-plugin-typescript/issues/23)
 
 ### 0.4.1
 * Does not attempt resolve or transform `.d.ts` files ([#22](https://github.com/rollup/rollup-plugin-typescript/pull/22))

src/fixExportClass.ts

@@ -36,14 +36,14 @@ export default function fix ( code: string, id: string ): string {
 	// Erase comments, strings etc. to avoid erroneous matches for the Regex.
 	const cleanCode = tippex.erase( code );
 
-	const re = /export\s+(default\s+)?class(?:\s+(\w+))?/g;
+	const re = /export\s+(default\s+)?((?:abstract\s+)?class)(?:\s+(\w+))?/g;
 	let match;
 
 	while ( match = re.exec( cleanCode ) ) {
 		// To keep source maps intact, replace non-whitespace characters with spaces.
-		code = erase( code, match.index, match[ 0 ].indexOf( 'class' ) );
+		code = erase( code, match.index, match[ 0 ].indexOf( match[ 2 ] ) );
 
-		let name = match[ 2 ];
+		let name = match[ 3 ];
 
 		if ( match[ 1 ] ) { // it is a default export
 

test/sample/export-abstract-class/main.ts

@@ -0,0 +1 @@
+export abstract class A {}

test/test.js

@@ -72,6 +72,13 @@ describe( 'rollup-plugin-typescript', function () {
 		});
 	});
 
+	it( 'works with named exports for abstract classes', function () {
+		return bundle( 'sample/export-abstract-class/main.ts' ).then(function ( bundle ) {
+			const code = bundle.generate().code;
+			assert.ok( code.length > 0, code );
+		});
+	});
+
 	it( 'should use named exports for classes', function () {
 		return bundle( 'sample/export-class/main.ts' ).then( function ( bundle ) {
 			assert.equal( evaluate( bundle ).foo, 'bar' );