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.

Sign up for GitHub

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

Jump to bottom

[tests-only] Bump core commit id for API tests #1353

Merged
merged 3 commits into from
Dec 2, 2020
Merged

[tests-only] Bump core commit id for API tests #1353

merged 3 commits into from
Dec 2, 2020

Conversation

swoichha
Copy link
Contributor

@swoichha swoichha commented Dec 2, 2020

@swoichha swoichha requested a review from labkode as a code owner December 2, 2020 04:53
@phil-davis
Copy link
Contributor

phil-davis commented Dec 2, 2020 •
edited
Loading

@swoichha needs some updating of expected-failures files.

runsh: Total unexpected failed scenarios throughout the test run:
apiWebdavUploadTUS/lowLevelCreationExtension.feature:19
apiWebdavUploadTUS/lowLevelCreationExtension.feature:20
apiWebdavUploadTUS/uploadToShare.feature:66
apiWebdavUploadTUS/uploadToShare.feature:67

Also, maybe there is some settings change that can be made so that lowLevelCreationExtension.feature will pass? Those test scenarios are passing in owncloud/ocis

@individual-it individual-it mentioned this pull request Dec 2, 2020
Merged
11 tasks
@swoichha
Copy link
Contributor Author

swoichha commented Dec 2, 2020

@swoichha needs some updating of expected-failures files.

runsh: Total unexpected failed scenarios throughout the test run:
apiWebdavUploadTUS/lowLevelCreationExtension.feature:19
apiWebdavUploadTUS/lowLevelCreationExtension.feature:20
apiWebdavUploadTUS/uploadToShare.feature:66
apiWebdavUploadTUS/uploadToShare.feature:67

Also, maybe there is some settings change that can be made so that lowLevelCreationExtension.feature will pass? Those test scenarios are passing in owncloud/ocis

I have updated the expected-failure file and owncloud/core#38180 must fix the problem with lowLevelCreationExtension.feature

@swoichha swoichha force-pushed the bump-core-commit-id-uploadToMove-tusTests branch from f0f71bd to b8a84b0 Compare December 2, 2020 09:23
@phil-davis phil-davis requested a review from ishank011 December 2, 2020 09:51
@ishank011 ishank011 merged commit d8c6290 into cs3org:master Dec 2, 2020
@phil-davis phil-davis mentioned this pull request Dec 3, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phil-davis phil-davis phil-davis approved these changes

@ishank011 ishank011 ishank011 approved these changes

@labkode labkode Awaiting requested review from labkode

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@swoichha @phil-davis @ishank011