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.

Sign up for GitHub

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

Jump to bottom

Make Test::Builder::Tester use the legacy TAP formatter #242

Closed
schwern opened this issue Nov 30, 2011 · 2 comments
Closed

Make Test::Builder::Tester use the legacy TAP formatter #242

schwern opened this issue Nov 30, 2011 · 2 comments
Labels
DO IT Formatter Test-Builder2
Milestone
Legacy format com...

Comments

@schwern
Copy link
Contributor

schwern commented Nov 30, 2011

Make tests run with Test::Builder::Tester use the legacy TAP formatter being written in #215.

This will allow tests currently written using TBT to remain working.

Include an option to change the formatter.
schwern added a commit that referenced this issue Nov 30, 2011
@schwern
Add the ability to clear the debugging streamer.
29b3aef 
Useful for Test::Builder::Tester

For #242
schwern added a commit that referenced this issue Nov 30, 2011
@schwern
Convert TBT to swap out TestState instead of individual file handles.
25b78e0 
This is much cleaner, using TB2::Streamer::Debug instead of individual tied handles.

It also allows switching over to the legacy TAP formatter.

For #242
schwern added a commit that referenced this issue Nov 30, 2011
@schwern
Strip the prefix and numbers off the Test::Builder::Tester tests.
070720c 
Redundant now they're in their own directory.

For #242
schwern added a commit that referenced this issue Nov 30, 2011
@schwern
Stray $DB::single
689d782 
For #242
schwern added a commit that referenced this issue Nov 30, 2011
@schwern
Fix it so the complaint shows what output we got.
abd79a5 
We've already read from the Streamer in check(), so a second read will return nothing.
Have to use output_for() to get everything.

For #242
schwern added a commit that referenced this issue Nov 30, 2011
@schwern
Test that TBT uses the legacy formatter.
ca67644 
For #242
@schwern
Copy link
Contributor Author

schwern commented Nov 30, 2011

Test::Builder::Tester never looked so good.

Gotta add the option to use an alternative formatter. The subtest tests are failing because of that.

schwern added a commit that referenced this issue Dec 6, 2011
@schwern
Emulate the existing behavior and pretend the test already started.
0089158 
Doing this mostly for compatibility.

For #242
schwern added a commit that referenced this issue Dec 6, 2011
@schwern
We can do "the ending stuff" because we're just going to throw this s…
cdc71cb 
…tate out.

The important thing is not to do the exit code, because the test will probably be
in a weird state... but I'm not 100% sure if this is because of TBT or because of
how we're testing TBT.

For #242
schwern added a commit that referenced this issue Dec 6, 2011
@schwern
Change the subtest tests to use the TAP formatter.
94186ad 
They also don't have to worry about HARNESS_ACTIVE, TBT takes care of it.

For #242
@schwern
Copy link
Contributor Author

schwern commented Dec 6, 2011

I don't know how much of a real impact this is going to have, but TBT is a lot nicer now.

@schwern schwern closed this as completed Dec 6, 2011
This was referenced Feb 12, 2012
Closed
Closed
@schwernbot schwernbot mentioned this issue Mar 16, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
DO IT Formatter Test-Builder2
Projects
None yet
Milestone
Legacy format compatibility
Development

No branches or pull requests
1 participant
@schwern