Skip to content

Commit 1edfaee

Browse files
committed
Improve MockServerBuilder::build_bare_https
Extend its docstring and make `.expect` message clearer.
1 parent 7ee26d7 commit 1edfaee

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/mock_server/builder.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,9 @@ impl MockServerBuilder {
128128
.await
129129
}
130130

131-
/// Finalise the builder to get an instance of a [`BareMockServer`].
131+
/// Finalise the builder to get an HTTPS instance of a [`BareMockServer`].
132+
///
133+
/// Panics if DER data the `certs` is invalid.
132134
#[cfg(feature = "tls")]
133135
pub(super) async fn build_bare_https(self, certs: MockServerTlsConfig) -> BareMockServer {
134136
use hyper_server::tls_rustls::RustlsAcceptor;
@@ -149,7 +151,7 @@ impl MockServerBuilder {
149151
certs.server_keypair_der,
150152
)
151153
.await
152-
.expect("Failed to build RustlsConfig");
154+
.expect("Failed to parse TLS configuration from DER data");
153155

154156
BareMockServer::start(
155157
listener,

0 commit comments

Comments
 (0)