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 core API tests #4295

Merged
merged 2 commits into from
Jul 28, 2022
Merged

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

merged 2 commits into from
Jul 28, 2022

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Jul 28, 2022 •
edited
Loading

Description

Brings in core PR owncloud/core#40247 which removes an unwanted scenario from oCIS CI

Related Issue

Part of issue owncloud/QA#753

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 28, 2022
@phil-davis phil-davis force-pushed the bump-commit-id-202207228 branch from 67997b5 to 5324bce Compare July 28, 2022 07:36
@phil-davis phil-davis marked this pull request as ready for review July 28, 2022 07:37
@phil-davis phil-davis requested review from kiranparajuli589, SagarGi, individual-it and SwikritiT and removed request for kiranparajuli589 July 28, 2022 07:37
This was referenced Jul 28, 2022
Closed
Closed
SwikritiT
SwikritiT approved these changes Jul 28, 2022
View reviewed changes
Copy link
Contributor

@SwikritiT SwikritiT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm if Ci passes

@phil-davis
Copy link
Contributor Author

phil-davis commented Jul 28, 2022 •
edited
Loading

https://drone.owncloud.com/owncloud/ocis/13884/77/15
bash: /drone/src/oc10/testrunner/tests/acceptance/run.sh: No such file or directory

That happened a few days ago, and has come back.

I will push a commit to skip the parallel deployment pipelines for now.

Issue #4296 created to look into the problem.

@phil-davis phil-davis mentioned this pull request Jul 28, 2022
Closed
@phil-davis
Copy link
Contributor Author

phil-davis commented Jul 28, 2022 •
edited
Loading

https://drone.owncloud.com/owncloud/ocis/13886/42/7

runsh: Total unexpected failed scenarios throughout the test run:
apiSpaces/search.feature:20

 Scenario: Alice can find data from the project space                  # /drone/src/tests/acceptance/features/apiSpaces/search.feature:20
    When user "Alice" searches for "fol" using the WebDAV API           # SearchContext::userSearchesUsingWebDavAPI()
    Then the HTTP status code should be "207"                           # FeatureContext::thenTheHTTPStatusCodeShouldBe()
      HTTP status code 500 is not the expected value 207
      Failed asserting that 500 matches expected '207'.
    And the search result should contain "4" entries                    # FeatureContext::propfindResultShouldContainNumEntries()
    And the search result of user "Alice" should contain these entries: # FeatureContext::thePropfindResultShouldContainEntries()
      | /folder                                           |
      | /folder/SubFolder1                                |
      | /folder/SubFolder1/subFOLDER2                     |
      | /folder/SubFolder1/subFOLDER2/insideTheFolder.txt |

Looks like a flaky test fail in the local API tests. I will make an issue - #4297

I restarted CI.

@sonarqubecloud
Copy link

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 merged commit 1b75f2d into master Jul 28, 2022
ownclouders pushed a commit that referenced this pull request Jul 28, 2022
@phil-davis
commit 1b75f2d
87e5464 
Merge: 0b442f1 f8e5b59
Author: Phil Davis <[email protected]>
Date:   Thu Jul 28 14:58:11 2022 +0545

    Merge pull request #4295 from owncloud/bump-commit-id-202207228

    [tests-only] [full-ci] Bump commit id for core API tests
@phil-davis phil-davis deleted the bump-commit-id-202207228 branch July 28, 2022 09:31
ownclouders pushed a commit that referenced this pull request Jul 29, 2022
@phil-davis
commit 1b75f2d
37875b4 
Merge: 0b442f1 f8e5b59
Author: Phil Davis <[email protected]>
Date:   Thu Jul 28 14:58:11 2022 +0545

    Merge pull request #4295 from owncloud/bump-commit-id-202207228

    [tests-only] [full-ci] Bump commit id for core API tests
ownclouders pushed a commit that referenced this pull request Jul 29, 2022
@phil-davis
commit 1b75f2d
2a33f60 
Merge: 0b442f1 f8e5b59
Author: Phil Davis <[email protected]>
Date:   Thu Jul 28 14:58:11 2022 +0545

    Merge pull request #4295 from owncloud/bump-commit-id-202207228

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

@SwikritiT SwikritiT SwikritiT approved these changes

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

@SagarGi SagarGi Awaiting requested review from SagarGi

@kiranparajuli589 kiranparajuli589 Awaiting requested review from kiranparajuli589

@wkloucek wkloucek Awaiting requested review from wkloucek

@micbar micbar Awaiting requested review from micbar

Assignees

@phil-davis phil-davis

Labels
QA:team
Projects
None yet
Milestone
No milestone
Development

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