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

[tests-only]tests: skip tests in reva #7955

Merged
merged 1 commit into from
Dec 14, 2023
Merged

Conversation

SwikritiT
Copy link
Contributor

@SwikritiT SwikritiT commented Dec 13, 2023

these tests were added recently as a part of the issue #7823 but these tests aren't how the Reva is expected to behave so skip these tests in Reva.

This will unblock the bump commit id PR in Reva cs3org/reva#4399

@SwikritiT SwikritiT self-assigned this Dec 13, 2023
@SwikritiT SwikritiT force-pushed the tests/fix-test-in-reva branch from 5327760 to 43f25f2 Compare December 14, 2023 09:23
@SwikritiT SwikritiT marked this pull request as ready for review December 14, 2023 09:36
@SwikritiT SwikritiT requested a review from saw-jan December 14, 2023 09:54
@SwikritiT SwikritiT force-pushed the tests/fix-test-in-reva branch from 43f25f2 to 942d75c Compare December 14, 2023 11:07
@SwikritiT SwikritiT changed the title [tests-only]tests: fix tests for reva [tests-only]tests: skip tests in reva Dec 14, 2023
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@saw-jan saw-jan merged commit 1e94245 into master Dec 14, 2023
1 check passed
@delete-merged-branch delete-merged-branch bot deleted the tests/fix-test-in-reva branch December 14, 2023 16:03
ownclouders pushed a commit that referenced this pull request Dec 14, 2023
ownclouders pushed a commit that referenced this pull request Dec 15, 2023
ownclouders pushed a commit that referenced this pull request Dec 15, 2023
2403905 pushed a commit to 2403905/ocis that referenced this pull request Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants