Skip to content

Commit

Permalink
Fix s3 tests (#3257)
Browse files Browse the repository at this point in the history
## Motivation and Context
Fix issue where S3 tests asserted on entire URI.

----

_By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice._
  • Loading branch information
rcoh authored Nov 26, 2023
1 parent 1061a3b commit fd0034e
Showing 1 changed file with 8 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -171,10 +171,10 @@ async fn behavior_mv_from_aws_config() {
.send()
.await
.expect_err("it should fail to send a request because there is no HTTP client");
assert_eq!(
req.expect_request().uri(),
"https://s3.us-west-2.amazonaws.com/"
);
assert!(req
.expect_request()
.uri()
.starts_with("https://s3.us-west-2.amazonaws.com/"));
}

#[tokio::test]
Expand All @@ -199,8 +199,8 @@ async fn behavior_mv_from_client_construction() {
.send()
.await
.expect_err("it should fail to send a request because there is no HTTP client"));
assert_eq!(
req.expect_request().uri(),
"https://s3.us-west-2.amazonaws.com/"
);
assert!(req
.expect_request()
.uri()
.starts_with("https://s3.us-west-2.amazonaws.com/"));
}

0 comments on commit fd0034e

Please sign in to comment.