Skip to content

Eliminate the workspace-wide #[ignore = "TODO: re-enable after test isolation issue"] annotations #750

Description

@nanaf6203-bit

Problem Statement

13+ #[ignore]s across staking/tests.rs, bridge/tests.rs, integration_bridge_oracle.rs, prediction-market/src/lib.rs:831, bridge/src/errors.rs:424, bridge_load_tests.rs:226. They all cite "test isolation issue".

Why it matters

Ignored tests mask real regressions — the source of any global-storage leak must be located and fixed.

Technical Context

Locate the shared Lazy/OnceCell/mutable statics that survive between tests. Use ink::env::test::reset() or rebuild test fixtures per test.

Expected Outcome

Zero #[ignore = "test isolation issue"] annotations remain; CI passes both individual and full-workspace cleanly.

Acceptance Criteria

  1. All ignored tests re-enabled.
  2. cargo test --workspace deterministic.
  3. New tests/test_isolation_check.rs enforces that.
  4. Test-runtime reduced to < 5 min total.
  5. CI diff: pre- and post- run logs identical.

Implementation Notes

Identify offending statics; replace with per-test fixtures.

Files or modules likely to be affected

contracts/staking/src/tests.rs, contracts/bridge/src/tests.rs, tests/bridge_load_tests.rs, plus several others.

Dependencies

None.

Difficulty level

HARD.

Estimated effort

L (~1 engineer-week).

Metadata

Metadata

Assignees

Labels

High ComplexityRustStellar WaveIssues in the Stellar wave programbugSomething isn't workinghelp wantedExtra attention is neededtestingtesting related issues

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions