chore: improve coverage (#57)

test/fixtures.ts

@@ -64,16 +64,3 @@ export async function withFixtures(
 
   return result;
 }
-
-// Problem: the following doesn't quite work with Ava. Ava expects direct calls
-// to `test`, and if it doesn't find them, it thinks that there do not exist
-// any tests in a file.
-// TODO: figure out a solution. Without this, the users of withFixtures have to
-// remember to use test.serial rather than test.
-export async function testWithFixtures(
-    name: string, fixtures: Fixtures, testFn: Test) {
-  return await withFixtures(fixtures, async () => {
-    // The tests are run serially because we chdir to temp directory.
-    return await test.serial(name, testFn);
-  });
-}

test/test-clean.ts

@@ -36,7 +36,6 @@ test.failing.serial(
     'should gracefully error if tsconfig is missing', async t => {
       await withFixtures({}, async () => {
         await clean(OPTIONS);
-        t.pass();
       });
     });
 

test/test-kitchen.ts

@@ -23,22 +23,18 @@ const execOpts = {
   cwd: `${stagingPath}/kitchen`
 };
 
+console.log(`${chalk.blue(`${__filename} staging area: ${stagingPath}`)}`);
+
 /**
  * Create a staging directory with temp fixtures used
  * to test on a fresh application.
  */
 test.before(async () => {
-  try {
-    await execp('npm pack');
-    const tarball = `${pkg.name}-${pkg.version}.tgz`;
-    await renamep(tarball, `${stagingPath}/gts.tgz`);
-    await ncpp('test/fixtures', `${stagingPath}/`);
-    await execp('npm install', execOpts);
-  } catch (e) {
-    console.error('Failed to prepare test staging sandbox.');
-    console.error(e);
-    throw e;
-  }
+  await execp('npm pack');
+  const tarball = `${pkg.name}-${pkg.version}.tgz`;
+  await renamep(tarball, `${stagingPath}/gts.tgz`);
+  await ncpp('test/fixtures', `${stagingPath}/`);
+  await execp('npm install', execOpts);
 });
 
 test.serial('init', async t => {
@@ -95,7 +91,5 @@ test.serial('clean', async t => {
 test.after.always('cleanup staging', async () => {
   if (!keep) {
     stagingDir.removeCallback();
-  } else {
-    console.log(`${chalk.blue(`${__filename} staging area: ${stagingPath}`)}`);
   }
 });