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] Combine coreApi and apiX suites #10497

Open
amrita-shrestha opened this issue Nov 7, 2024 · 1 comment
Open

[QA] Combine coreApi and apiX suites #10497

amrita-shrestha opened this issue Nov 7, 2024 · 1 comment
Labels

Comments

@amrita-shrestha
Copy link
Contributor

Describe the bug

Our API test folder includes two types of test suites: those prefixed with coreApiX, which contain transfer API tests migrated from the core repository and those prefixed with apiX, which include new tests specifically designed for oCIS. At present, there appears to be no need to differentiate these test suites by naming them coreApiX and apiX.

the word `coreApi` deceives us, tests in ocis both `apiX` and `coreApiX` are for the ocis server and not for the oc10. So, I don't see a reason not to add feature file there.

Originally posted by @saw-jan in #10170 (comment)

Considering the above comment, this issue has been created to merge the coreApi and api test suites that cover similar feature tests.

@saw-jan
Copy link
Member

saw-jan commented Nov 8, 2024

We should also think about the tests for reva-edge. How should we identify tests that run in reva? use skip tags to skip all apiX test suites (which we currently don't run in reva)?

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

2 participants