Skip to content

Conversation

@hubcio
Copy link
Contributor

@hubcio hubcio commented Feb 4, 2026

TestServer, ClientFactory, and protocol-specific client factories
duplicated functionality now provided by TestHarness and the
iggy_harness proc macro.

Changes:

  • Delete ~2000 lines of deprecated test utilities
  • Migrate all tests to unified TestHarness API
  • Move TLS cert generation into ServerHandle
  • Add port binding verification on server startup
  • Update BDD, CLI, and server scenario tests

@hubcio hubcio force-pushed the harness-final branch 2 times, most recently from 254af10 to d2f7280 Compare February 4, 2026 16:42
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.51%. Comparing base (d98176a) to head (178d278).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2678      +/-   ##
==========================================
- Coverage   69.80%   69.51%   -0.30%     
==========================================
  Files         568      568              
  Lines       55238    55238              
  Branches    55238    55238              
==========================================
- Hits        38560    38398     -162     
- Misses      14799    14961     +162     
  Partials     1879     1879              
Flag Coverage Δ
rust 69.51% <100.00%> (-0.30%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...ore/server/src/websocket/websocket_tls_listener.rs 70.34% <100.00%> (+2.06%) ⬆️

... and 16 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

numinnex
numinnex previously approved these changes Feb 5, 2026
mmodzelewski
mmodzelewski previously approved these changes Feb 5, 2026
TestServer, ClientFactory, and protocol-specific client factories
duplicated functionality now provided by TestHarness and the
iggy_harness proc macro.

Changes:
- Delete ~2000 lines of deprecated test utilities
- Migrate all tests to unified TestHarness API
- Move TLS cert generation into ServerHandle
- Add port binding verification on server startup
- Update BDD, CLI, and server scenario tests
@hubcio hubcio dismissed stale reviews from mmodzelewski and numinnex via 178d278 February 5, 2026 14:41
@hubcio hubcio merged commit 4052619 into master Feb 5, 2026
58 checks passed
@hubcio hubcio deleted the harness-final branch February 5, 2026 15:11
krishvishal pushed a commit to krishvishal/iggy that referenced this pull request Feb 9, 2026
TestServer, ClientFactory, and protocol-specific client factories
duplicated functionality now provided by TestHarness and the
iggy_harness proc macro.

Changes:
- Delete ~2000 lines of deprecated test utilities
- Migrate all tests to unified TestHarness API
- Move TLS cert generation into ServerHandle
- Add port binding verification on server startup
- Update BDD, CLI, and server scenario tests
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.

4 participants