Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ALFMOB-141: Optimize Unit Test Execution Time #37

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

JoaoPinhoMinder
Copy link
Contributor

@JoaoPinhoMinder JoaoPinhoMinder commented Feb 26, 2025

Ticket

https://mindera.atlassian.net/browse/ALFMOB-141

Description

Based on 10 test runs, i observed a significant reduction in execution times, now raging from 70.595s to 173.835s, with an average execution time of 109.860, about 63.631s faster than before.

To achieve this improvement, the follwoing optimizations were made:

  • Reduced timeouts in tests where no events where expected.
  • Optimized mocks to eliminate unnecessary delays.
  • Integrated CombineSchedulers to better control threading in tests
  • Fixed unit tests with false positives.
  • Fixed unit tests that occasionally resulted in false negatives.

@JoaoPinhoMinder JoaoPinhoMinder force-pushed the techdebt/ALFMOB-141-optimize-unit-test-execution-time branch 9 times, most recently from c6d31df to 94831f4 Compare February 28, 2025 14:56
Based on 10 test runs, i observed a **significant reduction** in execution times, now raging from **70.595s to 173.835s**, with an **average execution time of 109.860**, about **63.631s faster** than before.

To achieve this improvement, the follwoing optimizations were made:
- Reduced timeouts in tests where no events where expected.
- Optimized mocks to eliminate unnecessary delays.
- Integrated `CombineSchedulers` to better control threading in tests
- Fixed unit tests with false positives.
- Fixed unit tests that occasionally resulted in false negatives.
@JoaoPinhoMinder JoaoPinhoMinder force-pushed the techdebt/ALFMOB-141-optimize-unit-test-execution-time branch from fc5bf2d to 19438fd Compare March 3, 2025 13:05
@JoaoPinhoMinder JoaoPinhoMinder marked this pull request as ready for review March 3, 2025 13:06
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.

1 participant