Skip to content

Conversation

@Oscar-Pepper
Copy link
Contributor

@Oscar-Pepper Oscar-Pepper commented Oct 31, 2025

Motivation

Chain index integration was held up by flaky and slow tests and increased failure rates due to underlying issues with test framework interacting with chain index. Fixing the test framework to use correct sleeping and to poll when generating blokcs instead of using fixed delays not only improves dev efficiency but fixes many failing tests in the chain integration (#596)

Solution

  • remove OS sleeps
  • replace fixed delays with polling
  • return service subscriber in the test manager to reduce backend duplication and out-of-sync bugs

Tests

Specifications & References

Follow-up Work

PR Checklist

  • The PR name is suitable for the release notes.
  • The solution is tested.
  • The documentation is up to date.

@Oscar-Pepper Oscar-Pepper added the Top Priority Current objectives and issues label Oct 31, 2025
@dorianvp
Copy link
Member

dorianvp commented Nov 3, 2025

Some issues we're seeing here are highly related to #620, and #610.

@Oscar-Pepper Oscar-Pepper marked this pull request as ready for review November 6, 2025 10:15
@nachog00 nachog00 merged commit d917728 into zingolabs:dev Nov 6, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Top Priority Current objectives and issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants