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

Add support for multiple reporters #2183

Merged
merged 9 commits into from Jan 1, 2022
Merged

Add support for multiple reporters #2183

merged 9 commits into from Jan 1, 2022

Commits on Jan 1, 2022

  1. Add support for multiple parallel reporters 

    This requires a bunch of different changes across the reporter
subsystem.

* We need to handle multiple reporters and their differing
  preferences in `ListeningReporter`, e.g. what to do when
  we mix reporters that capture and don't capture stdout.
* We need to change how the reporter is given output and
  how we parse reporter's output destination from CLI.
* Approval tests need to handle multireporter option
    @mjerabek @horenmar
    mjerabek authored and horenmar committed Jan 1, 2022
    Copy the full SHA
    ccd67b2 View commit details
    Browse the repository at this point in the history

  2. Add more unit tests 

    * Call order of listeners/reporters in multireporter
* Adding listeners/reporters properly updates reporter preferences
    @horenmar
    horenmar committed Jan 1, 2022
    Copy the full SHA
    8baf9c0 View commit details
    Browse the repository at this point in the history

  3. Add test for stdout pass-through behaviour in MultiReporter

    @horenmar
    horenmar committed Jan 1, 2022
    Copy the full SHA
    22547a3 View commit details
    Browse the repository at this point in the history

  4. Add test for handling of Listeners requesting stdout capture

    @horenmar
    horenmar committed Jan 1, 2022
    Copy the full SHA
    b4efa47 View commit details
    Browse the repository at this point in the history

  5. Add test for handling of reporter's assertion preferences

    @horenmar
    horenmar committed Jan 1, 2022
    Copy the full SHA
    899554b View commit details
    Browse the repository at this point in the history

  6. Improve error handling in parsing reporter specs

    @horenmar
    horenmar committed Jan 1, 2022
    Copy the full SHA
    fae0be2 View commit details
    Browse the repository at this point in the history

  7. Add version placeholders to the reporter docs

    @horenmar
    horenmar committed Jan 1, 2022
    Copy the full SHA
    4752545 View commit details
    Browse the repository at this point in the history

  8. Support '-' as output path for stdout

    @horenmar
    horenmar committed Jan 1, 2022
    Copy the full SHA
    93882f7 View commit details
    Browse the repository at this point in the history

  9. Passthrough stdout/err in multireporter's partial test end 

    This should provide the same overall stdout/err, but the new
output should feel "faster" for test cases that are entered
and exited multiple times (e.g. due to generators).
    @horenmar
    horenmar committed Jan 1, 2022
    Copy the full SHA
    b5547f2 View commit details
    Browse the repository at this point in the history