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

Deep logging #23

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Deep logging #23

wants to merge 5 commits into from

Conversation

shackra
Copy link

@shackra shackra commented Jan 31, 2020

Improves on logging output and failure reporting by, well, adding more Printf here and there and explaining with diffs why a query does not match when looking for a fake response.

This PR also migrates the project to go modules as it is using third party dependencies for the first time.

Add dependencies for string diff'ing
Improves user experience explicitly showing the user what is the difference between the query and
the match string set in order to aid with test debugging. This commit should fix bug Selvatico#21
Improve user experience by explaining the user that a query does not match because it was already
triggered or because the arguments passed to the query did not match allowing the user to change his
testing strategy whoever he sees fit if that is required.
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

1 participant