Skip to content

Conversation

@saw-jan
Copy link
Contributor

@saw-jan saw-jan commented Dec 10, 2025

Description

Implementations:

  • check the uploaded file content in Given steps
  • add new step to check the file content with retry until timeout
  • wait for post-processing (425) to finish after performing successful dav requests (PUT, MOVE, COPY) if post-processing delay not enabled
  • test the access token, clear the store and refetch the token if got 401

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:

Screenshots (if appropriate):

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 added

@saw-jan saw-jan self-assigned this Dec 10, 2025
@saw-jan saw-jan force-pushed the test/check-uploaded-content branch 5 times, most recently from 4df751d to 3ce7fcc Compare December 11, 2025 10:55
@saw-jan saw-jan marked this pull request as ready for review December 11, 2025 11:35
@saw-jan saw-jan force-pushed the test/check-uploaded-content branch from 3ce7fcc to e15faaf Compare December 11, 2025 11:35
@saw-jan saw-jan requested a review from ScharfViktor December 11, 2025 11:36
@saw-jan saw-jan force-pushed the test/check-uploaded-content branch 5 times, most recently from a97e2ef to d30f105 Compare December 15, 2025 05:37
@saw-jan saw-jan changed the title [full-ci][tests-only] test: check content after upload [full-ci][tests-only] test: fix some test flakiness Dec 15, 2025
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

test: check content with retry

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

test: check body before json decoding

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

test: add wait step

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
@saw-jan saw-jan force-pushed the test/check-uploaded-content branch from a471c35 to 3d77cab Compare December 15, 2025 08:59
@saw-jan saw-jan merged commit cffeb4a into main Dec 15, 2025
1 of 59 checks passed
@saw-jan saw-jan deleted the test/check-uploaded-content branch December 15, 2025 09:00
@openclouders openclouders mentioned this pull request Dec 14, 2025
1 task
saw-jan added a commit that referenced this pull request Dec 16, 2025
* test: check content after upload

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

test: check content with retry

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: check empty body before json decoding

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: wait post-processing for webdav requests if applicable

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: check token before doing request

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

test: check body before json decoding

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

test: add wait step

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

---------

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
saw-jan added a commit that referenced this pull request Dec 16, 2025
* test: check content after upload

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

test: check content with retry

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: check empty body before json decoding

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: wait post-processing for webdav requests if applicable

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: check token before doing request

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

test: check body before json decoding

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

test: add wait step

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

---------

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants