Enable PT011 rule to prvoider tests (apache#56929)

Author: xchwan

devel-common/tests/unit/tests_common/test_utils/test_stream_capture_manager.py

@@ -463,7 +463,7 @@ def test_exception_during_capture_with_pytest_raises(stdout_capture):
             assert "Log before exception" not in output
 
         # Now test that exception in capture context still works
-        with pytest.raises(ValueError):
+        with pytest.raises(ValueError, match="Test exception"):
             with stdout_capture:
                 raise ValueError("Test exception")
 

providers/amazon/tests/unit/amazon/aws/assets/test_s3.py

@@ -38,7 +38,7 @@ def test_sanitize_uri():
 
 
 def test_sanitize_uri_no_netloc():
-    with pytest.raises(ValueError):
+    with pytest.raises(ValueError, match="URI format s3:// must contain a bucket name"):
         sanitize_uri(urllib.parse.urlsplit("s3://"))
 
 

providers/amazon/tests/unit/amazon/aws/executors/batch/test_batch_executor.py

@@ -744,11 +744,9 @@ def test_validate_config_defaults(self):
         ],
     )
     def test_executor_config_exceptions(self, bad_config, mock_executor):
-        with pytest.raises(ValueError) as raised:
+        with pytest.raises(ValueError, match='Executor Config should never override "command'):
             mock_executor.execute_async(mock_airflow_key, mock_cmd, executor_config=bad_config)
 
-        assert raised.match('Executor Config should never override "command')
-
     def test_config_defaults_are_applied(self):
         submit_kwargs = batch_executor_config.build_submit_kwargs()
         found_keys = {convert_camel_to_snake(key): key for key in submit_kwargs.keys()}

providers/amazon/tests/unit/amazon/aws/executors/ecs/test_ecs_executor.py

@@ -1096,10 +1096,9 @@ def sync_mock():
         ],
     )
     def test_executor_config_exceptions(self, bad_config, mock_executor, mock_cmd):
-        with pytest.raises(ValueError) as raised:
+        with pytest.raises(ValueError, match='Executor Config should never override "name" or "command"'):
             mock_executor.execute_async(mock_airflow_key, mock_cmd, executor_config=bad_config)
 
-        assert raised.match('Executor Config should never override "name" or "command"')
         assert len(mock_executor.pending_tasks) == 0
 
     @mock.patch.object(ecs_executor_config, "build_task_kwargs")
@@ -1344,9 +1343,8 @@ def test_subnets_required(self):
             (CONFIG_GROUP_NAME, AllEcsConfigKeys.SECURITY_GROUPS): "sg1,sg2",
         }
         with conf_vars(conf_overrides):
-            with pytest.raises(ValueError) as raised:
+            with pytest.raises(ValueError, match="At least one subnet is required to run a task"):
                 ecs_executor_config.build_task_kwargs(conf)
-        assert raised.match("At least one subnet is required to run a task.")
 
     # TODO: When merged this needs updating to the actually supported version
     @pytest.mark.skipif(

providers/standard/tests/unit/standard/operators/test_hitl.py

@@ -456,7 +456,7 @@ def test_init_with_options(self) -> None:
             )
 
     def test_init_with_multiple_set_to_true(self) -> None:
-        with pytest.raises(ValueError):
+        with pytest.raises(ValueError, match="Passing multiple to ApprovalOperator is not allowed."):
             ApprovalOperator(
                 task_id="hitl_test",
                 subject="This is subject",