Skip to content

[test] Test/user in share20ocscontroller#34609

Closed
phil-davis wants to merge 4 commits into
masterfrom
test/user-in-share20ocscontroller
Closed

[test] Test/user in share20ocscontroller#34609
phil-davis wants to merge 4 commits into
masterfrom
test/user-in-share20ocscontroller

Conversation

@phil-davis

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

Copy link
Copy Markdown
Contributor

Just run two of the apiFederation failing scenarios. This demonstrates the problem with declining and cleaning up entries in the oc_share_external table when deleting a user on the remote federated server.

See details in comment #34568 (comment)

@phil-davis phil-davis self-assigned this Feb 25, 2019
@codecov

codecov Bot commented Feb 25, 2019

Copy link
Copy Markdown

Codecov Report

Merging #34609 into master will decrease coverage by 12.17%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #34609       +/-   ##
=============================================
- Coverage     65.22%   53.05%   -12.18%     
=============================================
  Files          1203       61     -1142     
  Lines         69816     7208    -62608     
  Branches       1280     1280               
=============================================
- Hits          45540     3824    -41716     
+ Misses        23904     3012    -20892     
  Partials        372      372
Flag Coverage Δ Complexity Δ
#javascript 53.05% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit ? ?
Impacted Files Coverage Δ Complexity Δ
apps/updatenotification/templates/admin.php
lib/private/Encryption/Keys/Storage.php
lib/private/App/CodeChecker/NodeVisitor.php
lib/private/RedisFactory.php
apps/dav/lib/Avatars/AvatarNode.php
...s/dav/appinfo/Migrations/Version20170202213905.php
apps/dav/lib/Upload/ChunkLocationProvider.php
apps/files/lib/AppInfo/Application.php
apps/systemtags/list.php
apps/files_external/lib/Lib/Backend/DAV.php
... and 1127 more

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 2af4d3e...2b9a580. Read the comment docs.

@codecov

codecov Bot commented Feb 25, 2019

Copy link
Copy Markdown

Codecov Report

Merging #34609 into master will decrease coverage by 12.18%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #34609       +/-   ##
=============================================
- Coverage     65.23%   53.05%   -12.19%     
=============================================
  Files          1203       61     -1142     
  Lines         69825     7208    -62617     
  Branches       1280     1280               
=============================================
- Hits          45549     3824    -41725     
+ Misses        23904     3012    -20892     
  Partials        372      372
Flag Coverage Δ Complexity Δ
#javascript 53.05% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit ? ?
Impacted Files Coverage Δ Complexity Δ
apps/updatenotification/templates/admin.php
lib/private/Encryption/Keys/Storage.php
lib/private/App/CodeChecker/NodeVisitor.php
lib/private/RedisFactory.php
apps/dav/lib/Avatars/AvatarNode.php
...s/dav/appinfo/Migrations/Version20170202213905.php
apps/dav/lib/Upload/ChunkLocationProvider.php
apps/files/lib/AppInfo/Application.php
apps/systemtags/list.php
apps/files_external/lib/Lib/Backend/DAV.php
... and 1127 more

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 80aaf87...18b02eb. Read the comment docs.

@phil-davis phil-davis force-pushed the test/user-in-share20ocscontroller branch from 86b103e to 18b02eb Compare February 25, 2019 15:38
@phil-davis

Copy link
Copy Markdown
Contributor Author

This was for testing only - closing.

@phil-davis phil-davis closed this Feb 28, 2019
@phil-davis phil-davis deleted the test/user-in-share20ocscontroller branch February 28, 2019 03:36
@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.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants