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

Show class/method names in non-verbose mode. #76

Open
Ovid opened this issue May 5, 2017 · 6 comments
Open

Show class/method names in non-verbose mode. #76

Ovid opened this issue May 5, 2017 · 6 comments

Comments

@Ovid
Copy link

Ovid commented May 5, 2017

This is not a pull request because it's a proof of concept. This branch allows you to pass a verbose switch to the constructor to make it easier to see class and method names while running in non-verbose mode. I've often wanted that and in this case, it's because of this:

$ prove t/tcm.t
t/tcm.t .. 283/303

That was my tests hanging on test 283, but having no real way of knowing which test it was. Running the full test suite in verbose mode solved that, but it would be nice to see test progress without full verbose mode.

Thus, the branch I pushed allows that, but the output is very ugly. Further, I'm not familiar with the new Test::Builder, so the tests were mystifying to me. Thus, there's code and docs, but no tests. However, it runs just fine on my test suite.

This ticket is more for discussion than for merging the code (unless you think it's useful enough for that).

@autarch
Copy link
Member

autarch commented May 5, 2017

This looks mostly fine as is except it needs support in Role::CLI and it needs tests. That CLI should allow -v -v or -vv instead requiring -v 2. I think the former is more idiomatic.

@autarch
Copy link
Member

autarch commented May 5, 2017

Also note that Travis failed the branch because you misspelled "running" in the docs you added.

@autarch
Copy link
Member

autarch commented Nov 27, 2017

I just looked at this again and realized there's another issue. You're really not supposed to pass around Test2 context objects like that. You have to create a new one in each spot where you want to use it.

@Ovid
Copy link
Author

Ovid commented Nov 27, 2017

Sadly, I have absolutely zero time to work on this right now. I don't know Test2 well and I've not the foggiest idea of how to write tests for it. Sorry :(

@autarch
Copy link
Member

autarch commented Nov 28, 2017

When I have some round tuits I can work on this. Doing it right with the context isn't that hard.

@autarch
Copy link
Member

autarch commented Dec 23, 2017

Do you want to do this as an extension now that the API is amenable to it or do you still want this in the core? I'm certainly open to coring it since it does seem generally useful. But OTOH it'd be nice to actually test the extension mechanism now that it exists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants