Skip to content

CI Test: Run e2e cron tests on Android 16 and 17#6575

Merged
andremion merged 2 commits into
developfrom
tests/rahullohra/android-17-e2e-cron
Jul 20, 2026
Merged

CI Test: Run e2e cron tests on Android 16 and 17#6575
andremion merged 2 commits into
developfrom
tests/rahullohra/android-17-e2e-cron

Conversation

@rahul-lohra

@rahul-lohra rahul-lohra commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Goal

Closes #AND-1315
Run E2E tests on Android 16 and Android 17

Implementation

Run E2E tests on Android 16 and Android 17

🎨 UI Changes

None

Testing

None

Summary by CodeRabbit

  • Tests
    • Expanded nightly end-to-end testing to include Android API levels 36 and 37.

@rahul-lohra rahul-lohra self-assigned this Jul 20, 2026
@rahul-lohra rahul-lohra added the pr:ci CI changes label Jul 20, 2026
@rahul-lohra
rahul-lohra requested a review from a team as a code owner July 20, 2026 07:02
@github-actions

Copy link
Copy Markdown
Contributor

PR checklist ✅

All required conditions are satisfied:

  • Title length is OK (or ignored by label).
  • At least one pr: label exists.
  • Sections ### Goal, ### Implementation, and ### Testing are filled, or the PR is bot-authored.
  • An issue is linked (Linear ticket or GitHub issue), or the PR is bot-authored.

🎉 Great job! This PR is ready for review.

@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The nightly E2E workflow matrix now includes Android API levels 37 and 36, expanding scheduled emulator test runs.

Changes

Nightly E2E coverage

Layer / File(s) Summary
Extend Android API matrix
.github/workflows/e2e-test-cron.yml
Adds Android API levels 37 and 36 to the nightly Compose test matrix.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: peter-matkovski

Poem

A rabbit hops through nightly runs,
Adding API moons, 37 and 36 suns.
More emulators wake and test,
While carrots cheer the matrix quest.
Hop, hop—coverage grows best!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title accurately summarizes the main change: adding nightly E2E cron tests for Android API levels 36 and 37.
Description check ✅ Passed The description includes the required Goal, Implementation, UI Changes, and Testing sections, and it matches the PR objective.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch tests/rahullohra/android-17-e2e-cron

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

SDK Size Comparison 📏

SDK Before After Difference Status
stream-chat-android-client 5.95 MB 5.95 MB 0.00 MB 🟢
stream-chat-android-ui-components 11.21 MB 11.21 MB 0.00 MB 🟢
stream-chat-android-compose 12.69 MB 12.69 MB 0.00 MB 🟢

@rahul-lohra rahul-lohra changed the title [AND-1315] test: Run e2e cron tests on Android 16 and 17 CI Test: Run e2e cron tests on Android 16 and 17 Jul 20, 2026
@coderabbitai coderabbitai Bot mentioned this pull request Jul 20, 2026
@sonarqubecloud

Copy link
Copy Markdown

@andremion
andremion enabled auto-merge (squash) July 20, 2026 08:04
@andremion
andremion merged commit f4e7c55 into develop Jul 20, 2026
20 checks passed
@andremion
andremion deleted the tests/rahullohra/android-17-e2e-cron branch July 20, 2026 08:04
andremion added a commit that referenced this pull request Jul 20, 2026
@andremion andremion added the pr:ignore-for-release Exclude from release notes label Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:ci CI changes pr:ignore-for-release Exclude from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants