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

New Adapter for MiniTest 5 #127

Open
trans opened this issue Oct 25, 2013 · 4 comments
Open

New Adapter for MiniTest 5 #127

trans opened this issue Oct 25, 2013 · 4 comments

Comments

@trans
Copy link
Contributor

trans commented Oct 25, 2013

MiniTest's API changed to (finally) support custom reporters properly. Turn needs to be updated to support this new API.

Basically this means the MiniRunner class can probably be simplified to almost no code, and the heavy lifting will be taken over by an interface adapter between Turn's Reporter class and MiniTest's.

But it may be prudent to do this in such a way as to retain support MiniTest 4 still --on the other hand we could just require that users update to MiniTest 5.

(See also #122)

@clod81
Copy link

clod81 commented Apr 7, 2014

👍

2 similar comments
@knightq
Copy link

knightq commented Jun 6, 2014

+1

@ricardodovalle
Copy link

👍

@trans
Copy link
Contributor Author

trans commented Jul 11, 2014

I just pushed preliminary support to master. I guarantee some things are quite broken, but the basics are at least working.

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

No branches or pull requests

4 participants