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] [full-ci] Bump commit ids for tests #2278

Merged
merged 2 commits into from
Jul 12, 2021

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Jul 10, 2021

Description

  1. bump the commit ids for tests to the latest
  2. delete the expected-failures file for EOS. We don't test EOS in CI here in owncloud/ocis and the content of it gets out-of-date and useless.

How Has This Been Tested?

CI

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@phil-davis phil-davis self-assigned this Jul 10, 2021
@phil-davis
Copy link
Contributor Author

On owncloud storage: https://drone.owncloud.com/owncloud/ocis/5944/27/7

runsh: Total unexpected failed scenarios throughout the test run:
apiShareCreateSpecialToShares1/createShareReceivedInMultipleWays.feature:196
apiShareCreateSpecialToShares1/createShareReceivedInMultipleWays.feature:218
apiShareCreateSpecialToShares1/createShareReceivedInMultipleWays.feature:240
apiShareCreateSpecialToShares1/createShareReceivedInMultipleWays.feature:262
apiShareCreateSpecialToShares1/createShareReceivedInMultipleWays.feature:283
apiShareCreateSpecialToShares1/createShareReceivedInMultipleWays.feature:305
apiShareCreateSpecialToShares1/createShareReceivedInMultipleWays.feature:327
apiShareCreateSpecialToShares1/createShareReceivedInMultipleWays.feature:349
apiShareCreateSpecialToShares1/createShareReceivedInMultipleWays.feature:370
runsh: There were no unexpected success.

And the same on OCIS storage: https://drone.owncloud.com/owncloud/ocis/5944/28/6

Those are new scenarios. It needs someone to look at the fails and see if they fit with an existing issue, or if a new issue needs to be raised.

@phil-davis phil-davis removed their assignment Jul 12, 2021
@dpakach dpakach self-assigned this Jul 12, 2021
@phil-davis phil-davis marked this pull request as ready for review July 12, 2021 04:30
@dpakach dpakach force-pushed the bump-commit-ids-20210710 branch from 940d568 to cd30d2d Compare July 12, 2021 05:52
@saw-jan
Copy link
Member

saw-jan commented Jul 12, 2021

this PR can be merged. I will rebase my PR #2281 after it is merged

@sonarqubecloud
Copy link

Kudos, SonarCloud 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 merged commit 39d5e66 into master Jul 12, 2021
@delete-merged-branch delete-merged-branch bot deleted the bump-commit-ids-20210710 branch July 12, 2021 07:05
ownclouders pushed a commit that referenced this pull request Jul 12, 2021
Merge: c12d438 cd30d2d
Author: Phil Davis <[email protected]>
Date:   Mon Jul 12 12:50:31 2021 +0545

    Merge pull request #2278 from owncloud/bump-commit-ids-20210710

    [tests-only] [full-ci] Bump commit ids for tests
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