Merge pull request #46 from ssutar/tests_subfolders

Update transform to include subfolders from the testfixtures in the test run

Author: rwjblue

src/test-support.js

@@ -5,6 +5,7 @@
 const { runInlineTest } = require('jscodeshift/dist/testUtils');
 const fs = require('fs-extra');
 const path = require('path');
+const globby = require('globby');
 
 function transformDetails(options) {
   let root = process.cwd() + `/transforms/${options.name}/`;
@@ -23,8 +24,13 @@ function jscodeshiftTest(options) {
   let transform = require(details.transformPath);
 
   describe(details.name, function() {
-    fs.readdirSync(details.fixtureDir)
-      .filter(filename => /\.input$/.test(path.basename(filename, path.extname(filename))))
+    globby
+      .sync('**/*.input.*', {
+        cwd: details.fixtureDir,
+        absolute: true,
+        transform: entry =>
+          entry.slice(entry.indexOf('__testfixtures__') + '__testfixtures__'.length + 1),
+      })
       .forEach(filename => {
         let extension = path.extname(filename);
         let testName = filename.replace(`.input${extension}`, '');

tests/cli-test.js

@@ -225,6 +225,48 @@ QUnit.module('codemod-cli', function(hooks) {
           assert.equal(result.code, 0, 'exited with zero');
         })
       );
+
+      QUnit.test(
+        'transform should receive a subfolder file path in tests',
+        wrap(function*(assert) {
+          const realCodemodProjectPath = fs.realpathSync(codemodProject.path());
+          const expectedPath = `${realCodemodProjectPath}/transforms/main/__testfixtures__/foo/basic.input.js`;
+
+          yield execa(EXECUTABLE_PATH, ['generate', 'codemod', 'main']);
+
+          codemodProject.write({
+            transforms: {
+              main: {
+                'index.js': `
+                  const { getParser } = require('codemod-cli').jscodeshift;
+
+                  module.exports = function transformer(file, api) {
+                    const j = getParser(api);
+
+                    return j(file.source)
+                    .find(j.Literal)
+                    .forEach(path => {
+                      path.replace(
+                        j.stringLiteral(file.path)
+                      );
+                    })
+                    .toSource();
+                  }
+                `,
+                __testfixtures__: {
+                  foo: {
+                    'basic.input.js': 'var foo = "foo";',
+                    'basic.output.js': `var foo = "${expectedPath}";`,
+                  },
+                },
+              },
+            },
+          });
+
+          let result = yield execa(EXECUTABLE_PATH, ['test']);
+          assert.equal(result.code, 0, 'exited with zero');
+        })
+      );
     });
   });