Skip to content

Commit 00911ec

Browse files
authored
chore(test): remove some miri exception config (#222)
1 parent 5cefcd8 commit 00911ec

File tree

5 files changed

+3
-10
lines changed

5 files changed

+3
-10
lines changed

src/client/legacy/connect/capture.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ impl CaptureConnection {
119119
}
120120
}
121121

122-
#[cfg(all(test, not(miri)))]
122+
#[cfg(test)]
123123
mod test {
124124
use super::*;
125125

src/client/legacy/connect/http.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -509,7 +509,7 @@ fn get_host_port<'u>(config: &Config, dst: &'u Uri) -> Result<(&'u str, u16), Co
509509
msg: INVALID_MISSING_HOST,
510510
addr: None,
511511
cause: None,
512-
})
512+
});
513513
}
514514
};
515515
let port = match dst.port() {
@@ -1038,7 +1038,6 @@ mod tests {
10381038
}
10391039

10401040
#[tokio::test]
1041-
#[cfg_attr(miri, ignore)]
10421041
async fn test_errors_enforce_http() {
10431042
let dst = "https://example.domain/foo/bar?baz".parse().unwrap();
10441043
let connector = HttpConnector::new();
@@ -1082,7 +1081,6 @@ mod tests {
10821081
}
10831082

10841083
#[tokio::test]
1085-
#[cfg_attr(miri, ignore)]
10861084
async fn test_errors_missing_scheme() {
10871085
let dst = "example.domain".parse().unwrap();
10881086
let mut connector = HttpConnector::new();

src/client/legacy/pool.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -826,7 +826,7 @@ impl<T> WeakOpt<T> {
826826
}
827827
}
828828

829-
#[cfg(all(test, not(miri)))]
829+
#[cfg(test)]
830830
mod tests {
831831
use std::fmt::Debug;
832832
use std::future::Future;

src/rt/tokio.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,6 @@ mod tests {
326326
use hyper::rt::Executor;
327327
use tokio::sync::oneshot;
328328

329-
#[cfg(not(miri))]
330329
#[tokio::test]
331330
async fn simple_execute() -> Result<(), Box<dyn std::error::Error>> {
332331
let (tx, rx) = oneshot::channel();

tests/legacy_client.rs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -354,7 +354,6 @@ async fn socket_disconnect_closes_idle_conn() {
354354
future::select(t, close).await;
355355
}
356356

357-
#[cfg(not(miri))]
358357
#[test]
359358
fn connect_call_is_lazy() {
360359
// We especially don't want connects() triggered if there's
@@ -1328,7 +1327,6 @@ impl tower_service::Service<hyper::Uri> for MockConnector {
13281327
// Test for connection error propagation with PR #184.
13291328
// Simulates a connection failure by setting failed=true and returning a custom io::Error.
13301329
// Verifies the error propagates through hyper’s client as a hyper::Error(Io, ...).
1331-
#[cfg(not(miri))]
13321330
#[tokio::test]
13331331
async fn test_connection_error_propagation_pr184() {
13341332
// Define the error message for the simulated connection failure.
@@ -1386,7 +1384,6 @@ async fn test_connection_error_propagation_pr184() {
13861384
// Simulates a connection that returns EOF immediately, causing hyper’s HTTP/1.1 parser
13871385
// to fail with IncompleteMessage due to no response data.
13881386
// Uses MockConnector with conn_error=None to keep failed=false, ensuring EOF behavior.
1389-
#[cfg(not(miri))]
13901387
#[tokio::test]
13911388
async fn test_incomplete_message_error_pr184() {
13921389
// Create an empty IoBuilder to simulate a connection with no data.
@@ -1446,7 +1443,6 @@ async fn test_incomplete_message_error_pr184() {
14461443
// Test for a successful HTTP/1.1 connection using a mock connector.
14471444
// Simulates a server that accepts a request and responds with a 200 OK.
14481445
// Verifies the client correctly sends the request and receives the response.
1449-
#[cfg(not(miri))]
14501446
#[tokio::test]
14511447
async fn test_successful_connection() {
14521448
// Define the expected server response: a valid HTTP/1.1 200 OK with no body.

0 commit comments

Comments
 (0)