Fix AttributeError when user is None in nova extractor

Co-authored-by: alvarolopez <468751+alvarolopez@users.noreply.github.com>

Author: Copilot

caso/extract/openstack/nova.py

@@ -227,7 +227,7 @@ def _build_record(self, server):
         # Filter out non-ascii characters for APEL compatibility.
         vm_name = server.name.encode("ascii", errors="ignore")
         local_user_id = server.user_id.encode("ascii", errors="ignore")
-        global_username = user.encode("ascii", errors="ignore")
+        global_username = user.encode("ascii", errors="ignore") if user else None
 
         r = record.CloudRecord(
             uuid=server.id,

caso/tests/extract/test_nova.py

@@ -14,9 +14,11 @@
 
 """Tests for the OpenStack nova extractor."""
 
-from caso.extract.openstack import nova
-
+import datetime
 import unittest
+from unittest import mock
+
+from caso.extract.openstack import nova
 
 
 class TestCasoManager(unittest.TestCase):
@@ -33,3 +35,65 @@ def tearDown(self):
         self.reset_flags()
 
         super(TestCasoManager, self).tearDown()
+
+
+class TestNovaBuildRecord(unittest.TestCase):
+    """Test case for Nova _build_record method."""
+
+    @mock.patch("caso.extract.openstack.nova.CONF")
+    def test_build_record_with_none_user(self, mock_conf):
+        """Test that _build_record handles None user without AttributeError."""
+        # Setup mocks
+        mock_conf.site_name = "test-site"
+        mock_conf.service_name = "test-service"
+        mock_conf.benchmark.name_key = "benchmark_name"
+        mock_conf.benchmark.value_key = "benchmark_value"
+
+        # Create a mock extractor with None user
+        with mock.patch.object(nova.NovaExtractor, "__init__", lambda x, y, z: None):
+            extractor = nova.NovaExtractor(None, None)
+            extractor.vo = "test-vo"
+
+            # Mock the users dictionary to return None
+            extractor.users = {"test-user-id": None}
+
+            # Mock flavors and images
+            extractor.flavors = {
+                "test-flavor-id": {
+                    "ram": 2048,
+                    "vcpus": 2,
+                    "disk": 20,
+                    "OS-FLV-EXT-DATA:ephemeral": 0,
+                    "extra": {},
+                }
+            }
+            extractor.images = {}
+
+            # Create a mock server object
+            mock_server = mock.MagicMock()
+            mock_server.id = "550e8400-e29b-41d4-a716-446655440000"
+            mock_server.name = "test-server"
+            mock_server.user_id = "test-user-id"
+            mock_server.tenant_id = "test-tenant-id"
+            mock_server.status = "ACTIVE"
+            mock_server.image = {"id": "test-image-id"}
+            mock_server.flavor = {"id": "test-flavor-id"}
+            mock_server.created = "2023-01-01T00:00:00.000000"
+
+            # Mock methods
+            start_dt = datetime.datetime(2023, 1, 1)
+            end_dt = datetime.datetime(2023, 1, 2)
+            with mock.patch.object(
+                extractor, "_get_server_start", return_value=start_dt
+            ):
+                with mock.patch.object(
+                    extractor, "_get_server_end", return_value=end_dt
+                ):
+                    with mock.patch.object(
+                        extractor, "_count_ips_on_server", return_value=0
+                    ):
+                        # This should not raise AttributeError
+                        record = extractor._build_record(mock_server)
+
+                        # Verify the record was created with user_dn=None
+                        self.assertIsNone(record.user_dn)