more scope tests

Cargo.toml

@@ -188,6 +188,7 @@ hyper-rustls = { version = "0.27.5", default-features = false, features = [
 ] }
 icu_normalizer = "1.5.0"
 img-parts = "0.3.2"
+indexmap = { version = "2.7.0", features = ["serde"] }
 insta = { version = "1.42.0", default-features = false, features = [
     "glob",
     "json",

lib/komainu/Cargo.toml

@@ -20,6 +20,7 @@ compact_str.workspace = true
 http.workspace = true
 http-body.workspace = true
 http-body-util.workspace = true
+indexmap.workspace = true
 itertools.workspace = true
 memchr.workspace = true
 serde.workspace = true

lib/komainu/src/scope.rs

@@ -1,7 +1,7 @@
 use compact_str::CompactString;
+use indexmap::{set, IndexSet};
 use serde::Deserialize;
 use std::{
-    collections::{hash_set, HashSet},
     convert::Infallible,
     fmt::{self, Display},
     str::FromStr,
@@ -10,7 +10,7 @@ use std::{
 #[derive(Clone, Debug, Default, Deserialize)]
 #[serde(transparent)]
 pub struct Scope {
-    inner: HashSet<CompactString>,
+    inner: IndexSet<CompactString>,
 }
 
 impl FromStr for Scope {
@@ -86,7 +86,7 @@ where
 
 impl IntoIterator for Scope {
     type Item = CompactString;
-    type IntoIter = hash_set::IntoIter<Self::Item>;
+    type IntoIter = set::IntoIter<Self::Item>;
 
     #[inline]
     fn into_iter(self) -> Self::IntoIter {

lib/komainu/tests/scope.rs

@@ -1,3 +1,4 @@
+use compact_str::CompactString;
 use komainu::scope::Scope;
 use rstest::rstest;
 
@@ -45,3 +46,23 @@ fn cant_access(#[case] endpoint: &str, #[case] client: &str) {
 
     assert!(!endpoint.can_be_accessed_by(&client));
 }
+
+#[test]
+fn display_impl() {
+    let raw_scopes = "read write follow push";
+    let scope: Scope = raw_scopes.parse().unwrap();
+
+    assert_eq!(scope.to_string(), raw_scopes);
+}
+
+#[test]
+fn iter_impls() {
+    let raw_scopes = "read write follow";
+    let scope: Scope = raw_scopes.parse().unwrap();
+
+    let borrowed_iter: Vec<String> = scope.iter().map(ToString::to_string).collect();
+    let owned_iter: Vec<CompactString> = scope.into_iter().collect();
+
+    assert_eq!(borrowed_iter, owned_iter);
+    assert_eq!(borrowed_iter, ["read", "write", "follow"]);
+}