Skip to content

Conversation

@atharvalade
Copy link
Contributor

Remove all DependsOn attributes from 11 test files. Each test now creates its own resources with unique names via Guid.NewGuid(), ensuring full isolation and eliminating ordering dependencies.

Delete 16 unused fixture, helper, and model files that were only needed by the old DependsOn-based test structure.

Which issue does this PR close?

Closes #2654

Rationale

The DependsOn attribute creates ordering dependencies between tests, which leads to flaky failures and makes it hard to run tests in isolation. Independent tests are more reliable and easier to debug.

What changed?

Tests relied on DependsOn to run in a fixed sequence, sharing resources created by earlier tests. If one test failed, all downstream tests would skip or fail regardless of their own correctness.

Each test now creates its own stream, topic, and other resources with unique Guid-based names, then cleans up or simply leaves them isolated. The 16 fixture, helper, and model files that only existed to support the old shared-state pattern were removed as dead code.

Local Execution

  • Passed
  • Pre-commit hooks not ran (non-Rust changes only, prek targets Rust code)

AI Usage

  1. Opus 4.6
  2. Used for repetitive boilerplate and verifying no dangling references after deletions
  3. Verified by building locally (0 warnings, 0 errors) and running dotnet format
  4. Yes

…he#2654)

Remove all DependsOn attributes from 11 test files. Each test now
creates its own resources with unique names via Guid.NewGuid(),
ensuring full isolation and eliminating ordering dependencies.

Delete 16 unused fixture, helper, and model files that were only
needed by the old DependsOn-based test structure.
@atharvalade atharvalade force-pushed the refactor/csharp-rewrite-integration-tests branch from 9fe5903 to b2e7673 Compare February 10, 2026 03:19
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.

Rewrite .net integration tests

1 participant