fix: align actions to new cmd output format

Author: nsklikas

src/charm.py

@@ -952,7 +952,8 @@ def _on_delete_identity_action(self, event: ActionEvent) -> None:
 
         if email:
             identity = self.kratos.get_identity_from_email(email)
-            if not identity:
+            identity_id = identity.get("id")
+            if not identity_id:
                 event.fail("Couldn't retrieve identity_id from email.")
                 return
 

src/kratos.py

@@ -93,8 +93,10 @@ def list_identities(self) -> List:
 
         # TODO: Consider reading from the stream instead of waiting for output
         cmd_output = json.loads(self._run_cmd(cmd))
+        identities = cmd_output.get("identities")
         logger.info("Successfully fetched all identities")
-        return cmd_output
+
+        return identities
 
     def get_identity_from_email(self, email: str) -> Optional[Dict]:
         """Get an identity using an email.

tests/unit/test_kratos.py

@@ -130,7 +130,10 @@ def test_delete_identity(
 def test_list_identities(
     kratos_api: KratosAPI, kratos_identity_json: Dict, mocked_kratos_process: MagicMock
 ) -> None:
-    mocked_kratos_process.wait_output.return_value = (json.dumps([kratos_identity_json]), None)
+    mocked_kratos_process.wait_output.return_value = (
+        json.dumps({"identities": [kratos_identity_json]}),
+        None,
+    )
 
     kratos_api.list_identities()
 
@@ -148,7 +151,10 @@ def test_list_identities(
 def test_get_identity_from_email(
     kratos_api: KratosAPI, kratos_identity_json: Dict, mocked_kratos_process: MagicMock
 ) -> None:
-    mocked_kratos_process.wait_output.return_value = (json.dumps([kratos_identity_json]), None)
+    mocked_kratos_process.wait_output.return_value = (
+        json.dumps({"identities": [kratos_identity_json]}),
+        None,
+    )
 
     identity = kratos_api.get_identity_from_email(kratos_identity_json["traits"]["email"])
 
@@ -167,7 +173,10 @@ def test_get_identity_from_email(
 def test_get_identity_from_email_with_wrong_mail(
     kratos_api: KratosAPI, kratos_identity_json: Dict, mocked_kratos_process: MagicMock
 ) -> None:
-    mocked_kratos_process.wait_output.return_value = (json.dumps([kratos_identity_json]), None)
+    mocked_kratos_process.wait_output.return_value = (
+        json.dumps({"identities": [kratos_identity_json]}),
+        None,
+    )
 
     identity = kratos_api.get_identity_from_email("mail")