-
Notifications
You must be signed in to change notification settings - Fork 197
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
[tests-only][full-ci] add test to hide federated shared resource #11024
Conversation
396ea5a
to
4642a76
Compare
4642a76
to
a01b55a
Compare
d4ce85f
to
4b98c51
Compare
4b98c51
to
deb4e03
Compare
deb4e03
to
0714606
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
0714606
to
03df517
Compare
03df517
to
2d70b83
Compare
2d70b83
to
422be9a
Compare
@@ -1189,3 +1189,51 @@ Feature: an user shares resources using ScienceMesh application | |||
And user "Brian" should not have a federated share "folderToShare" shared by user "Alice" from space "Personal" | |||
And using server "LOCAL" | |||
And as "Alice" file "folderToShare/file.txt" should not exist | |||
|
|||
@issue-10719 | |||
Scenario: federated user hides the file shared by local user |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can cover two scenarios into a single Scenario Outline.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keeping separate should be fine
@@ -1189,3 +1189,51 @@ Feature: an user shares resources using ScienceMesh application | |||
And user "Brian" should not have a federated share "folderToShare" shared by user "Alice" from space "Personal" | |||
And using server "LOCAL" | |||
And as "Alice" file "folderToShare/file.txt" should not exist | |||
|
|||
@issue-10719 | |||
Scenario: federated user hides the file shared by local user |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keeping separate should be fine
422be9a
to
6f62535
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
6f62535
to
4bde4fa
Compare
Signed-off-by: prashant-gurung899 <prasantgrg777@gmail.com>
4bde4fa
to
0e7e3bb
Compare
|
[tests-only][full-ci] add test to hide federated shared resource
Description
This PR adds test to hide federated shared resource.
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: