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

reporter-specific options and support for the xunit reporter to output to a file #1106

Closed
wants to merge 5 commits into from
Closed

reporter-specific options and support for the xunit reporter to output to a file #1106

wants to merge 5 commits into from

Commits on Jan 9, 2014

  1. add support for passing options to the reporter

    @demmer
    demmer committed Jan 9, 2014
    Configuration menu
    Copy the full SHA
    1f3d354 View commit details
    Browse the repository at this point in the history

  2. add a done() method to the reporter API to give it a chance to clean up 

    asynchronously at the end of all tests
    @demmer
    demmer committed Jan 9, 2014
    Configuration menu
    Copy the full SHA
    9c388ab View commit details
    Browse the repository at this point in the history

  3. add support to the xunit reporter to optionally write to a file

    @demmer
    demmer committed Jan 9, 2014
    Configuration menu
    Copy the full SHA
    6900219 View commit details
    Browse the repository at this point in the history

  4. Tweak error handling

    @demmer
    demmer committed Jan 9, 2014
    Configuration menu
    Copy the full SHA
    1d4e067 View commit details
    Browse the repository at this point in the history

  5. Change the reporter.done() calling convention 

    Previously I implemented a no-op done() method in the Base reporter class.
This is a slight simplification to make the caller responsible for checking
whether the done() method exists and only call it if it does.

The main benefit is that if a reporter class is used that doesn't actually
inherit from Base but simply implements the expected API then it will still
work even after merging the change.
    @demmer
    demmer committed Jan 9, 2014
    Configuration menu
    Copy the full SHA
    71a95f3 View commit details
    Browse the repository at this point in the history