Scheduler follow-up

* Write synchronously and ignore errors
* Change filename
* Fix integration test

Author: novemberborn

lib/scheduler.js

@@ -3,14 +3,16 @@ const path = require('path');
 const writeFileAtomic = require('write-file-atomic');
 const isCi = require('./is-ci');
 
-const FILE_NAME_FAILING_TEST = 'failing-test.json';
+const FILENAME = 'failing-tests.json';
 
 module.exports.storeFailedTestFiles = (runStatus, cacheDir) => {
 	if (isCi || !cacheDir) {
 		return;
 	}
 
-	writeFileAtomic(path.join(cacheDir, FILE_NAME_FAILING_TEST), JSON.stringify(runStatus.getFailedTestFiles()));
+	try {
+		writeFileAtomic.sync(path.join(cacheDir, FILENAME), JSON.stringify(runStatus.getFailedTestFiles()));
+	} catch {}
 };
 
 // Order test-files, so that files with failing tests come first
@@ -19,7 +21,7 @@ module.exports.failingTestsFirst = (selectedFiles, cacheDir, cacheEnabled) => {
 		return selectedFiles;
 	}
 
-	const filePath = path.join(cacheDir, FILE_NAME_FAILING_TEST);
+	const filePath = path.join(cacheDir, FILENAME);
 	let failedTestFiles;
 	try {
 		failedTestFiles = JSON.parse(fs.readFileSync(filePath));

test-tap/api.js

@@ -542,7 +542,7 @@ for (const opt of opts) {
 		t.end();
 	});
 
-	test('caching is enabled by default - workerThreads: {opt.workerThreads}', t => {
+	test(`caching is enabled by default - workerThreads: ${opt.workerThreads}`, t => {
 		del.sync(path.join(__dirname, 'fixture/caching/node_modules'));
 
 		const api = apiCreator({
@@ -556,7 +556,12 @@ for (const opt of opts) {
 				const files = fs.readdirSync(path.join(__dirname, 'fixture/caching/node_modules/.cache/ava'));
 				t.is(files.filter(x => x.endsWith('.js')).length, 1);
 				t.is(files.filter(x => x.endsWith('.map')).length, 1);
-				t.is(files.length, 2);
+				if (files.length === 3) {
+					// This file may be written locally, but not in CI.
+					t.is(files.filter(x => x.startsWith('failing-tests.json')).length, 1);
+				} else {
+					t.is(files.length, 2);
+				}
 			});
 	});