Skip to content

Refactor: Migrate conformance tests to StorageTransport and enhance retry logic#7917

Draft
thiyaguk09 wants to merge 23 commits intogoogleapis:storage-node-18from
thiyaguk09:node18/conformance-test
Draft

Refactor: Migrate conformance tests to StorageTransport and enhance retry logic#7917
thiyaguk09 wants to merge 23 commits intogoogleapis:storage-node-18from
thiyaguk09:node18/conformance-test

Conversation

@thiyaguk09
Copy link
Copy Markdown
Contributor

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

All 720 test cases have been fixed, and the code has been refactored.
- Replace high-level bucket and file calls with
storageTransport.makeRequest.
- Fix Scenario 1 failures by implementing "create-or-get" logic for
buckets.
- Resolve metageneration mismatch in lock() by dynamically fetching
metadata.
- Normalize header keys to lowercase in transport response processing.
- Increase unit test coverage for shouldRetry logic and error handling.
- Fixed authentication headers/token exchange in the transport layer.
- Reverted to single-shot resumable upload to isolate Scenario 7
  failures while debugging mid-stream offset recovery.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant