Skip to content

Conversation

@jinliu9508
Copy link
Contributor

@jinliu9508 jinliu9508 commented Jan 26, 2026

Reverts #2524

Running into runtime issue with the CoroutineDispatcher being unable to be resolved by the ServiceProvider.


This change is Reviewable

Copy link
Contributor

@abdulraqeeb33 abdulraqeeb33 left a comment

Choose a reason for hiding this comment

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

Thank you for reverting

@jinliu9508 jinliu9508 merged commit 8a1ff87 into main Jan 26, 2026
3 of 7 checks passed
@jinliu9508 jinliu9508 deleted the revert-2524-chore/standard-test-pattern branch January 26, 2026 22:31
@github-actions
Copy link
Contributor

📊 Diff Coverage Report

Diff Coverage Report (Changed Lines Only)

Threshold: 80%

Changed Files Coverage

  • ⚠️ CoroutineDispatcherProvider.kt: Not in coverage report (may not be compiled/tested)
  • ⚠️ DefaultDispatcherProvider.kt: Not in coverage report (may not be compiled/tested)
  • StartupService.kt: 1/1 changed lines (100.0%) (3 changed lines)
  • OutcomeEventsRepository.kt: 0/6 changed lines (0.0%) (7 changed lines)
    • ⚠️ Below threshold: 6 uncovered changed lines
  • NotificationRepository.kt: 0/14 changed lines (0.0%) (15 changed lines)
    • ⚠️ Below threshold: 14 uncovered changed lines
  • ⚠️ TestDispatcherProvider.kt: Not in coverage report (may not be compiled/tested)

Overall Coverage (Changed Lines Only)

1/21 changed lines covered (4.8%)

❌ Coverage Check Failed

Files below 80% threshold:

  • OutcomeEventsRepository.kt: 0.0% (6 uncovered changed lines)

  • NotificationRepository.kt: 0.0% (14 uncovered changed lines)

📥 View workflow run

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.

3 participants