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

Add Step reporter with retryable-fail-event listener #1

Closed
wants to merge 1 commit into from
Closed

Add Step reporter with retryable-fail-event listener #1

wants to merge 1 commit into from

Conversation

LToronto
Copy link

Reporter for longer automated tests that have several steps.
This shows the usage of retryable-fail-event

Show usage of retryable-fail-event
@LToronto
Copy link
Author

I thought that an example of the retryable-fail may move things along.

@catdad
Copy link
Owner

catdad commented Jan 25, 2018

Sorry, I am a little confused by this. Are you proposing that we add another reporter to the mocha library? Your comments says you want to show the usage, but the code is actually adding a new step reporter to the project.

I am reluctant to add a new reporter, since this has already been discussed in mochajs#2529. I already have an open PR for this branch, and I think accepting such a change here would delay my PR or simply get it rejected. I think this would be more useful as a third party reporter.

@LToronto
Copy link
Author

Okay, sound good.

@LToronto LToronto closed this Jan 25, 2018
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

Successfully merging this pull request may close these issues.

2 participants