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

Add acceptance tests for occ #88

Merged
merged 3 commits into from
Jan 23, 2019
Merged

Add acceptance tests for occ #88

merged 3 commits into from
Jan 23, 2019

Conversation

skshetry
Copy link
Member

@skshetry skshetry commented Jan 14, 2019

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.

@codecov
Copy link

codecov bot commented Jan 14, 2019

Codecov Report

Merging #88 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b13643...3ff0913. Read the comment docs.

@codecov
Copy link

codecov bot commented Jan 14, 2019

Codecov Report

Merging #88 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2411c69...8c8b4f9. Read the comment docs.

@skshetry skshetry force-pushed the add-acceptance branch 5 times, most recently from 1cdc2c3 to 85abd9f Compare January 16, 2019 09:42
@skshetry skshetry changed the title [WIP] Add acceptance tests for occ Add acceptance tests for occ Jan 16, 2019
@skshetry skshetry force-pushed the add-acceptance branch 2 times, most recently from d398264 to 10eea5f Compare January 16, 2019 10:32
Copy link
Member

@paurakhsharma paurakhsharma left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@dpakach
Copy link
Contributor

dpakach commented Jan 18, 2019

👍

individual-it
individual-it previously approved these changes Jan 18, 2019
@skshetry skshetry changed the title Add acceptance tests for occ [DO NOT MERGE] Add acceptance tests for occ Jan 18, 2019
@skshetry skshetry dismissed individual-it’s stale review January 18, 2019 08:55

Needs to get requirements regarding exit code

@skshetry skshetry changed the title [DO NOT MERGE] Add acceptance tests for occ Add acceptance tests for occ Jan 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants