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

[tests-only] separate different SharingNG share steps #9097

Merged
merged 2 commits into from
May 8, 2024

Conversation

saw-jan
Copy link
Member

@saw-jan saw-jan commented May 8, 2024

Description

- has sent the following share invitation:
+ has sent the following resource share invitation:
+ has sent the following space share invitation:    ---> (done using drives.root endpoint)

- sends the following share invitation using the Graph API:
+ sends the following resource share invitation using the Graph API:

- has created the following link share:
+ has created the following resource link share:
+ has created the following space link share:    ---> (done using drives.root endpoint)

- creates the following link share using the Graph API:
+ creates the following resource link share using the Graph API:

Other new steps:
+ updates the space share for (user|group) "<user|group>" with the following using the Graph API:

Related Issue

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)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@saw-jan saw-jan added the QA:team label May 8, 2024
@saw-jan saw-jan self-assigned this May 8, 2024
@saw-jan saw-jan force-pushed the tests/refactor-sharing-steps branch from 28b1703 to d900e31 Compare May 8, 2024 04:26
@saw-jan saw-jan marked this pull request as ready for review May 8, 2024 05:00
test: separate resource sharing GIven step from space share

test: share space via root endpoint in Given steps

test: use existing step

test: fix more steps
@saw-jan saw-jan force-pushed the tests/refactor-sharing-steps branch from b34dccf to 657fad5 Compare May 8, 2024 07:48
Copy link

sonarcloud bot commented May 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link
Contributor

@grgprarup grgprarup left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@saw-jan saw-jan merged commit abb537f into master May 8, 2024
4 checks passed
@delete-merged-branch delete-merged-branch bot deleted the tests/refactor-sharing-steps branch May 8, 2024 08:37
@@ -56,6 +56,8 @@ public function before(BeforeScenarioScope $scope): void {
}

/**
* Create link share of itme (resource) or drive (space) using drives.permissions endpoint
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Create link share of itme (resource) or drive (space) using drives.permissions endpoint
* Create link share of item (resource) or drive (space) using drives.permissions endpoint

* @Given /^user "([^"]*)" has created the following space link share:$/
*
* @param string $user
* @param TableNode|null $body
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* @param TableNode|null $body
* @param TableNode $body

* Create link share of drive (space) using drives.root endpoint
*
* @param string $user
* @param TableNode|null $body
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* @param TableNode|null $body
* @param TableNode $body

Copy link
Member Author

Choose a reason for hiding this comment

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

will do in next PR

Copy link
Member Author

Choose a reason for hiding this comment

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

Done in #9102

ownclouders pushed a commit that referenced this pull request May 8, 2024
* separate When steps using root and permissions endpoints

test: separate resource sharing GIven step from space share

test: share space via root endpoint in Given steps

test: use existing step

test: fix more steps

* test: separate resource and space link share creation

test: fix php code style
DeepDiver1975 pushed a commit that referenced this pull request May 8, 2024
* separate When steps using root and permissions endpoints

test: separate resource sharing GIven step from space share

test: share space via root endpoint in Given steps

test: use existing step

test: fix more steps

* test: separate resource and space link share creation

test: fix php code style
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants