Skip to content

Commit

Permalink
Rework addressing code review feedback in smithy-rs#3859 (#3860)
Browse files Browse the repository at this point in the history
## Motivation and Context

1dbf37a
didn't address [code review
feedback](#3859 (comment))
correctly. This PR will fix it.

## Testing
Tests in CI

----

_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
ysaito1001 authored Oct 4, 2024
1 parent e7f1031 commit 037e433
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions aws/sdk/integration-tests/ec2/tests/paginators.rs
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@ fn stub_config(http_client: impl HttpClient + 'static) -> Config {
.build()
}

fn validate_query_string(expected: &str, actual: &str) {
let expected = expected.split('&').collect::<HashSet<&str>>();
let actual = actual.split('&').collect::<HashSet<&str>>();
fn validate_query_string(expected_str: &str, actual_str: &str) {
assert_eq!(expected_str.len(), actual_str.len());
let expected = expected_str.split('&').collect::<HashSet<_>>();
let actual = actual_str.split('&').collect::<HashSet<_>>();
assert_eq!(expected, actual);
assert_eq!(expected.len(), actual.len());
}

/// See https://github.com/awslabs/aws-sdk-rust/issues/391
Expand Down

0 comments on commit 037e433

Please sign in to comment.