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 tests to check re-share behavior inside blacklisted from sharing group #33865

Merged
merged 1 commit into from
Dec 14, 2018

Conversation

paurakhsharma
Copy link
Member

Description

This PR webUI tests for files_external delete local storage mount.

Related issue: #33824

How Has This Been Tested?

🤖

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Acceptance test

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@paurakhsharma paurakhsharma added this to the development milestone Dec 12, 2018
@paurakhsharma paurakhsharma self-assigned this Dec 12, 2018
@paurakhsharma paurakhsharma changed the title Add tests for to check reshare behaviour inside blacklisted from shar… Add tests to check re-share behavior inside blacklisted from sharing group Dec 12, 2018
@codecov
Copy link

codecov bot commented Dec 12, 2018

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #33865   +/-   ##
=========================================
  Coverage     64.42%   64.42%           
  Complexity    18318    18318           
=========================================
  Files          1197     1197           
  Lines         69260    69260           
  Branches       1276     1276           
=========================================
  Hits          44621    44621           
  Misses        24267    24267           
  Partials        372      372
Flag Coverage Δ Complexity Δ
#javascript 52.98% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 65.75% <ø> (ø) 18318 <ø> (ø) ⬇️

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 b0b384d...70a177f. Read the comment docs.

@codecov
Copy link

codecov bot commented Dec 12, 2018

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #33865   +/-   ##
=========================================
  Coverage     64.76%   64.76%           
  Complexity    18330    18330           
=========================================
  Files          1198     1198           
  Lines         69382    69382           
  Branches       1276     1276           
=========================================
  Hits          44938    44938           
  Misses        24072    24072           
  Partials        372      372
Flag Coverage Δ Complexity Δ
#javascript 52.98% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 66.13% <ø> (ø) 18330 <ø> (ø) ⬇️

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 98d8d44...74721de. Read the comment docs.

phil-davis
phil-davis previously approved these changes Dec 12, 2018
@paurakhsharma
Copy link
Member Author

Backport on: #33866

skshetry
skshetry previously approved these changes Dec 12, 2018
@paurakhsharma paurakhsharma dismissed stale reviews from skshetry and phil-davis December 12, 2018 09:16

I need to add more tests to check webUI

@paurakhsharma paurakhsharma force-pushed the webUI-test-excludeGroup-reshare branch 3 times, most recently from 034104d to 9de94be Compare December 14, 2018 07:41
Copy link
Contributor

@phil-davis phil-davis left a comment

Choose a reason for hiding this comment

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

Looks good, just some comments on naming

@paurakhsharma paurakhsharma merged commit 1be81f0 into master Dec 14, 2018
@delete-merged-branch delete-merged-branch bot deleted the webUI-test-excludeGroup-reshare branch December 14, 2018 10:44
@lock lock bot locked as resolved and limited conversation to collaborators Dec 14, 2019
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.

3 participants