ocis icon indicating copy to clipboard operation
ocis copied to clipboard

[QA] Test coverage for cli commands

Open amrita-shrestha opened this issue 1 year ago • 4 comments

NOTE: Test setup done in https://github.com/owncloud/ocis/pull/9546

Add test coverage for CLI commands introduced in oCIS: Docs: https://github.com/owncloud/ocis/tree/master/ocis

To Implement:

  • https://github.com/owncloud/ocis/issues/8925
  • https://github.com/owncloud/ocis/issues/5602

QA issues:

  • https://github.com/owncloud/ocis/issues/10167
  • https://github.com/owncloud/ocis/issues/10096

Completed tasks:

  • https://github.com/owncloud/ocis/issues/9286
    • [x] https://github.com/owncloud/ocis/pull/9578
  • https://github.com/owncloud/ocis/issues/9472 @prashant-gurung899
    • [x] all file versions https://github.com/owncloud/ocis/pull/9584
    • [x] from specific space https://github.com/owncloud/ocis/pull/9639
    • [x] from specific file https://github.com/owncloud/ocis/pull/9627
  • https://github.com/owncloud/ocis/issues/9474
    • [x] reset password: https://github.com/owncloud/ocis/pull/9546
    • [x] try to reset password of non-existing user @S-Panta https://github.com/owncloud/ocis/pull/9575
    • [x] #9581 @S-Panta https://github.com/owncloud/ocis/pull/9593
  • https://github.com/owncloud/ocis/issues/9393 @S-Panta
    • [x] https://github.com/owncloud/ocis/pull/9617
  • https://github.com/owncloud/ocis/issues/9103 @ishabaral
    • [x] reindex all spaces https://github.com/owncloud/ocis/pull/9630
    • [x] reindex specific space https://github.com/owncloud/ocis/pull/9630
    • [x] https://github.com/owncloud/ocis/issues/10793 : use ocis notifications send-email to use the service. Make sure to write test that email notification comes daily when daily is chosen in UI . (and other variation of this scenario) @anon-pradip https://github.com/owncloud/ocis/pull/10920

amrita-shrestha avatar Jul 04 '24 10:07 amrita-shrestha

Keeping the issue in backlog due to the bug : https://github.com/owncloud/ocis/issues/8925 and https://github.com/owncloud/ocis/issues/5602

ishabaral avatar Sep 16 '24 06:09 ishabaral

Keeping the issue in backlog due to the bug : #8925 and #5602

These are features to implement

saw-jan avatar Sep 20 '24 05:09 saw-jan

We should make a list of all available useful (testable) commands of each service so that we can know what we have convered. NOTE: we may not want to tests commands like help, init, server, health, version, benchmark, migrate (the requirement can change)

  • :heavy_check_mark: (has test)
  • :construction: (no test)

ocis commands:

Service Command Test
ocis backup consistency -p <basepath> :heavy_check_mark:
init --diff :construction: #3645, #10444, #10560

Per service:

Service Command Test
shares cleanup #10712

saw-jan avatar Sep 26 '24 04:09 saw-jan

  • [ ] https://github.com/owncloud/ocis/issues/10793 : use ocis notifications send-email to use the service. Make sure to write test that email notification comes daily when daily is chosen in UI . (and other variation of this scenario) @anon-pradip https://github.com/owncloud/ocis/pull/10920

The test coverage for the step: And user "Alice" has expired the last share of resource "lorem.txt" inside of the space "Personal" was added in: https://github.com/owncloud/ocis/pull/10920/commits/e0ec69f4a62990687d8a4f7d105ae4f7e53bf3eb but the message in inbox for the step is found to be flaky and therefore raised an issue: https://github.com/owncloud/ocis/issues/11001. So, the test coverage is removed for now but step can be added later in the future when this flakiness is solved.

anon-pradip avatar Feb 13 '25 08:02 anon-pradip