summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authormo khan <mo@mokhan.ca>2025-07-05 11:32:15 -0600
committermo khan <mo@mokhan.ca>2025-07-05 11:32:15 -0600
commit2dd12b6a8109f05f508f382604f49744158d2080 (patch)
treec86eb07669ccaa941e7933b71ad454d7b624a052
parentfc4bbd8efd805411239406c6323a37537d4534f2 (diff)
test: invalid request variables in tests
-rw-r--r--tests/authorization/cedar_authorizer_test.rs38
1 files changed, 17 insertions, 21 deletions
diff --git a/tests/authorization/cedar_authorizer_test.rs b/tests/authorization/cedar_authorizer_test.rs
index ccf8a1f8..490a0107 100644
--- a/tests/authorization/cedar_authorizer_test.rs
+++ b/tests/authorization/cedar_authorizer_test.rs
@@ -21,28 +21,28 @@ mod tests {
#[test]
fn test_cedar_authorizer_denies_invalid_token() {
- let request = build_request(|item: &mut HttpRequest| {
- item.headers = build_headers(vec![(
- "authorization".to_string(),
- "Bearer invalid-token".to_string(),
- )]);
- });
-
- assert!(!subject().authorize(request));
+ assert!(
+ !subject().authorize(build_request(|item: &mut HttpRequest| {
+ item.headers = build_headers(vec![(
+ "authorization".to_string(),
+ "Bearer invalid-token".to_string(),
+ )]);
+ }))
+ );
}
#[test]
fn test_cedar_authorizer_denies_missing_header() {
- let request = build_request(|item: &mut HttpRequest| {
- item.headers = HashMap::new();
- });
-
- assert!(!subject().authorize(request));
+ assert!(
+ !subject().authorize(build_request(|item: &mut HttpRequest| {
+ item.headers = HashMap::new();
+ }))
+ );
}
#[test]
fn test_cedar_authorizer_allows_static_assets() {
- let request = build_request(|item: &mut HttpRequest| {
+ assert!(subject().authorize(build_request(|item: &mut HttpRequest| {
let method = String::from("GET");
let host = String::from("sparkle.staging.runway.gitlab.net");
let path = "/public/style.css";
@@ -55,14 +55,12 @@ mod tests {
(String::from(":method"), method),
(String::from(":authority"), host),
]);
- });
-
- assert!(subject().authorize(request));
+ })));
}
#[test]
fn test_cedar_authorizer_allows_js_assets() {
- let request = build_request(|item: &mut HttpRequest| {
+ assert!(subject().authorize(build_request(|item: &mut HttpRequest| {
let method = String::from("GET");
let host = String::from("sparkle.staging.runway.gitlab.net");
let path = "/app.js";
@@ -75,8 +73,6 @@ mod tests {
(String::from(":method"), method),
(String::from(":authority"), host),
]);
- });
-
- assert!(subject().authorize(request));
+ })));
}
}