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 unit test for failures #8

Open
jewzaam opened this issue Sep 4, 2014 · 1 comment
Open

Add unit test for failures #8

jewzaam opened this issue Sep 4, 2014 · 1 comment
Labels
enhancement open questions
Milestone
Bill

Comments

@jewzaam
Copy link
Member

jewzaam commented Sep 4, 2014

Right now all unit tests expect success, none test failure in persisting audit data. This may expose need for tracking these failures somewhere other than server logs.
@jewzaam jewzaam added this to the 1.1.0 milestone Sep 15, 2014
@derek63
Copy link
Member

derek63 commented Nov 11, 2014

What should happen when persisting audit data fails
Log an error!
Should we inform client?

@derek63 derek63 modified the milestones: 1.2.0, 1.1.0 Nov 11, 2014
@derek63 derek63 modified the milestones: Bill, Ana Jan 19, 2015
@jewzaam jewzaam removed their assignment Feb 20, 2015
@jewzaam jewzaam removed the ready label Apr 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement open questions
Projects
None yet
Milestone
Bill
Development

No branches or pull requests
2 participants
@jewzaam @derek63