Skip to content

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

Open
MohamadJaara wants to merge 4 commits into
developfrom
mo/test-cells-s3-transfers
Open

test(cells): add S3 transfer e2e coverage [WPB-26729]#5048
MohamadJaara wants to merge 4 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.45%. Comparing base (9e50c4b) to head (23f213e).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #5048   +/-   ##
========================================
  Coverage    49.45%   49.45%           
========================================
  Files          657      657           
  Lines        23960    23960           
  Branches      3731     3731           
========================================
  Hits         11849    11849           
  Misses       10982    10982           
  Partials      1129     1129           

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 9e50c4b...23f213e. 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
@MohamadJaara
MohamadJaara force-pushed the mo/test-cells-s3-transfers branch from 43d0900 to e07a3a7 Compare July 20, 2026 16:47
@MohamadJaara
MohamadJaara marked this pull request as ready for review July 20, 2026 16:58
@MohamadJaara
MohamadJaara force-pushed the mo/test-cells-s3-transfers branch from 3b42f41 to 339b972 Compare July 20, 2026 17:43
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.

1 participant