Fixed Unit Tests

databrickslabs · Nov 4, 2024 · 97481be · 97481be
1 parent 1a914df
commit 97481be
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/databricks/labs/ucx/hive_metastore/grants.py b/src/databricks/labs/ucx/hive_metastore/grants.py
@@ -815,6 +815,7 @@ def apply(self, src: SecurableObject, dst: SecurableObject) -> bool:
 
     def retrieve(self, src: SecurableObject, dst: SecurableObject) -> list[Grant]:
         grants = []
+        discover_grants = []
         ownership_grant = self._match_ownership_grant(src)
         if ownership_grant:
             grants.append(ownership_grant)
@@ -829,8 +830,8 @@ def retrieve(self, src: SecurableObject, dst: SecurableObject) -> list[Grant]:
                 )
                 continue
             logger.debug(f"Retrieving acls on {dst.full_name} using SQL query: {acl_migrate_sql}")
-            grants.append(grant)
-        return grants
+            discover_grants.append(grant)
+        return discover_grants
 
     @cached_property
     def _workspace_to_account_group_names(self) -> dict[str, str]: