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

[QA] Use SharingNG to share/list resources in the Given/Then steps #8717

Closed
saw-jan opened this issue Mar 22, 2024 · 12 comments
Closed

[QA] Use SharingNG to share/list resources in the Given/Then steps #8717

saw-jan opened this issue Mar 22, 2024 · 12 comments
Assignees
Labels

Comments

@saw-jan
Copy link
Member

saw-jan commented Mar 22, 2024

We are moving from OCS sharing to SharingNG for resource sharing. So let's slowly start using SharingNG in the Given steps to share resource.

Open Tasks:

  • share using SharingNg in the Given steps

    Available Steps
    • Resource sharing via permission endpoint
    And user "Alice" has sent the following resource share invitation:
        | resource        | <resource-name>    |
        | space           | <space-name>       |
        | sharee          | <sharee-name>      |
        | shareType       | <user|group>       |
        | permissionsRole | <permissions-role> |
    
    • Space sharing via drives.root endpoint
    And user "Alice" has sent the following space share invitation:
        | space           | <space-name>       |
        | sharee          | <sharee-name>      |
        | shareType       | <user|group>       |
        | permissionsRole | <permissions-role> |
    
    • Resource link creation via permission endpoint
    And user "Alice" has created the following resource link share:
        | resource           | <resource-name>    |
        | space              | <space-name>       |
        | permissionsRole    | <permissions-role> |
        | password           | <password>         |
        | expirationDateTime | <datetime>         |
        | displayName        | <link-name>        |
    
    • Space link creation via drives.root endpoint
    And user "Alice" has created the following space link share:
        | space              | <space-name>       |
        | permissionsRole    | <permissions-role> |
        | password           | <password>         |
        | expirationDateTime | <datetime>         |
        | displayName        | <link-name>        |
    
  • check Then steps (need to be careful)

  • Cleanup leftovers and step implementation @grgprarup (NOTE)

NOTE: ocs sharing in When steps should not be changed (Keep ocs sharing suites)

Check over these files

Local API Test Suites
API Test Suites
@S-Panta
Copy link
Contributor

S-Panta commented Apr 23, 2024

What about sharing spaces? @saw-jan

@saw-jan
Copy link
Member Author

saw-jan commented Apr 23, 2024

What about sharing spaces? @saw-jan

All sharing Given steps need to be done using graphAPI wherever possible

@S-Panta
Copy link
Contributor

S-Panta commented Apr 29, 2024

@saw-jan should this acceptance/features/apiReshare/resharing.feature file be refactored?

@saw-jan
Copy link
Member Author

saw-jan commented Apr 29, 2024

@saw-jan should this acceptance/features/apiReshare/resharing.feature file be refactored?

Let's skip that one for now

@S-Panta
Copy link
Contributor

S-Panta commented May 2, 2024

I think anyone working in this issue, it would be easier to work as folderwise.

@saw-jan
Copy link
Member Author

saw-jan commented May 22, 2024

  • Cleanup leftovers and step implementation @grgprarup

let's wait for removal of ocs sharing for this.

we can close this if all suites and features have been through

@grgprarup
Copy link
Contributor

The graph API steps are used instead of the sharing API in the given steps. All the suites are covered and related PRs are merged so closing this issue.

@ScharfViktor
Copy link
Contributor

great job 👍
maybe we could change Given steps for e2e tests?
example https://github.com/owncloud/web/blob/master/tests/e2e/support/api/share/share.ts#L35-L67
@saw-jan do we have task for that?

@saw-jan
Copy link
Member Author

saw-jan commented May 22, 2024

do we have task for that?

I don't remember any ticket. If we want to go full-on with sharingNG in e2e then I can create one.

@saw-jan
Copy link
Member Author

saw-jan commented May 22, 2024

Anyway, I created the one. owncloud/web#10949

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants