Extend test coverage for ocm

[amrita-shrestha]

Describe

OCM setup : https://owncloud.dev/ocis/development/testing/#running-test-suite-with-federated-sharing-ocm

• check activities on external shared resource
  • share with federated ocis user
    • resource at Personal drive
      • Comment: Extend test coverage for ocm #10073 (comment)
    • resource at Spaces
      • [tests-only][full-ci] add test to share project drive's resource to a federated user #10264
    • Share space: OCM. Prevent federated users from being added as members of the space #10051
      • [tests-only][full-ci] test: Test coverage for bug #10051 #10203
      • [test-only] add federated users to space #10166
      • [tests-only][full-ci] refactor test to add federated user as a member of a project space using permissions endpoint #10379
• list externally shared shares - /permissions endpoint
  • bug coverage of Permissions endpoint is not listing federated shares #9898
  • [tests-only][full-ci] add test to list permissions of a resource shared to a federated user #10294
• list externally shared shares
  • [OCM] SharedByMe / SharedWithOthers is missing resources that have been shared externally #9908
  • [tests-only][full-ci] add test to list shares shared to federated user #10290
• check that federated user can upload files. releated OCM. federated user gets a server crash when uploading files via tus #10285
  • (upload not fixed) : [QA] OCM: upload into received federated folder shows an error #10305 (comment)
  • [tests-only][full-ci] add test to upload file to a shared folder via TUS by federated user #10415
• Open a shared file as share receiver OCM cannot open shared file #10358 @nirajacharya2 [tests-only][full-ci] adding test for editing shared file(OCM) by the share receiver #10523
• list external sharewithme OCM. user loses the first share #10222 @nirajacharya2 [tests-only][full-ci] adding test coverage for sharing file with ocm #10565
  • check mimetype (Bug: OCM. user cannot see the thumbnails of the shared image file #10495)

     "file": {
       "mimeType": "image/png"
     },
     "remoteIten": {
       "file": {
         "mimeType": "image/png"
       }
    }

• delete ocm connection [test-only] delete connection tests #10224
• delete ocm share (bug: OCM. server crash after deleting share for ocm user #10213)
  • check if [test-only] delete connection tests #10224 has covered it or not
  • COVERED in this PR: [tests-only][full-ci] add tests to remove access of shared resource to federated user #10672
• download thumbnail from ocm share (bug: OCM. user cannot see the thumbnails of the shared image file #10495)
  • [tests-only][full-ci] add tests to download thumbnail preview of a federated share image #10651

• refactor multiple when as comment here => [tests-only][full-ci] add test to share project drive's resource to a federated user #10264 (comment)
  • [tests-only][full-ci] refactor multiple when steps in OCM tests #10626

Bug Report

• OCM. Removing a federated share does not update the resource's share-types #10262
• OCM. delete-accepted-user endpoint always returns 200 #10223
• try to find federation users after deleting federated conection
  • OCM. federated connection is not dropped when one of the users deletes the connection #10216
• ocm. user cannot change share role #10546
• OCM. folders are created with “/” in the shares #10582

[prashant-gurung899]

• share with federated ocis user
  - resource at Personal drive

@saw-jan
I think test for sharing personal drive's resource to a federated user is covered by following scenario:

ocis/tests/acceptance/features/apiOcm/share.feature

Line 12 in d70c210

Scenario Outline: local user shares resources to federation user

Can you confirm?Or do we need more coverage.

[saw-jan]

yes, that's correct

[nirajacharya2]

Ive added a point at the end of the issue list.

[prashant-gurung899]

Reference of OCM bugs collection: #9735

[prashant-gurung899]

• delete ocm share (bug: OCM. server crash after deleting share for ocm user #10213)
• check if [test-only] delete connection tests #10224 has covered it or not

The PR: #10224 has covered tests for listing the shares after deleting the connections but not after removing the access to the share.
I have added tests for removing access to the share here: #10672

CC @saw-jan