Fix warnings in tests

Author: tyranron

.github/workflows/ci.yml

@@ -240,6 +240,8 @@ jobs:
       - run: make test.cargo crate=${{ matrix.crate }}
                   careful=${{ (matrix.toolchain == 'nightly' && 'yes')
                            ||                                   'no' }}
+        env:
+          RUSTFLAGS: -D warnings
 
   test-book:
     name: test (Book)

benches/src/lib.rs

@@ -1,8 +1,4 @@
-#[cfg(test)]
-mod for_benches_only {
-    use criterion as _;
-    use tokio as _;
-}
+#![cfg_attr(test, expect(unused_crate_dependencies, reason = "benches"))]
 
 use juniper::{
     DefaultScalarValue, EmptyMutation, EmptySubscription, ExecutionError, FieldError, GraphQLEnum,

juniper_actix/examples/subscription.rs

@@ -1,5 +1,7 @@
 //! This example demonstrates asynchronous subscriptions with [`actix_web`].
 
+#![expect(unused_crate_dependencies, reason = "example")]
+
 use std::{pin::Pin, time::Duration};
 
 use actix_cors::Cors;

juniper_actix/src/lib.rs

@@ -1,6 +1,7 @@
 #![cfg_attr(docsrs, feature(doc_cfg))]
 #![cfg_attr(any(doc, test), doc = include_str!("../README.md"))]
 #![cfg_attr(not(any(doc, test)), doc = env!("CARGO_PKG_NAME"))]
+#![cfg_attr(test, expect(unused_crate_dependencies, reason = "examples"))]
 
 use actix_web::{
     Error, FromRequest, HttpMessage, HttpRequest, HttpResponse, error::JsonPayloadError,
@@ -855,6 +856,7 @@ mod subscription_tests {
         (HttpRequest, web::Payload, web::Data<Schema>),
         Output = Result<HttpResponse, Error>,
     > {
+        #[expect(closure_returning_async_block, reason = "not possible")]
         move |req: HttpRequest, stream: web::Payload, schema: web::Data<Schema>| async move {
             let context = Database::new();
             let schema = schema.into_inner();

juniper_axum/examples/custom.rs

@@ -3,6 +3,8 @@
 //! [`Handler`]: axum::handler::Handler
 //! [`starwars::schema`]: juniper::tests::fixtures::starwars::schema
 
+#![expect(unused_crate_dependencies, reason = "example")]
+
 use std::{net::SocketAddr, sync::Arc};
 
 use axum::{
@@ -31,7 +33,7 @@ async fn homepage() -> Html<&'static str> {
         .into()
 }
 
-pub async fn custom_subscriptions(
+async fn custom_subscriptions(
     Extension(schema): Extension<Arc<Schema>>,
     Extension(database): Extension<Database>,
     ws: WebSocketUpgrade,

juniper_axum/examples/simple.rs

@@ -1,5 +1,7 @@
 //! This example demonstrates simple default integration with [`axum`].
 
+#![expect(unused_crate_dependencies, reason = "example")]
+
 use std::{net::SocketAddr, sync::Arc, time::Duration};
 
 use axum::{
@@ -15,7 +17,7 @@ use tokio::{net::TcpListener, time::interval};
 use tokio_stream::wrappers::IntervalStream;
 
 #[derive(Clone, Copy, Debug)]
-pub struct Query;
+struct Query;
 
 #[graphql_object]
 impl Query {
@@ -26,7 +28,7 @@ impl Query {
 }
 
 #[derive(Clone, Copy, Debug)]
-pub struct Subscription;
+struct Subscription;
 
 type NumberStream = BoxStream<'static, Result<i32, FieldError>>;
 

juniper_axum/src/lib.rs

@@ -1,6 +1,10 @@
 #![cfg_attr(docsrs, feature(doc_cfg))]
 #![cfg_attr(any(doc, test), doc = include_str!("../README.md"))]
 #![cfg_attr(not(any(doc, test)), doc = env!("CARGO_PKG_NAME"))]
+#![cfg_attr(
+    test,
+    expect(unused_crate_dependencies, reason = "examples and integration tests")
+)]
 
 // TODO: Try remove on upgrade of `axum` crate.
 mod for_minimal_versions_check_only {

juniper_axum/tests/http_test_suite.rs

@@ -1,3 +1,7 @@
+//! [`HttpIntegration`] testing for [`axum`].
+
+#![expect(unused_crate_dependencies, reason = "integration tests")]
+
 use std::sync::Arc;
 
 use axum::{

juniper_axum/tests/ws_test_suite.rs

@@ -1,4 +1,7 @@
-#![cfg(not(windows))]
+//! [`WsIntegration`] testing for [`axum`].
+
+#![expect(unused_crate_dependencies, reason = "integration tests")]
+#![cfg_attr(windows, expect(dead_code, unused_imports, reason = "disabled"))]
 
 use std::{net::SocketAddr, sync::Arc};
 
@@ -127,12 +130,14 @@ impl WsIntegration for TestApp {
     }
 }
 
+#[cfg(not(windows))]
 #[tokio::test]
 async fn test_graphql_ws_integration() {
     let app = TestApp::new("graphql-ws");
     graphql_ws::run_test_suite(&app).await;
 }
 
+#[cfg(not(windows))]
 #[tokio::test]
 async fn test_graphql_transport_integration() {
     let app = TestApp::new("graphql-transport-ws");

juniper_codegen/src/lib.rs

@@ -1,6 +1,10 @@
 #![cfg_attr(any(doc, test), doc = include_str!("../README.md"))]
 #![cfg_attr(not(any(doc, test)), doc = env!("CARGO_PKG_NAME"))]
 #![recursion_limit = "1024"]
+#![cfg_attr(
+    all(test, not(doctest)),
+    expect(unused_crate_dependencies, reason = "for doc tests only")
+)]
 
 // NOTICE: Unfortunately this macro MUST be defined here, in the crate's root module, because Rust
 //         doesn't allow to export `macro_rules!` macros from a `proc-macro` crate type currently,