diff --git a/tests/app_data.rs b/tests/app_data.rs index 62300eef..6601c615 100644 --- a/tests/app_data.rs +++ b/tests/app_data.rs @@ -12,35 +12,17 @@ use wireframe::extractor::{ SharedState, }; -#[expect( - clippy::allow_attributes, - reason = "rstest single-line fixtures need allow to avoid unfulfilled lint expectations" -)] -#[allow( - unfulfilled_lint_expectations, - reason = "rstest occasionally misses the expected lint for single-line fixtures on stable" -)] -#[expect( - unused_braces, - reason = "rustc false positive for single line rstest fixtures" -)] #[fixture] -fn request() -> MessageRequest { MessageRequest::default() } +fn request() -> MessageRequest { + // Default request for shared state extraction tests + MessageRequest::default() +} -#[expect( - clippy::allow_attributes, - reason = "rstest single-line fixtures need allow to avoid unfulfilled lint expectations" -)] -#[allow( - unfulfilled_lint_expectations, - reason = "rstest occasionally misses the expected lint for single-line fixtures on stable" -)] -#[expect( - unused_braces, - reason = "rustc false positive for single line rstest fixtures" -)] #[fixture] -fn empty_payload() -> Payload<'static> { Payload::default() } +fn empty_payload() -> Payload<'static> { + // Empty payload for shared state extraction tests + Payload::default() +} #[rstest] fn shared_state_extractor_returns_data( diff --git a/tests/connection_actor_errors.rs b/tests/connection_actor_errors.rs index b9c44393..2a605398 100644 --- a/tests/connection_actor_errors.rs +++ b/tests/connection_actor_errors.rs @@ -25,41 +25,21 @@ use wireframe_testing::{LoggerHandle, logger, push_expect}; mod common; use common::TestResult; -#[expect( - clippy::allow_attributes, - reason = "rstest single-line fixtures need allow to avoid unfulfilled lint expectations" -)] -#[allow( - unfulfilled_lint_expectations, - reason = "rstest occasionally misses the expected lint for single-line fixtures on stable" -)] -#[expect( - unused_braces, - reason = "rustc false positive for single line rstest fixtures" -)] #[fixture] fn queues() -> Result<(PushQueues, wireframe::push::PushHandle), wireframe::push::PushConfigError> { + // Push queues with default capacities for error propagation tests PushQueues::::builder() .high_capacity(8) .low_capacity(8) .build() } -#[expect( - clippy::allow_attributes, - reason = "rstest single-line fixtures need allow to avoid unfulfilled lint expectations" -)] -#[allow( - unfulfilled_lint_expectations, - reason = "rstest occasionally misses the expected lint for single-line fixtures on stable" -)] -#[expect( - unused_braces, - reason = "rustc false positive for single line rstest fixtures" -)] #[fixture] -fn shutdown_token() -> CancellationToken { CancellationToken::new() } +fn shutdown_token() -> CancellationToken { + // Shutdown token for connection actor tests + CancellationToken::new() +} #[rstest] #[tokio::test] diff --git a/tests/connection_actor_shutdown.rs b/tests/connection_actor_shutdown.rs index 80bde3d2..d117cd33 100644 --- a/tests/connection_actor_shutdown.rs +++ b/tests/connection_actor_shutdown.rs @@ -12,42 +12,20 @@ use wireframe_testing::push_expect; mod common; use common::TestResult; -// Apply expected lint suppressions for single-line rstest fixtures. -// Context: https://github.com/la10736/rstest/issues/222 -macro_rules! single_line_fixture { - ($item:item) => { - #[expect( - clippy::allow_attributes, - reason = "rstest single-line fixtures need allow to avoid unfulfilled lint \ - expectations" - )] - #[allow( - unfulfilled_lint_expectations, - reason = "rstest occasionally misses the expected lint for single-line fixtures on \ - stable" - )] - #[expect( - unused_braces, - reason = "rustc false positive for single line rstest fixtures" - )] - $item - }; -} - -single_line_fixture! { - #[fixture] - fn queues() - -> Result<(PushQueues, wireframe::push::PushHandle), wireframe::push::PushConfigError> { - PushQueues::::builder() - .high_capacity(8) - .low_capacity(8) - .build() - } +#[fixture] +fn queues() +-> Result<(PushQueues, wireframe::push::PushHandle), wireframe::push::PushConfigError> { + // Push queues with default capacities for shutdown tests + PushQueues::::builder() + .high_capacity(8) + .low_capacity(8) + .build() } -single_line_fixture! { - #[fixture] - fn shutdown_token() -> CancellationToken { CancellationToken::new() } +#[fixture] +fn shutdown_token() -> CancellationToken { + // Shutdown token for connection actor tests + CancellationToken::new() } #[rstest] diff --git a/tests/push_policies.rs b/tests/push_policies.rs index cdc6d548..90ca3c5e 100644 --- a/tests/push_policies.rs +++ b/tests/push_policies.rs @@ -15,20 +15,11 @@ use wireframe_testing::{LoggerHandle, logger}; mod common; use common::TestResult; -#[expect( - clippy::allow_attributes, - reason = "rstest single-line fixtures need allow to avoid unfulfilled lint expectations" -)] -#[allow( - unfulfilled_lint_expectations, - reason = "rstest occasionally misses the expected lint for single-line fixtures on stable" -)] -#[expect( - unused_braces, - reason = "rustc false positive for single-line rstest fixtures" -)] #[fixture] -fn builder() -> PushQueuesBuilder { support::builder::() } +fn builder() -> PushQueuesBuilder { + // PushQueuesBuilder for push queue policy tests + support::builder::() +} #[derive(Clone, Copy)] struct PolicyCase { diff --git a/wireframe_testing/src/logging.rs b/wireframe_testing/src/logging.rs index a98205c0..ac729769 100644 --- a/wireframe_testing/src/logging.rs +++ b/wireframe_testing/src/logging.rs @@ -74,9 +74,8 @@ impl std::ops::DerefMut for LoggerHandle { } /// rstest fixture returning a [`LoggerHandle`] for log assertions. -#[allow( - unused_braces, - reason = "rustc false positive for single line rstest fixtures" -)] #[fixture] -pub fn logger() -> LoggerHandle { LoggerHandle::new() } +pub fn logger() -> LoggerHandle { + // Acquire exclusive access to the global logger for test assertions + LoggerHandle::new() +} diff --git a/wireframe_testing/src/multi_packet.rs b/wireframe_testing/src/multi_packet.rs index 39fe88fd..8a00f1e0 100644 --- a/wireframe_testing/src/multi_packet.rs +++ b/wireframe_testing/src/multi_packet.rs @@ -28,7 +28,12 @@ use wireframe::Response; /// the received variant and is attributed to the caller. #[must_use] #[track_caller] -#[allow(ungated_async_fn_track_caller)] // track_caller on async is unstable +// FIXME: Remove when `track_caller` on async fns is stabilized. +// Tracking: https://github.com/rust-lang/rust/issues/110011 +#[expect( + ungated_async_fn_track_caller, + reason = "track_caller on async fns is unstable" +)] pub async fn collect_multi_packet(resp: Response) -> Vec { match resp { Response::MultiPacket(mut rx) => {