Skip to content

Commit

Permalink
fix: upgrades test doesn’t require TLS
Browse files Browse the repository at this point in the history
- Remove TLS from client configuration.
- Specify list of required features in Cargo.toml
  • Loading branch information
alexrudy committed Dec 10, 2024
1 parent cfacfbb commit 988aad9
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
5 changes: 5 additions & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -178,6 +178,11 @@ name = "tls"
path = "tests/tls.rs"
required-features = ["client", "server", "tls", "tls-ring"]

[[test]]
name = "upgrades"
path = "tests/upgrades.rs"
required-features = ["server", "client", "stream"]

[package.metadata.cargo-udeps.ignore]
normal = ["rustls-native-certs", "rustls", "tokio-rustls"]

Expand Down
2 changes: 0 additions & 2 deletions tests/upgrades.rs
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,6 @@ async fn client_auto() {
let mut client = hyperdriver::Client::builder()
.with_auto_http()
.with_default_pool()
.with_default_tls()
.with_transport(transport)
.build();

Expand All @@ -124,7 +123,6 @@ async fn client_http1() {
let mut client = hyperdriver::Client::builder()
.with_protocol(hyper::client::conn::http1::Builder::new())
.with_default_pool()
.with_default_tls()
.with_transport(transport)
.build();

Expand Down

0 comments on commit 988aad9

Please sign in to comment.