-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add acceptance tests for occ #88
Conversation
18febfd
to
3ff0913
Compare
Codecov Report
@@ Coverage Diff @@
## master #88 +/- ##
=========================================
Coverage 60.28% 60.28%
Complexity 52 52
=========================================
Files 11 11
Lines 214 214
=========================================
Hits 129 129
Misses 85 85 Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #88 +/- ##
=========================================
Coverage 60.28% 60.28%
Complexity 52 52
=========================================
Files 11 11
Lines 214 214
=========================================
Hits 129 129
Misses 85 85 Continue to review full report at Codecov.
|
1cdc2c3
to
85abd9f
Compare
d398264
to
10eea5f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
👍 |
Needs to get requirements regarding exit code
e023de3
to
9ee0fd6
Compare
cab1d63
to
33ed100
Compare
4e3a885
to
e5dc938
Compare
e5dc938
to
7599cf7
Compare
2d24157
to
8362584
Compare
8362584
to
8c8b4f9
Compare
Fixes #69
I have invoked a command within the steps. I'll change that if we think that's bad. Currently, we havenot tested with multiple users as this requires webui to login and logout multiple users.