-
Notifications
You must be signed in to change notification settings - Fork 187
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
Cleanup core API test codes from oCIS #5337
Comments
@SwikritiT product repo was used a long time ago for discussions/issues about features and how they should be implemented. It is not used anymore. but I think we can keep @phil-davis What should we do ❓ |
Check that the "product" issues are still correct - most of them should have been moved to the ocis repo. And update them to have the current ocis issue number. That will make it easy for developers to find the issue - they will not have to realize about the history of the "product" repo. If there are ocis issues still open in the product repo, then we should review them (write updated information about the issue) and if still relevant, moved them to the ocis repo. Ping me if you cannot move them yourself. |
We can do a final check and then close this one. |
@amrita-shrestha let's close this one if everything in done |
All task has been done. so, closing this issue |
Copying core API tests completed with PRs #5280 and #5306
Features and test codes need some cleanup:
***Issue***.feature
) @grgprarup [full-ci] [tests-only] Remove core bug demonstration scenarios #5485, [full-ci] [tests-only] Removed core bug demonstration scenarios #5489@skipOnOcis
,@notToImplementOnOcis
@grgprarup@notToImplementOnOcis
[full-ci] [tests-only] Removed scenarios tagged withnotToImplementOnOcis
tag #5491@skipOnOcis
[full-ci] [tests-only] Remove scenarios tagged skiponocis #5520Current skipped tags: (@amrita-shrestha https://github.com/owncloud/ocis/pull/6165)`
@skipOnGraph&&@skipOnOcis&&@notToImplementOnOCIS&&@toImplementOnOCIS&&comments-app-required&&@federation-app-required&&@notifications-app-required&&@skipOnOcis-OCIS-Storage&&systemtags-app-required&&@local_storage&&@caldav&&@carddav&&@api&&~@skip`@skipOnGraph@skipOnOcis-S3NG-Storage@skipOnStable3.0decided to not remove@provisioning_api-app-required@Skip@skipOnStorage:ceph@skipOnStorage:scality@skipOnOcis-OCIS-StorageTranslationHelper
,TagsHelper
,TagsContext
and other) @ScharfViktor I will remove it while implementing tag tests. [test-only] ApiTest. create tag #5391runOcc
method fromSetupHelper
#5691runOcc
method fromFeatureContext
#5824isTestingOnOcisOrReva
is used @grgprarup [full-ci] [tests-only] RemoveisTestingOnOcisOrReva
method #5705isTestingOnOcis
is used @grgprarup [full-ci] [tests-only] RemoveisTestingOnOcis
method #5780isTestingOnOc10
is used @grgprarup [full-ci] [tests-only] RemoveisTestingOnOc10
method #5722CC @individual-it @phil-davis Please, add other necessary cleanup todos if any.
@issue-
tag as they are issues of core, and change@issue-ocis-
tag to@issue-
@grgprarup [full-ci] [tests-only] Removed core issue tag, refactored ocis-reva, ocis issue tag #5983@personalSpace
tag, we might not need to separate the outline with the/spaces
endpoint anymore @grgprarup [full-ci] [tests-only] Removed tag personalSpace, and refactored the example tables #6250@issue-product-
, need to clarify if we still need them, and if not maybe we can remove them [tests-only][full-ci] update issue-product tag to ocis issue tag #6708TEST_OCIS
env Remove deprecatedenv
while running the test and update testing documentation #7300 [docs-only]Remove deprecated env #7376dav_version
,dav-path-version
,dav-path
(not dav_path), @S-Panta [tests-only] [full-ci] Refactoring example table dav-version text #6697The text was updated successfully, but these errors were encountered: