Skip to content

Show instrumentation test progress in Semaphore logs - #3288

Merged
sosnovsky merged 2 commits into
masterfrom
issue_3287_test_names
Sep 21, 2026
Merged

sosnovsky merged 2 commits into
masterfrom
issue_3287_test_names

Conversation

@DenBond7

@DenBond7 DenBond7 commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Instrumentation test jobs previously redirected TestRunner output exclusively to an artifact, leaving the live Semaphore log with mostly emulator messages and no indication of which test was running.

This change:

  • Prints STARTED, PASSED, FAILED, and SKIPPED events with test durations.
  • Preserves the original TestRunner output in logcat_log.txt.
  • Prints a final summary with passed, failed, skipped, and incomplete test counts.
  • Lists failed tests with GitHub links to the corresponding source file and test method at the tested commit.
  • Waits for log processing to finish before publishing artifacts.
  • Preserves the original instrumentation test command exit code.

Example output:

[TEST] WkdClientTest#existingEmailFlowCryptDomainTest STARTED
[TEST] WkdClientTest#existingEmailFlowCryptDomainTest FAILED (12s)

[TEST] SUMMARY: 48 total, 46 passed, 2 failed, 0 skipped (1542s)

[TEST] FAILED TESTS
[TEST] - WkdClientTest#existingEmailFlowCryptDomainTest:
[https://github.com/FlowCrypt/flowcrypt-android/blob/](https://github.com/FlowCrypt/flowcrypt-android/blob/)<commit>/FlowCrypt/src/androidTest/java/com/flowcrypt/email/WkdClientTest.kt#L86

close #3287


Tests (delete all except exactly one):

  • Does not need tests (refactor only, docs or internal changes)

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities

@DenBond7
DenBond7 marked this pull request as ready for review September 21, 2026 17:50
@DenBond7
DenBond7 requested a review from sosnovsky as a code owner September 21, 2026 17:50

@sosnovsky sosnovsky left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good 👍

@sosnovsky
sosnovsky merged commit 0d34884 into master Sep 21, 2026
7 checks passed
@sosnovsky
sosnovsky deleted the issue_3287_test_names branch September 21, 2026 20:10
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.

Improve CI testing process

2 participants