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

Run core cli acceptance tests with encryption #177

Closed
wants to merge 5 commits into from

Conversation

phil-davis
Copy link
Contributor

We are getting core test scenarios for CLI occ commands that check a user's filesystem and do other file-related things. It would be useful to know that they work with encryption.

Add drone pipelines to run them, like we have already done for core API and webUI acceptance tests.

I noticed this because of owncloud/user_ldap#509 - in user_ldap we are already running some core CLI test scenarios with both user_ldap and encryption together.

@phil-davis phil-davis self-assigned this Feb 7, 2020
@phil-davis phil-davis force-pushed the run-cli-acceptance-tests-with-encryption branch 2 times, most recently from 50821ea to 26f3263 Compare February 7, 2020 10:19
@phil-davis phil-davis force-pushed the run-cli-acceptance-tests-with-encryption branch from 2923a27 to c3e8eb0 Compare February 10, 2020 06:33
@codecov
Copy link

codecov bot commented Feb 10, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #177   +/-   ##
=========================================
  Coverage     71.94%   71.94%           
  Complexity      612      612           
=========================================
  Files            35       35           
  Lines          2299     2299           
=========================================
  Hits           1654     1654           
  Misses          645      645

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 0093cd7...c3e8eb0. Read the comment docs.

@phil-davis
Copy link
Contributor Author

This got done already in #179

@phil-davis phil-davis closed this Feb 25, 2020
@phil-davis phil-davis deleted the run-cli-acceptance-tests-with-encryption branch February 25, 2020 06:20
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.

1 participant