Skip to content

test: Port NIOThreadPoolTest to Swift Testing.#3490

Merged
Lukasa merged 1 commit intoapple:mainfrom
PassiveLogic:test/portNIOThreadPoolTestsToSwiftTesting
Feb 6, 2026
Merged

test: Port NIOThreadPoolTest to Swift Testing.#3490
Lukasa merged 1 commit intoapple:mainfrom
PassiveLogic:test/portNIOThreadPoolTestsToSwiftTesting

Conversation

@scottmarchant
Copy link
Copy Markdown
Contributor

Ports NIOThreadPoolTest in NIOPosix from XCTest to Swift Testing.

Motivation:

Tests should use the latest Swift Testing framework when possible.

Modifications:

  • Ported NIOThreadPoolTest from XCTest to Swift Testing
  • Updated test implementation from Dispatch to Swift Concurrency as needed to support test functionality

Result:

All tests continue to pass, NIOThreadPoolTest now has more modern Swift Testing syntax.

@scottmarchant scottmarchant marked this pull request as ready for review January 22, 2026 21:41
@scottmarchant scottmarchant force-pushed the test/portNIOThreadPoolTestsToSwiftTesting branch from f6ef527 to 6b49f2c Compare January 23, 2026 16:56
@scottmarchant scottmarchant force-pushed the test/portNIOThreadPoolTestsToSwiftTesting branch from 6b49f2c to e2257cc Compare February 3, 2026 17:42
@scottmarchant scottmarchant force-pushed the test/portNIOThreadPoolTestsToSwiftTesting branch from e2257cc to 50ed9d4 Compare February 6, 2026 00:00
@Lukasa Lukasa added the semver/none No version bump required. label Feb 6, 2026
@Lukasa Lukasa enabled auto-merge (squash) February 6, 2026 14:57
@Lukasa Lukasa merged commit 90ed88b into apple:main Feb 6, 2026
56 checks passed
@scottmarchant scottmarchant deleted the test/portNIOThreadPoolTestsToSwiftTesting branch February 6, 2026 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants