Skip to content
This repository has been archived by the owner on Feb 22, 2020. It is now read-only.

Add support for JSON output as well as a consolidated ResultWriter #161

Merged
merged 1 commit into from
Jan 16, 2020
Merged

Add support for JSON output as well as a consolidated ResultWriter #161

merged 1 commit into from
Jan 16, 2020

Conversation

suchitadoshi1987
Copy link
Collaborator

@suchitadoshi1987 suchitadoshi1987 commented Dec 19, 2019 •
edited
Loading

This change does the following:

  • Changed the name of write to toConsole
  • Added support for JSON output through toJson function
  • Renamed the result-console-writer to result-writer, renamed write to toConsole and added the functionality to support JSON output.
  • Added MockConsole for testing console specific outputs.
  • Added a test for the result-writer

Fixes #122

scalvert
scalvert suggested changes Jan 15, 2020
View reviewed changes
@scalvert scalvert merged commit 191df08 into master Jan 16, 2020
@scalvert scalvert deleted the suchita/resultWriter branch January 16, 2020 16:47
@scalvert scalvert added enhancement New feature or request internal and removed enhancement New feature or request labels Jan 16, 2020
lbdm44 pushed a commit that referenced this pull request Jan 30, 2020
@suchitadoshi1987
Add support for JSON output as well as a consolidated ResultWrit… (#161)
0b0f8f1
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@scalvert scalvert scalvert approved these changes

@lbdm44 lbdm44 Awaiting requested review from lbdm44

Assignees
No one assigned
Labels
internal
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

JSON output
2 participants
@suchitadoshi1987 @scalvert