Merge pull request #2980 from AndreMiras/feature/improve_sh_mocking

✅ improve sh mocking reliability

Author: AndreMiras

tests/recipes/recipe_lib_test.py

@@ -88,7 +88,7 @@ def test_build_arch(
         with mock.patch(
             f"pythonforandroid.recipes.{self.recipe_name}.sh.Command"
         ) as mock_sh_command, mock.patch(
-            f"pythonforandroid.recipes.{self.recipe_name}.sh.make"
+            f"pythonforandroid.recipes.{self.recipe_name}.sh.make", create=True
         ) as mock_make:
             self.recipe.build_arch(self.arch)
 
@@ -130,9 +130,9 @@ def test_build_arch(
         # Since the following mocks are dynamic,
         # we mock it inside a Context Manager
         with mock.patch(
-            f"pythonforandroid.recipes.{self.recipe_name}.sh.make"
+            f"pythonforandroid.recipes.{self.recipe_name}.sh.make", create=True
         ) as mock_make, mock.patch(
-            f"pythonforandroid.recipes.{self.recipe_name}.sh.cmake"
+            f"pythonforandroid.recipes.{self.recipe_name}.sh.cmake", create=True
         ) as mock_cmake:
             self.recipe.build_arch(self.arch)
 

tests/recipes/test_openal.py

@@ -9,8 +9,8 @@ class TestOpenalRecipe(BaseTestForCmakeRecipe, unittest.TestCase):
     """
     recipe_name = "openal"
 
-    @mock.patch("pythonforandroid.recipes.openal.sh.cmake")
-    @mock.patch("pythonforandroid.recipes.openal.sh.make")
+    @mock.patch("pythonforandroid.recipes.openal.sh.cmake", create=True)
+    @mock.patch("pythonforandroid.recipes.openal.sh.make", create=True)
     @mock.patch("pythonforandroid.recipes.openal.sh.cp")
     @mock.patch("pythonforandroid.util.chdir")
     @mock.patch("pythonforandroid.build.ensure_dir")