Adds the Ability To Define Custom Reporters #299

Merged
merged 3 commits into from Aug 11, 2016

Projects

None yet

4 participants

@samjonester
Contributor
  • Adds the ability to define Standard Out reporters and File System reporters through plugin configuration.
  • Adds documentation for defining custom reporters.
  • Adds example project with cucumber test for defining custom reporters.
  • Refactors responsibilities out of larger classes.
samjonester added some commits Jul 21, 2016
@samjonester samjonester Updates error message tested for when test suite fails on a syntax error f320fcf
@samjonester samjonester Adds the ability to define Standard Out reporters through plugin conf…
…iguration.

- Adds `reporters` property to support list of reports.
- Adds example project and cucumber test defining custom reporters
- Adds documentation for defining custom reporters.
- Allows `STANDARD` to be used as a reporter in the list defined
  reporters.
- Defaults to the `STANDARD` reporter if none are provided.
- Splits responsibilities out of classes that were growing large and
  difficult to test. `AbstractJasmineMojo` and `SpecRunnerExecutor`.
026a33b
@samjonester samjonester Adds the ability to define File System reporters through plugin confi…
…guration.

    - Adds `fileSystemReporters` property to support list of reports.
    - Adds file system reporter to custom reporters example project.
    - Updates documentation for defining custom reporters.
    - Allows `JUNIT_XML` to be used as a file system reporter in the list of defined reporters.
    - Defaults to the `JUNIT_XML` reporter if none are provided.
911f9ec
@coveralls
coveralls commented Jul 28, 2016 edited

Coverage Status

Coverage increased (+3.3%) to 58.591% when pulling 911f9ec on samjonester:multiple-reporters into 3621727 on searls:master.

@dylanb
dylanb commented Aug 11, 2016

@searls @klieber any feedback on this PR?

@klieber
Collaborator
klieber commented Aug 11, 2016

@dylanb, thanks for the pull request! It looks good to me. I'll try to get it in a release soon.

@klieber klieber merged commit 88e4387 into searls:master Aug 11, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+3.3%) to 58.591%
Details
@dylanb
dylanb commented Aug 11, 2016

Thanks to @samjonester

@klieber
Collaborator
klieber commented Aug 11, 2016

Yes, sorry about that. Thanks to @samjonester for the pull request. Thanks @dylanb for reminding me to look at it. :)

@dylanb
dylanb commented Sep 12, 2016

@klieber When do you think this change will be published?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment