Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make it easy to use with existing tape test. #2

Open
Raynos opened this issue Mar 20, 2014 · 3 comments
Open

Make it easy to use with existing tape test. #2

Raynos opened this issue Mar 20, 2014 · 3 comments

Comments

@Raynos
Copy link

Raynos commented Mar 20, 2014

Would love a prova --tape test.js command that rewrites the test.js file and all its dependencies so that require('tape') becomes require('prova').

This way whether we use tape or prova is a per user decision rather then a permanent once in the code decision.

@azer
Copy link
Owner

azer commented Mar 20, 2014

How about:

var tape = require(process.env.with || "tape")

so we can do

with=prova node test

@Raynos
Copy link
Author

Raynos commented Mar 20, 2014

@azer that could work but would require rewriting my codebase.

My workflow would be to use prova locally for a week or two at work and then recommend changing to it once I feel confident with it. I would like to do this without editing my tape codebase.

@vvo
Copy link
Contributor

vvo commented Apr 1, 2014

@azer @Raynos Using either https://github.com/thlorenz/proxyquire or https://github.com/felixge/node-sandboxed-module you should be able to add a --tape option to prova right?

MylesBorins pushed a commit to MylesBorins/prova that referenced this issue Sep 10, 2014
This transform will replace all instances of "tap" or "tape" require statements in a script with require('prova').  This closes azer#2.
MylesBorins pushed a commit to MylesBorins/prova that referenced this issue Sep 10, 2014
This transform will replace all instances of "tap" or "tape" require statements in a script with require('prova').  This closes azer#2.
MylesBorins pushed a commit to MylesBorins/prova that referenced this issue Sep 18, 2014
This transform will replace all instances of "tap" or "tape" require statements in a script with require('prova').  This closes azer#2.
aghassemi pushed a commit to aghassemi/prova that referenced this issue Dec 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants