imrpovements

Author: szmarczak

README.md

@@ -24,13 +24,15 @@ Then, enable TypeScript support either in `package.json` or `ava.config.*`:
 		"typescript": {
 			"rewritePaths": {
 				"src/": "build/"
-			}
+			},
+			"compile": true
 		}
 	}
 }
 ```
 
-Both keys and values of the `rewritePaths` object must end with a `/`. Paths are relative to your project directory.
+Both keys and values of the `rewritePaths` object must end with a `/`. Paths are relative to your project directory.\
+You can enable compilation via the `compile` property. It is recommended to set it to `false` when running `ts` in a watcher mode.
 
 Output files are expected to have the `.js` extension.
 

index.js

@@ -6,10 +6,6 @@ const execa = require('execa');
 
 const pkg = require('./package.json');
 
-function isPlainObject(x) {
-	return x !== null && typeof x === 'object' && Reflect.getPrototypeOf(x) === Object.prototype;
-}
-
 function isValidExtensions(extensions) {
 	return Array.isArray(extensions) &&
 		extensions.length > 0 &&
@@ -28,11 +24,11 @@ function isValidRewritePaths(rewritePaths) {
 }
 
 function isValidCompile(compile) {
-	return typeof compile === 'boolean' || compile === undefined;
+	return typeof compile === 'boolean';
 }
 
 async function compileTypeScript(projectDir) {
-	return execa('tsc', ['--build', path.resolve(projectDir, 'tsconfig.json'), '--incremental'], {preferLocal: true, cwd: projectDir});
+	return execa('tsc', ['--incremental'], {preferLocal: true, cwd: projectDir});
 }
 
 module.exports = ({negotiateProtocol}) => {

test/compilation.js

@@ -15,7 +15,8 @@ const compile = async provider => {
 				rewritePaths: {
 					'typescript/': 'fixtures/',
 					'compiled/': 'fixtures/compiled/'
-				}
+				},
+				compile: true
 			}
 		}).compile()
 	};

test/protocol-ava-3.2.js

@@ -5,20 +5,20 @@ const withProvider = require('./_with-provider');
 test('negotiates ava-3.2 protocol', withProvider, t => t.plan(2));
 
 test('main() ignoreChange()', withProvider, (t, provider) => {
-	const main = provider.main({config: {rewritePaths: {'src/': 'build/'}}});
+	const main = provider.main({config: {rewritePaths: {'src/': 'build/'}, compile: false}});
 	t.true(main.ignoreChange(path.join(__dirname, 'src/foo.ts')));
 	t.false(main.ignoreChange(path.join(__dirname, 'build/foo.js')));
 });
 
 test('main() resolveTestfile()', withProvider, (t, provider) => {
-	const main = provider.main({config: {rewritePaths: {'src/': 'build/'}}});
+	const main = provider.main({config: {rewritePaths: {'src/': 'build/'}, compile: false}});
 	t.is(main.resolveTestFile(path.join(__dirname, 'src/foo.ts')), path.join(__dirname, 'build/foo.js'));
 	t.is(main.resolveTestFile(path.join(__dirname, 'build/foo.js')), path.join(__dirname, 'build/foo.js'));
 	t.is(main.resolveTestFile(path.join(__dirname, 'foo/bar.ts')), path.join(__dirname, 'foo/bar.ts'));
 });
 
 test('main() updateGlobs()', withProvider, (t, provider) => {
-	const main = provider.main({config: {rewritePaths: {'src/': 'build/'}}});
+	const main = provider.main({config: {rewritePaths: {'src/': 'build/'}, compile: false}});
 	t.snapshot(main.updateGlobs({
 		filePatterns: ['src/test.ts'],
 		ignoredByWatcherPatterns: ['assets/**']

test/protocol-ava-3.js

@@ -17,7 +17,7 @@ test('main() config validation: throw when config is not a plain object', withPr
 	validateConfig(t, provider, []);
 });
 
-test('main() config validation: throw when config contains keys other than \'extensions\' or \'rewritePaths\'', withProvider, (t, provider) => {
+test('main() config validation: throw when config contains keys other than \'extensions\', \'rewritePaths\' or \'compile\'', withProvider, (t, provider) => {
 	validateConfig(t, provider, {foo: 1});
 });
 
@@ -37,25 +37,26 @@ test('main() config validation: config may not be an empty object', withProvider
 	validateConfig(t, provider, {});
 });
 
-test('main() config validation: throw when config.compile is not a boolean nor undefined', withProvider, (t, provider) => {
+test('main() config validation: throw when config.compile is not a boolean', withProvider, (t, provider) => {
 	validateConfig(t, provider, {rewritePaths: {'src/': 'build/'}, compile: 1});
+	validateConfig(t, provider, {rewritePaths: {'src/': 'build/'}, compile: undefined});
 });
 
 test('main() config validation: rewrite paths must end in a /', withProvider, (t, provider) => {
-	validateConfig(t, provider, {rewritePaths: {src: 'build/'}});
-	validateConfig(t, provider, {rewritePaths: {'src/': 'build'}});
+	validateConfig(t, provider, {rewritePaths: {src: 'build/', compile: false}});
+	validateConfig(t, provider, {rewritePaths: {'src/': 'build', compile: false}});
 });
 
 test('main() extensions: defaults to [\'ts\']', withProvider, (t, provider) => {
-	t.deepEqual(provider.main({config: {rewritePaths: {'src/': 'build/'}}}).extensions, ['ts']);
+	t.deepEqual(provider.main({config: {rewritePaths: {'src/': 'build/'}, compile: false}}).extensions, ['ts']);
 });
 
 test('main() extensions: returns configured extensions', withProvider, (t, provider) => {
 	const extensions = ['tsx'];
-	t.deepEqual(provider.main({config: {extensions, rewritePaths: {'src/': 'build/'}}}).extensions, extensions);
+	t.deepEqual(provider.main({config: {extensions, rewritePaths: {'src/': 'build/'}, compile: false}}).extensions, extensions);
 });
 
 test('main() extensions: always returns new arrays', withProvider, (t, provider) => {
-	const main = provider.main({config: {rewritePaths: {'src/': 'build/'}}});
+	const main = provider.main({config: {rewritePaths: {'src/': 'build/'}, compile: false}});
 	t.not(main.extensions, main.extensions);
 });

test/snapshots/protocol-ava-3.js.md

@@ -91,3 +91,25 @@ Generated by [AVA](https://avajs.dev).
     Error {
       message: 'Unexpected Typescript configuration for AVA. See https://github.com/avajs/typescript/blob/v${pkg.version}/README.md for allowed values.',
     }
+
+## main() config validation: throw when config contains keys other than 'extensions', 'rewritePaths' or 'compile'
+
+> Snapshot 1
+
+    Error {
+      message: 'Unexpected Typescript configuration for AVA. See https://github.com/avajs/typescript/blob/v${pkg.version}/README.md for allowed values.',
+    }
+
+## main() config validation: throw when config.compile is not a boolean
+
+> Snapshot 1
+
+    Error {
+      message: 'Unexpected Typescript configuration for AVA. See https://github.com/avajs/typescript/blob/v${pkg.version}/README.md for allowed values.',
+    }
+
+> Snapshot 2
+
+    Error {
+      message: 'Unexpected Typescript configuration for AVA. See https://github.com/avajs/typescript/blob/v${pkg.version}/README.md for allowed values.',
+    }