Tape transformer should work with any variable name

[domoritz]

This is a feature request.

Conversion of this code works well but

var test = require('tape');

test('timing test', function (t) {
    t.plan(2);

    t.equal(typeof Date.now, 'function');
    var start = Date.now();

    setTimeout(function () {
        t.equal(Date.now() - start, 100);
    }, 100);
});

But this does not work (jest-codemods warning: (...) Argument to test function should be named "t" not "x").

var test = require('tape');

test('timing test', function (x) {
    x.plan(2);

    x.equal(typeof Date.now, 'function');
    var start = Date.now();

    setTimeout(function () {
        x.equal(Date.now() - start, 100);
    }, 100);
});

I think it would be great if the codemod supported arbitrary variable names.

[skovhus]

Good idea, but might be slightly tricky to find all the places where x is used. But I’m always open for PRs, and I’m happy to help if you are stuck.

[domoritz]

I decided to rename all test variables but still thinn it would be good if this codemod was more robust.