fix: removed unused patch on builtins.open in test_create_with_byoc which was not being used and causing unintended patches to open calls elsewhere

Ryan Tanaka · Ryan Tanaka · commit 4f0bbba420d4 · 2026-03-24T12:48:43.000-07:00
diff --git a/sagemaker-train/tests/unit/ai_registry/test_evaluator.py b/sagemaker-train/tests/unit/ai_registry/test_evaluator.py
@@ -61,8 +61,7 @@ def test_create_with_byoc(self, mock_air_hub, mock_boto3):
             RESPONSE_KEY_LAST_MODIFIED_TIME: "2024-01-01"
         }
         
-        with patch("builtins.open", MagicMock()), \
-             patch("zipfile.ZipFile") as mock_zip, \
+        with patch("zipfile.ZipFile") as mock_zip, \
              patch("os.path.splitext", return_value=("function", ".py")), \
              patch("os.path.basename", return_value="function.py"):
             

Original file line number	Diff line number	Diff line change
`@@ -61,8 +61,7 @@ def test_create_with_byoc(self, mock_air_hub, mock_boto3):`
`61`	`61`	`RESPONSE_KEY_LAST_MODIFIED_TIME: "2024-01-01"`
`62`	`62`	`}`
`63`	`63`
`64`		`- with patch("builtins.open", MagicMock()), \`
`65`		`- patch("zipfile.ZipFile") as mock_zip, \`
	`64`	`+ with patch("zipfile.ZipFile") as mock_zip, \`
`66`	`65`	`patch("os.path.splitext", return_value=("function", ".py")), \`
`67`	`66`	`patch("os.path.basename", return_value="function.py"):`
`68`	`67`