extract ava-files to it's own module (#925)

* ava-files should honor a `cwd` option.

* Properly resolve test files in non-watch mode.

* update test/watcher.js to reflect new AvaFiles Api

* extract ava-files to it's own thing

Author: jamestalmage

api.js

@@ -10,10 +10,10 @@ var uniqueTempDir = require('unique-temp-dir');
 var findCacheDir = require('find-cache-dir');
 var debounce = require('lodash.debounce');
 var ms = require('ms');
+var AvaFiles = require('ava-files');
 var AvaError = require('./lib/ava-error');
 var fork = require('./lib/fork');
 var CachingPrecompiler = require('./lib/caching-precompiler');
-var AvaFiles = require('./lib/ava-files');
 var RunStatus = require('./lib/run-status');
 
 function Api(options) {
@@ -23,7 +23,9 @@ function Api(options) {
 
 	EventEmitter.call(this);
 
-	this.options = options || {};
+	this.options = objectAssign({}, options);
+	this.options.cwd = this.options.cwd || process.cwd();
+	this.options.resolveTestsFrom = this.options.resolveTestsFrom || this.options.cwd;
 	this.options.match = this.options.match || [];
 	this.options.require = (this.options.require || []).map(function (moduleId) {
 		var ret = resolveCwd(moduleId);
@@ -73,7 +75,7 @@ Api.prototype._onTimeout = function (runStatus) {
 Api.prototype.run = function (files, options) {
 	var self = this;
 
-	return new AvaFiles(files)
+	return new AvaFiles({files: files, cwd: this.options.resolveTestsFrom})
 		.findTestFiles()
 		.then(function (files) {
 			return self._run(files, options);

cli.js

@@ -1,6 +1,7 @@
 #!/usr/bin/env node
 'use strict';
 
+var path = require('path');
 var debug = require('debug')('ava');
 
 // Prefer the local installation of AVA.
@@ -43,6 +44,8 @@ var conf = pkgConf.sync('ava', {
 	}
 });
 
+var pkgDir = path.dirname(pkgConf.filepath(conf));
+
 // check for valid babel config shortcuts (can be either "default" or "inherit")
 var isValidShortcut = ['default', 'inherit'].indexOf(conf.babel) !== -1;
 
@@ -136,6 +139,7 @@ var api = new Api({
 	explicitTitles: cli.flags.watch,
 	match: arrify(cli.flags.match),
 	babelConfig: conf.babel,
+	resolveTestsFrom: cli.input.length === 0 ? pkgDir : process.cwd(),
 	timeout: cli.flags.timeout,
 	concurrency: cli.flags.concurrency ? parseInt(cli.flags.concurrency, 10) : 0
 });

lib/ava-files.js

@@ -6,7 +6,7 @@ var chokidar = require('chokidar');
 var flatten = require('arr-flatten');
 var union = require('array-union');
 var uniq = require('array-uniq');
-var AvaFiles = require('./ava-files');
+var AvaFiles = require('ava-files');
 
 function rethrowAsync(err) {
 	// Don't swallow exceptions. Note that any expected error should already have
@@ -18,7 +18,10 @@ function rethrowAsync(err) {
 
 function Watcher(logger, api, files, sources) {
 	this.debouncer = new Debouncer(this);
-	this.avaFiles = new AvaFiles(files, sources);
+	this.avaFiles = new AvaFiles({
+		files: files,
+		sources: sources
+	});
 
 	this.isTest = this.avaFiles.makeTestMatcher();
 

lib/watcher.js

@@ -81,6 +81,7 @@
     "array-union": "^1.0.1",
     "array-uniq": "^1.0.2",
     "arrify": "^1.0.0",
+    "ava-files": "^0.1.1",
     "ava-init": "^0.1.0",
     "babel-code-frame": "^6.7.5",
     "babel-core": "^6.3.21",