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

Make error output more readable #14

Merged
merged 5 commits into from Dec 9, 2015
Merged

Conversation

jacekjagiello
Copy link
Member

No description provided.

@@ -13,6 +13,7 @@ Feature: Handle command
Then command should end unsuccessfully
And the output should be
"""
The tests\Clearcode\CommandBusConsole\CommandBus\UnsuccessfulCommand failed to execute
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe something like this:

[2015-12-09 10:23:51] Command "tests\Clearcode\CommandBusConsole\CommandBus\UnsuccessfulCommand" with arguments [arg1="val1", arg2="val2", ..., argN="valN"] has failed due to "\Exception" ("Here exception message.").

@@ -1,27 +1,27 @@
Feature: Handle command

Scenario: Successfully handle command
When I run command "command-bus:successful-command --no-interaction"
When I run command "command-bus:successful-command --no-interaction" on "2015-12-09 11:00:59"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move setting date to Background.

pawaclawczyk added a commit that referenced this pull request Dec 9, 2015
@pawaclawczyk pawaclawczyk merged commit a3e1ae5 into master Dec 9, 2015
@pawaclawczyk pawaclawczyk deleted the pretty-error-output branch December 9, 2015 11:29
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.

None yet

2 participants