update tests

leapp internally performs os.chdir now, which is problematic in tests. Add mocks for os.chdir where needed to not crash during tests.
oamg · Oct 29, 2024 · 41618ae · 41618ae
1 parent 4b81e73
commit 41618ae
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/tests/scripts/test_metadata.py b/tests/scripts/test_metadata.py
@@ -20,7 +20,8 @@
 _DIALOG_SCOPE = 'test-dialog'
 
 _WORKFLOW_METADATA_FIELDS = ('description', 'name', 'phases', 'short_name', 'tag')
-_ACTOR_METADATA_FIELDS = ('class_name', 'name', 'description', 'phase', 'tags', 'consumes', 'produces', 'path')
+_ACTOR_METADATA_FIELDS = ('class_name', 'config', 'config_schemas', 'name', 'description',
+                          'phase', 'tags', 'consumes', 'produces', 'path')
 
 _TEST_WORKFLOW_METADATA = {
     'description': 'No description has been provided for the UnitTest workflow.',
@@ -144,15 +145,15 @@ def test_store_actor_metadata(monkeypatch, repository_dir):
     # ---
     with repository_dir.as_cwd():
         logger = logging.getLogger('leapp.actor.test')
-        with mock.patch.object(logger, 'log') as log_mock:
+        with mock.patch.object(logger, 'log') as log_mock, mock.patch('os.chdir'):
             definition = ActorDefinition('actors/test', '.', log=log_mock)
             with mock.patch('leapp.repository.actor_definition.get_actor_metadata', return_value=_TEST_ACTOR_METADATA):
                 with mock.patch('leapp.repository.actor_definition.get_actors', return_value=[True]):
                     definition._module = True
 
                     monkeypatch.setenv('LEAPP_EXECUTION_ID', _CONTEXT_NAME)
                     monkeypatch.setenv('LEAPP_HOSTNAME', _HOSTNAME)
-                    store_actor_metadata(definition, 'test-phase')
+                    store_actor_metadata(definition, 'test-phase', config={})
                     monkeypatch.delenv('LEAPP_EXECUTION_ID')
                     monkeypatch.delenv('LEAPP_HOSTNAME')
 

diff --git a/tests/scripts/test_repository_actor_definition.py b/tests/scripts/test_repository_actor_definition.py
@@ -22,7 +22,7 @@
 
 
 def test_actor_definition(repository_dir):
-    with repository_dir.as_cwd():
+    with repository_dir.as_cwd(), mock.patch('os.chdir', return_value=None):
         logger = logging.getLogger('leapp.actor.test')
         with mock.patch.object(logger, 'log') as log_mock:
             definition = ActorDefinition('actors/test', '.', log=log_mock)