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 CORE_COMMITID for API tests #3976

Merged
merged 1 commit into from
Jun 17, 2022
Merged

[tests-only] [full-ci] Bump CORE_COMMITID for API tests #3976

merged 1 commit into from
Jun 17, 2022

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Jun 16, 2022 •
edited
Loading

Description

Includes new API test scenarios related to core oC10 issue owncloud/core#40140

Note: those are test scenarios related to quota. Those are currently skipped on oCIS/reva test pipelines, because with the graphAPI for users/groups/spaces the quota concept will work differently. So actually the new test scenarios are not run in this CI, so we do not get any new results or unexpected failures.

Related Issue

owncloud/QA#744

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 Jun 16, 2022
@phil-davis phil-davis force-pushed the bump-core-commit-id-20220616 branch from 31cb260 to d456aff Compare June 17, 2022 02:54
@phil-davis phil-davis mentioned this pull request Jun 17, 2022
Closed
14 tasks
@sonarcloud
Copy link

sonarcloud bot commented Jun 17, 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
Copy link
Contributor Author

@kiranparajuli589 @grgprarup I started this commit id bump last night, because I added core API tests and wanted to see what happened to them. But now I realise that they do not run on oCIS-graphAPI anyway,

Please approve and merge. And get someone else to do the other commit id bumps for reva, web...

@kiranparajuli589 kiranparajuli589 merged commit 27b7190 into master Jun 17, 2022
@delete-merged-branch delete-merged-branch bot deleted the bump-core-commit-id-20220616 branch June 17, 2022 04:53
ownclouders pushed a commit that referenced this pull request Jun 17, 2022
@kiranparajuli589
commit 27b7190
98c25fb 
Merge: 945f20f d456aff
Author: PKiran <39373750+kiranparajuli589@users.noreply.github.com>
Date:   Fri Jun 17 10:38:24 2022 +0545

    Merge pull request #3976 from owncloud/bump-core-commit-id-20220616

    [tests-only] [full-ci] Bump CORE_COMMITID for API tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kiranparajuli589 kiranparajuli589 kiranparajuli589 approved these changes

@grgprarup grgprarup Awaiting requested review from grgprarup

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 @kiranparajuli589