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] [full-ci] Bump commit id for API tests #3794

Merged
merged 2 commits into from
May 16, 2022
Merged

[tests-only] [full-ci] Bump commit id for API tests #3794

merged 2 commits into from
May 16, 2022

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented May 13, 2022 •
edited
Loading

@phil-davis phil-davis self-assigned this May 13, 2022
@phil-davis phil-davis force-pushed the bump-commit-id-20220513 branch 3 times, most recently from 646d0ba to ee6a703 Compare May 13, 2022 15:40
@phil-davis
Copy link
Contributor Author

https://drone.owncloud.com/owncloud/ocis/11822/52/9

runsh: Total unexpected passed scenarios throughout the test run:
webUIFilesSearch/search.feature:153
webUIFilesSearch/search.feature:165

Unexpected passes - how did that happen?

I will rebase again.

@phil-davis
Copy link
Contributor Author

phil-davis commented May 16, 2022 •
edited
Loading

https://drone.owncloud.com/owncloud/ocis/11825/52/9

runsh: There were no unexpected failures.
runsh: Total unexpected passed scenarios throughout the test run:
webUIFilesSearch/search.feature:153
webUIFilesSearch/search.feature:165

Same scenarios passed again. I removed them from expected-failures.

The scenarios are about searching in favorites and trashbin. Maybe they now pass because of reva-oCIS changes/fixes related to #1330 and/or PR #3797 which merged in the latest web 5.5.0-rc.6 without running full-ci - probably the tests started passing there, but we did not know.

@sonarcloud
Copy link

sonarcloud bot commented May 16, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@phil-davis phil-davis marked this pull request as ready for review May 16, 2022 05:39
@phil-davis phil-davis merged commit ae4ec53 into master May 16, 2022
@delete-merged-branch delete-merged-branch bot deleted the bump-commit-id-20220513 branch May 16, 2022 06:12
ownclouders pushed a commit that referenced this pull request May 16, 2022
@phil-davis
commit ae4ec53
e9bc1ea 
Merge: 7b95b40 a033767
Author: Phil Davis <phil@jankaritech.com>
Date:   Mon May 16 11:57:48 2022 +0545

    Merge pull request #3794 from owncloud/bump-commit-id-20220513

    [tests-only] [full-ci] Bump commit id for API tests
@phil-davis phil-davis mentioned this pull request May 16, 2022
Closed
19 tasks
@SwikritiT SwikritiT mentioned this pull request May 19, 2022
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@individual-it individual-it individual-it approved these changes

Assignees

@phil-davis phil-davis

Labels
QA:team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@phil-davis @individual-it