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

[stable10] test authentication on more ocs endpoints #34629

Merged
merged 2 commits into from
Feb 27, 2019

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Feb 27, 2019

Backport #34627

and issue-34626 #34626 does not seem to be a problem in stable10

@phil-davis
Copy link
Contributor Author

drone https://drone.owncloud.com/owncloud/core/15646/523

The "http://repo.packagist.org/p/provider-2018%24921af8b442f5e8b8a79be6c00f8238a65f6eaa9c1dd87ca9e8c278aed8c784b8.json" file could not be downloaded: php_network_getaddresses: getaddrinfo failed: Temporary failure in name resolution

https://drone.owncloud.com/owncloud/core/15646/168

./tests/drone/test-phpunit.sh: line 71: 28 Segmentation fault (core dumped)

drone build has been restarted - maybe it will have a better day

@codecov
Copy link

codecov bot commented Feb 27, 2019

Codecov Report

Merging #34629 into stable10 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             stable10   #34629   +/-   ##
===========================================
  Coverage       63.98%   63.98%           
  Complexity      19227    19227           
===========================================
  Files            1272     1272           
  Lines           75724    75724           
  Branches         1291     1291           
===========================================
  Hits            48454    48454           
  Misses          26891    26891           
  Partials          379      379
Flag Coverage Δ Complexity Δ
#javascript 53.22% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 65.13% <ø> (ø) 19227 <ø> (ø) ⬇️

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 10b9ea2...3885218. Read the comment docs.

@codecov
Copy link

codecov bot commented Feb 27, 2019

Codecov Report

Merging #34629 into stable10 will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##             stable10   #34629      +/-   ##
==============================================
- Coverage       63.98%   63.98%   -0.01%     
  Complexity      19227    19227              
==============================================
  Files            1272     1272              
  Lines           75724    75723       -1     
  Branches         1291     1291              
==============================================
- Hits            48454    48450       -4     
- Misses          26891    26894       +3     
  Partials          379      379
Flag Coverage Δ Complexity Δ
#javascript 53.22% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 65.13% <ø> (-0.01%) 19227 <ø> (ø)
Impacted Files Coverage Δ Complexity Δ
apps/encryption/templates/settings-admin.php 88.23% <0%> (-5.89%) 0% <0%> (ø)
apps/encryption/lib/Session.php 98.03% <0%> (-1.97%) 22% <0%> (ø)
apps/files_trashbin/lib/Expiration.php 96.55% <0%> (-1.73%) 29% <0%> (ø)
...eratedfilesharing/lib/Controller/OcmController.php 66.06% <0%> (-0.21%) 30% <0%> (ø)

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 10b9ea2...3885218. Read the comment docs.

@individual-it individual-it merged commit d0fca8f into stable10 Feb 27, 2019
@delete-merged-branch delete-merged-branch bot deleted the stable10-authTestsOnMoreEndpoints branch February 27, 2019 12:27
@lock lock bot locked as resolved and limited conversation to collaborators Feb 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants