Skip to content

Commit c215812

Browse files
bassmanitramMartin Bartlett
and
Martin Bartlett
authored
Remove cfg(test) on with_stage_variables (#713)
None of the other with_ methods are constrained to test-only, although this method did have a specific comment indicating test-only, so there may have been a reason. Anyway, this commit removes that constraint. Co-authored-by: Martin Bartlett <Martin [email protected]>
1 parent 45525e0 commit c215812

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

lambda-http/src/ext/extensions.rs

+1-5
Original file line numberDiff line numberDiff line change
@@ -108,10 +108,9 @@ pub trait RequestExt {
108108
/// These will always be `None` for ALB triggered requests.
109109
fn stage_variables_ref(&self) -> Option<&QueryMap>;
110110

111-
/// Configures instance with stage variables under `#[cfg(test)]` configurations
111+
/// Configures instance with stage variables
112112
///
113113
/// This is intended for use in mock testing contexts.
114-
#[cfg(test)]
115114
fn with_stage_variables<V>(self, variables: V) -> Self
116115
where
117116
V: Into<QueryMap>;
@@ -216,7 +215,6 @@ impl RequestExt for http::Extensions {
216215
.and_then(|StageVariables(vars)| if vars.is_empty() { None } else { Some(vars) })
217216
}
218217

219-
#[cfg(test)]
220218
fn with_stage_variables<V>(self, variables: V) -> Self
221219
where
222220
V: Into<QueryMap>,
@@ -318,7 +316,6 @@ impl RequestExt for Parts {
318316
self.extensions.stage_variables_ref()
319317
}
320318

321-
#[cfg(test)]
322319
fn with_stage_variables<V>(self, variables: V) -> Self
323320
where
324321
V: Into<QueryMap>,
@@ -420,7 +417,6 @@ impl<B> RequestExt for http::Request<B> {
420417
self.extensions().stage_variables_ref()
421418
}
422419

423-
#[cfg(test)]
424420
fn with_stage_variables<V>(self, variables: V) -> Self
425421
where
426422
V: Into<QueryMap>,

0 commit comments

Comments
 (0)