-
-
Notifications
You must be signed in to change notification settings - Fork 255
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
Add registerDebugInfoHelper
API to allow customization of console output during tests.
#608
Conversation
@rwjblue any thoughts on this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, can you add a name
field to DebugInfoHelper
?
Done, and also simplified the interface. The conditional check can now be done inside the |
@rwjblue I think this is good to go once CI passes. |
/** | ||
* Registers a custom debug info helper to augment the output for test isolation validation. | ||
* | ||
* @public |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😱 *
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, what's wrong here? I'm confused :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OH YOU KNOW WHAT YOU DID
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hehe
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to go once we fix the doc comment block
I'm seriously not trolling you (tempting though it is!), but there's a bunch of other modules in here that use the |
registerDebugInfoHelper
API to allow customization of console output during tests.
This PR is a WIP of an API allowing applications and addons to add their own custom test debug info. Providing this API can allow for tailored information that can assist in debugging test issues.