fix: make all Policy attributes private

src/abe_policy/parser.rs

@@ -69,9 +69,9 @@ impl TryFrom<HashMap<String, Vec<String>>> for Policy {
     ///            "FIN"
     ///        ]
     ///    }
-    /// ````
+    /// ```
     fn try_from(value: HashMap<String, Vec<String>>) -> Result<Self, Self::Error> {
-        let mut policy = Policy::new();
+        let mut policy = Self::new();
 
         for (axis, attributes) in &value {
             // Split the axis into axis name and hierarchy flag
@@ -81,10 +81,7 @@ impl TryFrom<HashMap<String, Vec<String>>> for Policy {
                     let hierarchical = match specs {
                         "<" => true,
                         x => {
-                            return Err(Error::ConversionFailed(format!(
-                                "unknown axis spec {}",
-                                x
-                            )));
+                            return Err(Error::ConversionFailed(format!("unknown axis spec {x}")));
                         }
                     };
                     (name, hierarchical)
@@ -104,8 +101,7 @@ impl TryFrom<HashMap<String, Vec<String>>> for Policy {
                             "+" => EncryptionHint::Hybridized,
                             x => {
                                 return Err(Error::ConversionFailed(format!(
-                                    "unknown attribute spec {}",
-                                    x
+                                    "unknown attribute spec {x}"
                                 )));
                             }
                         };
@@ -140,8 +136,7 @@ impl TryFrom<Policy> for HashMap<String, Vec<String>> {
     type Error = Error;
 
     fn try_from(policy: Policy) -> Result<Self, Self::Error> {
-        let mut result: HashMap<String, Vec<String>> =
-            HashMap::with_capacity(policy.dimensions.len());
+        let mut result: Self = Self::with_capacity(policy.dimensions.len());
 
         for (dim_name, dimension) in policy.dimensions {
             let (dim_full_name, attributes_list) = match dimension {
@@ -206,8 +201,7 @@ mod tests {
                 .get("Security Level")
                 .unwrap()
                 .attributes()
-                .collect::<Vec<_>>()
-                .len(),
+                .count(),
             3
         );
         assert_eq!(

src/abe_policy/policy_versions.rs

@@ -24,14 +24,14 @@ pub struct PolicyV2 {
 
 #[derive(Clone, PartialEq, Eq, Serialize, Deserialize, Debug)]
 pub struct PolicyV1AttributeParameters {
-    pub values: Vec<u32>,
-    pub encryption_hint: EncryptionHint,
+    values: Vec<u32>,
+    encryption_hint: EncryptionHint,
 }
 
 #[derive(Clone, PartialEq, Eq, Serialize, Deserialize, Debug)]
 pub struct OldPolicyAxisParameters {
-    pub attribute_names: Vec<String>,
-    pub is_hierarchical: bool,
+    attribute_names: Vec<String>,
+    is_hierarchical: bool,
 }
 
 #[derive(Clone, PartialEq, Eq, Serialize, Deserialize, Debug)]

src/abe_policy/tests.rs

@@ -206,8 +206,7 @@ fn specification_conversion_round_trip() -> Result<(), Error> {
             .get("Security Level")
             .unwrap()
             .attributes()
-            .collect::<Vec<_>>()
-            .len(),
+            .count(),
         3
     );
     assert_eq!(

src/core/primitives.rs

@@ -117,7 +117,7 @@ fn update_subkey_pair(
     let (sk_pq, sk_i) = msk;
 
     // update public subkey
-    *pk_i = h * &sk_i;
+    *pk_i = h * sk_i;
 
     // create or reuse Kyber keys
     if is_hybridized.into() {
@@ -541,7 +541,7 @@ mod tests {
             ),
         ]);
         // user list
-        let users_set = vec![
+        let users_set = [
             HashSet::from([dev_partition.clone()]),
             HashSet::from([admin_partition.clone(), dev_partition.clone()]),
         ];