chore: remove API key tests

Author: pratik50

src/apikeys.rs

@@ -113,31 +113,3 @@ impl actix_web::ResponseError for ApiKeyError {
         }
     }
 }
-
-#[cfg(test)]
-mod request_tests {
-    use super::{ApiKeyType, CreateApiKeyRequest};
-
-    #[test]
-    fn typed_keys_and_existing_role_requests_deserialize() {
-        let ingestor: CreateApiKeyRequest =
-            serde_json::from_str(r#"{"keyName":"ingest-key","apiKeyType":"ingestor"}"#).unwrap();
-        assert_eq!(ingestor.api_key_type, Some(ApiKeyType::Ingestor));
-
-        let query: CreateApiKeyRequest =
-            serde_json::from_str(r#"{"keyName":"query-key","apiKeyType":"query"}"#).unwrap();
-        assert_eq!(query.api_key_type, Some(ApiKeyType::Query));
-
-        let ui: CreateApiKeyRequest =
-            serde_json::from_str(r#"{"keyName":"ui-key","roles":["reader-role"]}"#).unwrap();
-        assert_eq!(ui.api_key_type, None);
-        assert!(ui.roles.contains("reader-role"));
-
-        assert!(
-            serde_json::from_str::<CreateApiKeyRequest>(
-                r#"{"keyName":"old-key","ingestorOnly":true}"#
-            )
-            .is_err()
-        );
-    }
-}

src/rbac/mod.rs

@@ -428,30 +428,3 @@ pub fn roles_to_permission(roles: Vec<String>, tenant_id: &str) -> Vec<Permissio
     }
     perms.into_iter().collect()
 }
-
-#[cfg(test)]
-mod global_api_key_role_tests {
-    use super::*;
-
-    #[test]
-    fn internal_ingestor_role_grants_only_global_ingest() {
-        let permissions = roles_to_permission(vec![GLOBAL_INGESTOR_ROLE.to_string()], "tenant");
-        assert_eq!(
-            permissions,
-            vec![Permission::Resource(Action::Ingest, None)]
-        );
-    }
-
-    #[test]
-    fn internal_querier_role_has_reader_access_to_all_datasets() {
-        let permissions = roles_to_permission(vec![GLOBAL_QUERIER_ROLE.to_string()], "tenant");
-        let all_datasets = Some(ParseableResourceType::Stream("*".to_string()));
-        assert!(permissions.contains(&Permission::Resource(Action::Query, all_datasets.clone())));
-        assert!(permissions.contains(&Permission::Resource(Action::ListStream, all_datasets)));
-        assert!(!permissions.iter().any(|permission| matches!(
-            permission,
-            Permission::Resource(Action::Ingest | Action::All, _)
-                | Permission::Unit(Action::DeleteStream | Action::PutRole)
-        )));
-    }
-}