-
Notifications
You must be signed in to change notification settings - Fork 7
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
Minitest #1
Comments
That would be awesome! Let me know if I can help in any way.
|
Thanks! I've been doing some research. I was wondering if you could point me to any good examples of gems that allow for integration with either Rspec or Minitest? I'd like to see how other people have handled it. |
Hi Coraline, I've been working on this for a few days and have finally come up with what I think is a viable solution. However, I've run up against a wall and don't know what I'm missing (I'm not really sure how to test what I'm doing except by pulling my changes into my own application and running the test suite to see if it works). I feel like it might be something really obvious such as missing a require or include somewhere. I was wondering if you could spare a few minutes to review my changes and see if there's anything you notice. What I've done is hook into If you could point out anything I've missed or help me figure out how to test this better and produce helpful errors, I would really appreciate it. The idea of trying to generate a fake test suite to run this against is kind of mind-boggling. Here is the link to the current diff between your work and mine. |
Hi Wendy, I have a party tonight but can definitely look at this tomorrow. Thanks for all the great work! —Coraline
|
Hi Wendy, I’m lining up someone who is more familiar with minitest than me to help you out if that’s okay with you? Let me know… Thank!
|
That's great. Thanks for doing that! |
Can you email me at [email protected] mailto:[email protected]? I’ll introduce you to Carol and then you two can pair on this. :) Best,
|
Would you welcome a pull request that makes it so that this can run with Minitest? It sounds more fun to me to try and get that working than to spend days and days figuring out how to make my test suite run faster. :)
The text was updated successfully, but these errors were encountered: