Skip to content

test(cells): add S3 transfer e2e coverage [WPB-26729]#5048

Draft
MohamadJaara wants to merge 2 commits into
developfrom
mo/test-cells-s3-transfers
Draft

test(cells): add S3 transfer e2e coverage [WPB-26729]#5048
MohamadJaara wants to merge 2 commits into
developfrom
mo/test-cells-s3-transfers

Conversation

@MohamadJaara

@MohamadJaara MohamadJaara commented Jul 10, 2026

Copy link
Copy Markdown
Member

https://wearezeta.atlassian.net/browse/WPB-26729

Goal

Add focused Android E2E coverage for the Kalium Cells S3 transport refactor without exercising the full Shared Drive feature set.

Changes

  • provision Cells-enabled teams and conversations in the E2E backend setup
  • verify a regular 2 MiB upload and receiver-side download with exact size and SHA-256 checks
  • verify the multipart boundary with a 100 MiB + 1 byte file
  • cover direct-download cancellation, partial-file cleanup, retry, and final checksum verification
  • reuse and extend the existing UI Automator page objects, DocumentsUI flow, and test-support file helpers

@MohamadJaara
MohamadJaara marked this pull request as ready for review July 13, 2026 10:38
@MohamadJaara MohamadJaara changed the title test(cells): add S3 transfer e2e coverage test(cells): add S3 transfer e2e coverage [WPB-26729] Jul 13, 2026
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Jul 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.40%. Comparing base (ba5c450) to head (43d0900).
⚠️ Report is 8 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #5048      +/-   ##
===========================================
- Coverage    49.52%   49.40%   -0.12%     
===========================================
  Files          651      653       +2     
  Lines        23458    23598     +140     
  Branches      3602     3625      +23     
===========================================
+ Hits         11617    11659      +42     
- Misses       10767    10861      +94     
- Partials      1074     1078       +4     

see 10 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba5c450...43d0900. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MohamadJaara
MohamadJaara marked this pull request as draft July 14, 2026 15:46
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.

2 participants