UI tests for restricting the sharing

[individual-it]

Description

UI test to see if various ways of restricting the sharing options work e.g "Share with group membership only" #29241

This PR contains also the changes from #29388 because it depends on them

Related Issue

#29075

Motivation and Context

test sharing behaviour

How Has This Been Tested?

run tests locally, travis will test again

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)

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.

[individual-it]

@jvillafanez could you please check if the tests described in f0a4b4d#diff-4a8972ffadc126987f05eebd52151aec are what you have programmed
And if you can up with more sensible checks, that would be great!

[phil-davis]

Just little things.
This seems a good start to testing the sharing restrictions. After this we can easily look through the other sharing settings and make basic UI tests to confirm that other combinations work.

[phil-davis]

s/readable/readable

[phil-davis]

s/globaly/globally/

[codecov]

Codecov Report

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

@@            Coverage Diff            @@
##             master   #29406   +/-   ##
=========================================
  Coverage     60.47%   60.47%           
  Complexity    17235    17235           
=========================================
  Files          1032     1032           
  Lines         57389    57389           
=========================================
  Hits          34707    34707           
  Misses        22682    22682

---

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 6a3a485...8626ac2. Read the comment docs.

[PVince81]

I suggest rebasing for CI

[jvillafanez]

I think we should include tests for the case where the checkbox aren't checked. Other than that, the tests look fine.

[individual-it]

added one more test and rebased again

[phil-davis]

LGTM - assuming CI passes

[individual-it]

backport in #29470

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.