feat(src): add support for "only", "skip", and "title" test options in fixtures (#90)

Author: Xunnamius

src/__tests__/__snapshots__/plugin-tester.js.snap

@@ -10,6 +10,8 @@ exports[`can fail tests in fixtures at an absolute path 1`] = `actual output doe
 
 exports[`default will throw if output changes 1`] = `Expected output to not change, but it did`;
 
+exports[`fixture tests cannot be both only-ed and skipped 1`] = `Cannot enable both skip and only on a test`;
+
 exports[`plugin is required 1`] = `plugin is a required parameter.`;
 
 exports[`snapshot option can be derived from the root config 1`] = `\`output\` cannot be provided with \`snapshot: true\``;

src/__tests__/fixtures/custom-title-fixtures/fixture/code.js

@@ -0,0 +1 @@
+'use strict';

src/__tests__/fixtures/custom-title-fixtures/fixture/options.json

@@ -0,0 +1 @@
+{"title": "custom fixture test title"}

src/__tests__/fixtures/custom-title-fixtures/fixture/output.js

@@ -0,0 +1 @@
+'use strict';

src/__tests__/fixtures/only-fixtures/only/code.js

@@ -0,0 +1 @@
+'use strict';

src/__tests__/fixtures/only-fixtures/only/options.json

@@ -0,0 +1 @@
+{"only": true}

src/__tests__/fixtures/only-fixtures/only/output.js

@@ -0,0 +1 @@
+'use strict';

src/__tests__/fixtures/skip-fixtures/skip/code.js

@@ -0,0 +1 @@
+'use strict';

src/__tests__/fixtures/skip-fixtures/skip/options.json

@@ -0,0 +1 @@
+{"skip": true}

src/__tests__/fixtures/skip-fixtures/skip/output.js

@@ -0,0 +1 @@
+'use strict';

src/__tests__/fixtures/skip-only-fixtures/failure/code.js

@@ -0,0 +1 @@
+'use strict';

src/__tests__/fixtures/skip-only-fixtures/failure/options.json

@@ -0,0 +1 @@
+{"skip": true, "only": true}

src/__tests__/fixtures/skip-only-fixtures/failure/output.js

@@ -0,0 +1 @@
+'use strict';

src/__tests__/plugin-tester.js

@@ -147,6 +147,55 @@ test('tests cannot be both only-ed and skipped', async () => {
   ).rejects.toThrowErrorMatchingSnapshot()
 })
 
+test('fixture tests can be skipped', async () => {
+  await runPluginTester(
+    getOptions({
+      fixtures: getFixturePath('skip-fixtures'),
+      tests: null,
+    }),
+  )
+
+  expect(itSkipSpy).toHaveBeenCalledTimes(1)
+  expect(itSpy).not.toHaveBeenCalled()
+})
+
+test('fixture tests can be only-ed', async () => {
+  await runPluginTester(
+    getOptions({
+      fixtures: getFixturePath('only-fixtures'),
+      tests: null,
+    }),
+  )
+
+  expect(itOnlySpy).toHaveBeenCalledTimes(1)
+  expect(itSpy).not.toHaveBeenCalled()
+})
+
+test('fixture tests cannot be both only-ed and skipped', async () => {
+  await expect(
+    runPluginTester(
+      getOptions({
+        fixtures: getFixturePath('skip-only-fixtures'),
+        tests: null,
+      }),
+    ),
+  ).rejects.toThrowErrorMatchingSnapshot()
+})
+
+test('fixture tests accept custom test title', async () => {
+  await runPluginTester(
+    getOptions({
+      fixtures: getFixturePath('custom-title-fixtures'),
+      tests: null,
+    }),
+  )
+
+  expect(itSpy).toHaveBeenCalledWith(
+    'custom fixture test title',
+    expect.any(Function),
+  )
+})
+
 test('default will throw if output changes', async () => {
   const tests = ['var hello = "hi";']
   const options = getOptions({plugin: identifierReversePlugin, tests})

src/plugin-tester.js

@@ -91,6 +91,7 @@ function pluginTester({
         fixture,
         testFilepath: testFilename = fixture || filename,
       } = mergeWith({}, testerConfig, toTestConfig(testConfig), mergeCustomizer)
+
       assert(
         (!skip && !only) || skip !== only,
         'Cannot enable both skip and only on a test',
@@ -300,7 +301,13 @@ const createFixtureTests = (fixturesDir, options) => {
       return
     }
 
-    it(blockTitle, async () => {
+    const {only, skip, title} = localFixtureOptions
+
+    assert(
+      (!skip && !only) || skip !== only,
+      'Cannot enable both skip and only on a test',
+    )
+    ;(only ? it.only : skip ? it.skip : it)(title || blockTitle, async () => {
       const {
         plugin,
         pluginOptions,
@@ -311,9 +318,11 @@ const createFixtureTests = (fixturesDir, options) => {
         ...rest
       } = options
 
-      const hasBabelrc = ['.babelrc', '.babelrc.js', '.babelrc.cjs'].some(
-        babelrc => fs.existsSync(path.join(fixtureDir, babelrc)),
-      )
+      const hasBabelrc = [
+        '.babelrc',
+        '.babelrc.js',
+        '.babelrc.cjs',
+      ].some(babelrc => fs.existsSync(path.join(fixtureDir, babelrc)))
 
       const {babelOptions} = mergeWith(
         {},