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

WIP: Adds CLI Tests #90

Closed
wants to merge 18 commits into from
Closed

WIP: Adds CLI Tests #90

wants to merge 18 commits into from

Conversation

ccpricenytimes
Copy link
Contributor

@ccpricenytimes ccpricenytimes commented Sep 3, 2016

No description provided.

@mattdougnyt mattdougnyt added QA and removed in progress labels Sep 8, 2016
@ccpricenytimes
Copy link
Contributor Author

Closing this in favor of Jest tests.

We still should have some end to end tests. I'll create an issue to address it.

@tizmagik
Copy link
Contributor

@ccpricenytimes I don't think we need to lose the work you've done here. We should be able to refactor these tests in Jest pretty easily. Thoughts?

@tizmagik tizmagik mentioned this pull request Sep 15, 2016
@ccpricenytimes ccpricenytimes deleted the testing branch February 12, 2017 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants