o-nikolas commented on code in PR #35518:
URL: https://github.com/apache/airflow/pull/35518#discussion_r1385721527


##########
tests/providers/amazon/aws/links/test_base_aws.py:
##########
@@ -209,6 +225,16 @@ def test_empty_xcom(self):
             deserialized_task.get_extra_links(ti, self.link_class.name) == ""
         ), "Operator link should be empty for deserialized task with no XCom 
push"
 
+    def test_suppress_error_on_xcom_pull(self):
+        """Test ignore any error on XCom pull"""
+        with mock.patch.object(XCom, "get_value", 
side_effect=OSError("FakeError")) as m:
+            # m.get_value = mock.MagicMock(name="Foo")

Review Comment:
   Some commented out code left behind



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to