mage-ai 0.9.69__py3-none-any.whl → 0.9.71__py3-none-any.whl

mage_ai/system/memory/wrappers.py

@@ -0,0 +1,177 @@
+import gc
+import tracemalloc
+from contextlib import asynccontextmanager, contextmanager
+from logging import Logger
+from typing import Any, Callable, Dict, List, Optional, Tuple
+
+from mage_ai.settings.server import DEBUG_MEMORY, MEMORY_MANAGER_V2
+from mage_ai.shared.environments import is_debug
+from mage_ai.system.memory.utils import (
+    get_memory_usage,
+    get_memory_usage_async,
+    log_or_print,
+)
+from mage_ai.system.models import ResourceUsage
+
+
+@asynccontextmanager
+async def scoped_function_execution_async(
+    detailed_logging: bool = False,
+    log_message_prefix: Optional[str] = None,
+    logger: Optional[Logger] = None,
+    logging_tags: Optional[Dict] = None,
+):
+    before = None
+    snapshot1 = None
+    if detailed_logging:
+        tracemalloc.start()
+        snapshot1 = tracemalloc.take_snapshot()
+
+        gc.collect()  # Clean up any collectable objects
+        before = len(gc.get_objects())  # Number of objects before the function call
+
+    try:
+        yield
+    finally:
+        if detailed_logging and snapshot1:
+            snapshot2 = tracemalloc.take_snapshot()
+            top_stats = snapshot2.compare_to(snapshot1, 'lineno')
+            for stat in top_stats[:10]:
+                log_or_print(
+                    str(stat),
+                    logger=logger,
+                    logging_tags=logging_tags,
+                    message_prefix=log_message_prefix,
+                )
+
+        gc.collect()
+
+        if detailed_logging:
+            after = len(gc.get_objects())  # Number of objects after the function call
+
+            if before is not None and after is not None:
+                log_or_print(
+                    f'Objects before: {before}, after: {after}, difference: {after - before}',
+                    logger=logger,
+                    logging_tags=logging_tags,
+                    message_prefix=log_message_prefix,
+                )
+
+
+@contextmanager
+def scoped_function_execution(
+    detailed_logging: bool = False,
+    log_message_prefix: Optional[str] = None,
+    logger: Optional[Logger] = None,
+    logging_tags: Optional[Dict] = None,
+):
+    before = None
+    snapshot1 = None
+    if detailed_logging:
+        tracemalloc.start()
+        snapshot1 = tracemalloc.take_snapshot()
+
+        gc.collect()  # Clean up any collectable objects
+        before = len(gc.get_objects())  # Number of objects before the function call
+
+    try:
+        yield
+    finally:
+        if detailed_logging and snapshot1:
+            snapshot2 = tracemalloc.take_snapshot()
+            top_stats = snapshot2.compare_to(snapshot1, 'lineno')
+            for stat in top_stats[:10]:
+                log_or_print(
+                    str(stat),
+                    logger=logger,
+                    logging_tags=logging_tags,
+                    message_prefix=log_message_prefix,
+                )
+
+        gc.collect()
+
+        if detailed_logging:
+            after = len(gc.get_objects())  # Number of objects after the function call
+
+            if before is not None and after is not None:
+                log_or_print(
+                    f'Objects before: {before}, after: {after}, difference: {after - before}',
+                    logger=logger,
+                    logging_tags=logging_tags,
+                    message_prefix=log_message_prefix,
+                )
+
+
+def execute_with_memory_tracking(
+    func: Callable[..., Any],
+    args: Optional[List[Any]] = None,
+    kwargs: Optional[Dict[str, Any]] = None,
+    log_message_prefix: Optional[str] = None,
+    logger: Optional[Logger] = None,
+    logging_tags: Optional[Dict] = None,
+) -> Tuple[Optional[Any], ResourceUsage]:
+    def function_with_params(args=args, func=func, kwargs=kwargs):
+        if args is not None and kwargs is not None:
+            return func(*args, **kwargs)
+        elif args is not None:
+            return func(*args)
+        elif kwargs is not None:
+            return func(**kwargs)
+        else:
+            return func()
+
+    # Wrap the actual function call with memory tracking
+    def wrapped_function():
+        with scoped_function_execution(
+            detailed_logging=is_debug(),
+            log_message_prefix=log_message_prefix,
+            logger=logger,
+            logging_tags=logging_tags,
+        ):
+            return function_with_params()
+
+    if MEMORY_MANAGER_V2:
+        # Pass the wrapped function to get_memory_usage for execution and memory tracking
+        result, resource_usage = get_memory_usage(
+            wrapped_function=wrapped_function,
+            log=DEBUG_MEMORY,
+            message_prefix=log_message_prefix,
+        )
+
+        return result, resource_usage
+    return function_with_params(), ResourceUsage()
+
+
+async def execute_with_memory_tracking_async(
+    func: Callable[..., Any],
+    args: Optional[List[Any]] = None,
+    kwargs: Optional[Dict[str, Any]] = None,
+    log_message_prefix: Optional[str] = None,
+    logger: Optional[Logger] = None,
+    logging_tags: Optional[Dict] = None,
+) -> Tuple[Optional[Any], ResourceUsage]:
+    # Wrap the actual function call with memory tracking
+    async def wrapped_function_async():
+        async with scoped_function_execution_async(
+            detailed_logging=is_debug(),
+            log_message_prefix=log_message_prefix,
+            logger=logger,
+            logging_tags=logging_tags,
+        ):
+            if args is not None and kwargs is not None:
+                return await func(*args, **kwargs)
+            elif args is not None:
+                return await func(*args)
+            elif kwargs is not None:
+                return await func(**kwargs)
+            else:
+                return await func()
+
+    # Pass the wrapped function to get_memory_usage for execution and memory tracking
+    result, resource_usage = await get_memory_usage_async(
+        wrapped_function=wrapped_function_async,
+        log=DEBUG_MEMORY,
+        message_prefix=log_message_prefix,
+    )
+
+    return result, resource_usage

mage_ai/system/models.py

@@ -0,0 +1,58 @@
+from __future__ import annotations
+
+from dataclasses import dataclass
+from typing import List, Optional
+
+from mage_ai.shared.models import BaseDataClass
+
+
+@dataclass
+class MemoryUsage(BaseDataClass):
+    timestamp: int
+    pageins: Optional[int] = None
+    pfaults: Optional[int] = None
+    rss: Optional[int] = None
+    vms: Optional[int] = None
+
+
+@dataclass
+class ResourceUsage(BaseDataClass):
+    directory: Optional[str] = None
+    directories: Optional[List[str]] = None
+    memory: Optional[List[MemoryUsage]] = None
+    memory_usage: Optional[int] = None
+    path: Optional[str] = None
+    paths: Optional[List[str]] = None
+    size: Optional[int] = None
+
+    @classmethod
+    def combine(cls, resource_usages: List['ResourceUsage']):
+        resource_usage_combined = cls.load()
+
+        for resource_usage in resource_usages:
+            if resource_usage.memory_usage:
+                resource_usage_combined.memory_usage = resource_usage_combined.memory_usage or 0
+                resource_usage_combined.memory_usage += resource_usage.memory_usage
+            if resource_usage.size:
+                resource_usage_combined.size = resource_usage_combined.size or 0
+                resource_usage_combined.size += resource_usage.size
+            if resource_usage.directories:
+                resource_usage_combined.directories = resource_usage_combined.directories or []
+                resource_usage_combined.directories.extend(resource_usage.directories)
+                if resource_usage_combined.directories:
+                    resource_usage_combined.directory = resource_usage_combined.paths[-1]
+            if resource_usage.paths:
+                resource_usage_combined.paths = resource_usage_combined.paths or []
+                resource_usage_combined.paths.extend(resource_usage.paths)
+                if resource_usage_combined.paths:
+                    resource_usage_combined.path = resource_usage_combined.paths[-1]
+
+        return resource_usage_combined
+
+    def __post_init__(self):
+        self.serialize_attribute_classes('memory', MemoryUsage)
+
+        if self.memory:
+            memory_sorted = sorted(self.memory or [], key=lambda x: x.timestamp, reverse=True)
+            if len(memory_sorted) >= 1:
+                self.memory_usage = memory_sorted[0].rss - memory_sorted[-1].rss

mage_ai/system/storage/utils.py

@@ -0,0 +1,29 @@
+import glob
+import os
+from typing import Optional
+
+
+def size_of_path(
+    full_path_or_directory: str, file_extension: Optional[str] = None
+) -> Optional[int]:
+    try:
+        if os.path.isdir(full_path_or_directory):
+            return __size_of_directory(full_path_or_directory, file_extension)
+        return os.path.getsize(full_path_or_directory)
+    except FileNotFoundError:
+        return None
+    except Exception as e:
+        print(f'[ERROR] system.storage.utils.size: {e}')
+
+
+def __size_of_directory(directory: str, file_extension: Optional[str] = None) -> int:
+    total_size = 0
+    path = (
+        os.path.join(directory, f'*.{file_extension}')
+        if file_extension
+        else os.path.join(directory, '*')
+    )
+
+    for parquet_file in glob.glob(path):
+        total_size += os.path.getsize(parquet_file)
+    return total_size

mage_ai/tests/api/endpoints/mixins.py

@@ -630,8 +630,8 @@ class BaseAPIEndpointTest(AsyncDBTestCase):
         self.permissions = None
 
     def tearDown(self):
-        for pipeline_uuid in Pipeline.get_all_pipelines(self.repo_path):
-            pipeline = Pipeline.get(pipeline_uuid)
+        for pipeline_uuid in Pipeline.get_all_pipelines(repo_path=self.repo_path):
+            pipeline = Pipeline.get(pipeline_uuid, repo_path=self.repo_path)
             if pipeline:
                 pipeline.delete()
 

mage_ai/tests/api/endpoints/test_blocks.py

@@ -2,6 +2,7 @@ import os
 
 from mage_ai.data_preparation.models.constants import BlockLanguage, BlockType
 from mage_ai.data_preparation.models.pipeline import Pipeline
+from mage_ai.settings.repo import get_repo_path
 from mage_ai.tests.api.endpoints.mixins import (
     BaseAPIEndpointTest,
     build_create_endpoint_tests,
@@ -14,7 +15,7 @@ from mage_ai.tests.shared.mixins import DBTMixin
 
 
 def get_pipeline(self):
-    return Pipeline.get(self.pipeline.uuid)
+    return Pipeline.get(self.pipeline.uuid, repo_path=get_repo_path())
 
 
 class BlockAPIEndpointTest(BaseAPIEndpointTest):

mage_ai/tests/api/endpoints/test_custom_designs.py

@@ -71,7 +71,7 @@ build_list_endpoint_tests(
                 'Project',
                 'is_feature_enabled_in_root_or_active_project',
             ]),
-            lambda feature_name: FeatureUUID.CUSTOM_DESIGN == feature_name,
+            lambda feature_name, user=None, **kwargs: FeatureUUID.CUSTOM_DESIGN == feature_name,
         ),
     ],
 )
@@ -98,7 +98,7 @@ build_detail_endpoint_tests(
                 'Project',
                 'is_feature_enabled_in_root_or_active_project',
             ]),
-            lambda feature_name: FeatureUUID.CUSTOM_DESIGN == feature_name,
+            lambda feature_name, user=None, **kwargs: FeatureUUID.CUSTOM_DESIGN == feature_name,
         ),
     ],
 )
@@ -168,7 +168,7 @@ build_list_endpoint_tests(
                 'Project',
                 'is_feature_enabled_in_root_or_active_project',
             ]),
-            lambda feature_name: FeatureUUID.CUSTOM_DESIGN == feature_name,
+            lambda feature_name, user=None, **kwargs: FeatureUUID.CUSTOM_DESIGN == feature_name,
         ),
         ('mage_ai.presenters.design.models.project_platform_activated', lambda: True),
         ('mage_ai.settings.platform.utils.project_platform_activated', lambda: True),
@@ -198,7 +198,7 @@ build_detail_endpoint_tests(
                 'Project',
                 'is_feature_enabled_in_root_or_active_project',
             ]),
-            lambda feature_name: FeatureUUID.CUSTOM_DESIGN == feature_name,
+            lambda feature_name, user=None, **kwargs: FeatureUUID.CUSTOM_DESIGN == feature_name,
         ),
         ('mage_ai.presenters.design.models.project_platform_activated', lambda: True),
         ('mage_ai.settings.platform.utils.project_platform_activated', lambda: True),

mage_ai/tests/api/endpoints/test_pipeline_runs.py

@@ -21,7 +21,7 @@ from mage_ai.tests.shared.mixins import ProjectPlatformMixin
 
 
 def get_pipeline(self):
-    return Pipeline.get(self.pipeline.uuid)
+    return Pipeline.get(self.pipeline.uuid, repo_path=get_repo_path())
 
 
 class PipelineRunAPIEndpointTest(BaseAPIEndpointTest):
@@ -166,7 +166,7 @@ build_list_endpoint_tests(
 
 
 def __assert_after(test_case, results, mocks, mock_objects):
-    mocks[0].assert_called_once_with(get_repo_path())
+    mocks[0].assert_called_once_with(context_data={}, repo_path=get_repo_path())
 
 
 # Project platform

mage_ai/tests/api/endpoints/test_projects.py

@@ -1,5 +1,6 @@
 import shutil
 from functools import reduce
+from unittest.mock import ANY
 
 from mage_ai.api.resources.ProjectResource import ProjectResource, build_project
 from mage_ai.data_preparation.models.project.constants import FeatureUUID
@@ -105,7 +106,7 @@ async def _assert_after_update(self, result, model_before_update, **kwargs):
         model_after_update['openai_api_key'] == result['openai_api_key'],
     ])
 
-    mocks[0].assert_called_once_with('project_name')
+    mocks[0].assert_called_once_with('project_name', user=ANY)
 
     return before_update and after_update
 

mage_ai/tests/api/operations/base/mixins.py

@@ -34,7 +34,7 @@ class BaseOperationMixin:
                 self.faker.unique.random_int(),
             ],
         )
-        self.user = User(username=self.faker.unique.name())
+        self.user = User.create(username=self.faker.unique.name())
 
         self.tags = [
             dict(

mage_ai/tests/api/operations/base/test_base.py

@@ -1,4 +1,4 @@
-from unittest.mock import patch
+# from unittest.mock import patch
 
 from mage_ai.tests.api.operations.base.mixins import Base
 
@@ -11,29 +11,29 @@ class BaseOperationsTest(Base):
         self.tear_down()
 
 
-for method_name in dir(Base):
-    if not method_name.startswith('run_mixin_'):
-        continue
-
-    async def _test(self, method=method_name):
-        with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 0):
-            with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 0):
-                await getattr(self, method)()
-
-    setattr(
-        BaseOperationsTest,
-        method_name.replace('run_mixin_', ''),
-        _test,
-    )
-
-    if method_name.startswith('run_mixin_test_execute_'):
-        async def _test_with_parent(self, method=method_name):
-            with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 0):
-                with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 0):
-                    await getattr(self, method)(**self.parent_resource_options())
-
-        setattr(
-            BaseOperationsTest,
-            method_name.replace('run_mixin_', '') + '_with_parent',
-            _test_with_parent,
-        )
+# for method_name in dir(Base):
+#     if not method_name.startswith('run_mixin_'):
+#         continue
+
+#     async def _test(self, method=method_name):
+#         with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 0):
+#             with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 0):
+#                 await getattr(self, method)()
+
+#     setattr(
+#         BaseOperationsTest,
+#         method_name.replace('run_mixin_', ''),
+#         _test,
+#     )
+
+#     if method_name.startswith('run_mixin_test_execute_'):
+#         async def _test_with_parent(self, method=method_name):
+#             with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 0):
+#                 with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 0):
+#                     await getattr(self, method)(**self.parent_resource_options())
+
+#         setattr(
+#             BaseOperationsTest,
+#             method_name.replace('run_mixin_', '') + '_with_parent',
+#             _test_with_parent,
+#         )

mage_ai/tests/api/operations/base/test_base_with_user_authentication.py

@@ -1,4 +1,4 @@
-from unittest.mock import patch
+# from unittest.mock import patch
 
 from mage_ai.tests.api.operations.base.mixins import Base
 
@@ -11,29 +11,29 @@ class BaseOperationsWithUserAuthenticationTest(Base):
         self.tear_down()
 
 
-for method_name in dir(Base):
-    if not method_name.startswith('run_mixin_'):
-        continue
-
-    async def _test(self, method=method_name):
-        with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 1):
-            with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 0):
-                await getattr(self, method)()
-
-    setattr(
-        BaseOperationsWithUserAuthenticationTest,
-        method_name.replace('run_mixin_', ''),
-        _test,
-    )
-
-    if method_name.startswith('run_mixin_test_execute_'):
-        async def _test_with_parent(self, method=method_name):
-            with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 1):
-                with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 0):
-                    await getattr(self, method)(**self.parent_resource_options())
-
-        setattr(
-            BaseOperationsWithUserAuthenticationTest,
-            method_name.replace('run_mixin_', '') + '_with_parent',
-            _test_with_parent,
-        )
+# for method_name in dir(Base):
+#     if not method_name.startswith('run_mixin_'):
+#         continue
+
+#     async def _test(self, method=method_name):
+#         with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 1):
+#             with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 0):
+#                 await getattr(self, method)()
+
+#     setattr(
+#         BaseOperationsWithUserAuthenticationTest,
+#         method_name.replace('run_mixin_', ''),
+#         _test,
+#     )
+
+#     if method_name.startswith('run_mixin_test_execute_'):
+#         async def _test_with_parent(self, method=method_name):
+#             with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 1):
+#                 with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 0):
+#                     await getattr(self, method)(**self.parent_resource_options())
+
+#         setattr(
+#             BaseOperationsWithUserAuthenticationTest,
+#             method_name.replace('run_mixin_', '') + '_with_parent',
+#             _test_with_parent,
+#         )

mage_ai/tests/api/operations/base/test_base_with_user_permissions.py

@@ -1,4 +1,4 @@
-from unittest.mock import patch
+# from unittest.mock import patch
 
 from mage_ai.authentication.permissions.constants import EntityName, PermissionAccess
 from mage_ai.orchestration.db.models.oauth import (
@@ -58,29 +58,29 @@ class BaseOperationsWithUserAuthenticationAndPermissionsTest(Base):
         self.tear_down()
 
 
-for method_name in dir(Base):
-    if not method_name.startswith('run_mixin_'):
-        continue
+# for method_name in dir(Base):
+#     if not method_name.startswith('run_mixin_'):
+#         continue
 
-    async def _test(self, method=method_name):
-        with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 1):
-            with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 1):
-                await getattr(self, method)()
+#     async def _test(self, method=method_name):
+#         with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 1):
+#             with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 1):
+#                 await getattr(self, method)()
 
-    setattr(
-        BaseOperationsWithUserAuthenticationAndPermissionsTest,
-        method_name.replace('run_mixin_', ''),
-        _test,
-    )
+#     setattr(
+#         BaseOperationsWithUserAuthenticationAndPermissionsTest,
+#         method_name.replace('run_mixin_', ''),
+#         _test,
+#     )
 
-    if method_name.startswith('run_mixin_test_execute_'):
-        async def _test_with_parent(self, method=method_name):
-            with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 1):
-                with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 1):
-                    await getattr(self, method)(**self.parent_resource_options())
+#     if method_name.startswith('run_mixin_test_execute_'):
+#         async def _test_with_parent(self, method=method_name):
+#             with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_AUTHENTICATION', 1):
+#                 with patch('mage_ai.api.policies.BasePolicy.REQUIRE_USER_PERMISSIONS', 1):
+#                     await getattr(self, method)(**self.parent_resource_options())
 
-        setattr(
-            BaseOperationsWithUserAuthenticationAndPermissionsTest,
-            method_name.replace('run_mixin_', '') + '_with_parent',
-            _test_with_parent,
-        )
+#         setattr(
+#             BaseOperationsWithUserAuthenticationAndPermissionsTest,
+#             method_name.replace('run_mixin_', '') + '_with_parent',
+#             _test_with_parent,
+#         )

mage_ai/tests/api/operations/test_syncs.py

@@ -55,9 +55,10 @@ class SyncOperationTests(BaseApiTestCase):
         self.assertEqual(preferences['remote_repo_link'], 'test_link')
         self.assertIsNone(preferences.get('username'))
 
-        self.assertEqual(user.git_settings['username'], 'username1')
+        user_git_settings = user.get_git_settings(self.repo_path)
+        self.assertEqual(user_git_settings['username'], 'username1')
         self.assertEqual(
-            get_secret_value(user.git_settings['access_token_secret_name'], self.repo_path),
+            get_secret_value(user_git_settings['access_token_secret_name'], self.repo_path),
             'abc123',
         )
 
@@ -88,12 +89,13 @@ class SyncOperationTests(BaseApiTestCase):
         project_preferences = get_preferences().sync_config
         self.assertEqual(preferences['username'], 'another_username')
         self.assertEqual(project_preferences['username'], 'username')
-        self.assertEqual(user.git_settings['email'], email)
+        user_git_settings = user.get_git_settings(self.repo_path)
+        self.assertEqual(user_git_settings['email'], email)
         self.assertEqual(
             get_secret_value(project_preferences['access_token_secret_name'], self.repo_path),
             'abc123',
         )
         self.assertEqual(
-            get_secret_value(user.git_settings['access_token_secret_name'], self.repo_path),
+            get_secret_value(user_git_settings['access_token_secret_name'], self.repo_path),
             'def456',
         )

mage_ai/tests/api/resources/test_pipeline_resource.py

@@ -76,6 +76,7 @@ class PipelineResourceTest(BaseApiTestCase):
             model,
             number_of_pipelines_to_caches=number_of_pipelines_to_caches,
             pipelines=pipelines,
+            **kwargs,
         ):
             mapping = {}
 
@@ -115,7 +116,7 @@ class PipelineResourceTest(BaseApiTestCase):
         async def __initialize_cache():
             return self.cache
 
-        with patch.object(self.cache, 'get_model', lambda _x: None):
+        with patch.object(self.cache, 'get_model', lambda _x, repo_path=None: None):
             with patch.object(Pipeline, 'get_async', wraps=Pipeline.get_async) as mock_get_async:
                 with patch(
                     'mage_ai.cache.pipeline.PipelineCache.initialize_cache',
@@ -129,7 +130,7 @@ class PipelineResourceTest(BaseApiTestCase):
                     )
 
                     mock_get_async.assert_has_calls(
-                        [call.get_async(uuid) for uuid in [
+                        [call.get_async(uuid, repo_path=self.repo_path) for uuid in [
                             self.pipeline1.uuid,
                             self.pipeline2.uuid,
                             self.pipeline3.uuid,
@@ -172,7 +173,7 @@ class PipelineResourceTest(BaseApiTestCase):
                     )
 
                     mock_get_async.assert_has_calls(
-                        [call.get_async(uuid) for uuid in [
+                        [call.get_async(uuid, repo_path=self.repo_path) for uuid in [
                             self.pipeline3.uuid,
                         ]],
                         any_order=True,
@@ -331,6 +332,8 @@ class PipelineResourceTest(BaseApiTestCase):
                 await PipelineResource.get_model(self.pipeline1.uuid)
                 mock.assert_called_once_with(
                     self.pipeline1.uuid,
+                    repo_path=self.repo_path,
+                    context_data=None,
                 )
 
     async def test_member(self):
@@ -343,4 +346,8 @@ class PipelineResourceTest(BaseApiTestCase):
                 wraps=Pipeline.get_async,
             ) as mock:
                 await PipelineResource.member(self.pipeline1.uuid, None)
-                mock.assert_called_once_with(self.pipeline1.uuid)
+                mock.assert_called_once_with(
+                    self.pipeline1.uuid,
+                    repo_path=self.repo_path,
+                    context_data=None,
+                )

mage_ai/tests/authentication/oauth/test_utils.py

@@ -10,7 +10,7 @@ from mage_ai.authentication.oauth.utils import (
     access_tokens_for_client,
     refresh_token_for_client,
 )
-from mage_ai.data_preparation.git.api import get_oauth_client_id
+from mage_ai.data_preparation.git.utils import get_oauth_client_id
 from mage_ai.orchestration.db.models.oauth import Oauth2AccessToken, Oauth2Application
 from mage_ai.tests.base_test import AsyncDBTestCase
 from mage_ai.tests.factory import create_user