: python/tests/test_actors.py: no fork re-enable some tests

Differential Revision: D87507353

Author: shayne-fletcher

python/tests/test_python_actors.py

@@ -541,8 +541,7 @@ def redirected_stdio(capture_stderr: bool = True):
             stderr_file.close()
 
 
-# oss_skip: pytest keeps complaining about mocking get_ipython module
-@pytest.mark.oss_skip
+@pytest.mark.timeout(180)
 async def test_actor_log_streaming() -> None:
     with configured(
         enable_log_forwarding=True, enable_file_capture=True, tail_log_lines=100
@@ -648,9 +647,7 @@ async def test_actor_log_streaming() -> None:
         ), stderr_content
 
 
-# oss_skip: pytest keeps complaining about mocking get_ipython module
-# oss_skip: (SF) broken in GitHub by D86994420. Passes internally.
-@pytest.mark.oss_skip
+@pytest.mark.timeout(180)
 async def test_alloc_based_log_streaming() -> None:
     """Test both AllocHandle.stream_logs = False and True cases."""
 
@@ -721,8 +718,7 @@ def _stream_logs(self) -> bool:
     await test_stream_logs_case(True, "stream_logs_true")
 
 
-# oss_skip: (SF) broken in GitHub by D86994420. Passes internally.
-@pytest.mark.oss_skip
+@pytest.mark.timeout(180)
 async def test_logging_option_defaults() -> None:
     with configured(
         enable_log_forwarding=True, enable_file_capture=True, tail_log_lines=100
@@ -787,8 +783,7 @@ def __init__(self):
         self.events = MockEvents()
 
 
-# oss_skip: pytest keeps complaining about mocking get_ipython module
-@pytest.mark.oss_skip
+@pytest.mark.timeout(180)
 async def test_flush_called_only_once() -> None:
     """Test that flush is called only once when ending an ipython cell"""
     with configured(
@@ -818,8 +813,6 @@ async def test_flush_called_only_once() -> None:
             assert mock_flush.call_count == 1
 
 
-# oss_skip: pytest keeps complaining about mocking get_ipython module
-@pytest.mark.oss_skip
 @pytest.mark.timeout(180)
 async def test_flush_logs_ipython() -> None:
     """Test that logs are flushed when get_ipython is available and post_run_cell event is triggered."""
@@ -879,7 +872,8 @@ async def test_flush_logs_ipython() -> None:
         assert len(re.findall(pattern2, stdout_content)) >= 3, stdout_content
 
 
-# oss_skip: importlib not pulling resource correctly in git CI, needs to be revisited
+# oss_skip: importlib not pulling resource correctly in git CI, needs
+# to be revisited
 @pytest.mark.oss_skip
 async def test_flush_logs_fast_exit() -> None:
     with configured(
@@ -912,8 +906,7 @@ async def test_flush_logs_fast_exit() -> None:
         ), process.stdout
 
 
-# oss_skip: (SF) broken in GitHub by D86994420. Passes internally.
-@pytest.mark.oss_skip
+@pytest.mark.timeout(180)
 async def test_flush_on_disable_aggregation() -> None:
     """Test that logs are flushed when disabling aggregation.
 
@@ -1008,8 +1001,7 @@ async def test_multiple_ongoing_flushes_no_deadlock() -> None:
         futures[-1].get()
 
 
-# oss_skip: (SF) broken in GitHub by D86994420. Passes internally.
-@pytest.mark.oss_skip
+@pytest.mark.timeout(180)
 async def test_adjust_aggregation_window() -> None:
     """Test that the flush deadline is updated when the aggregation window is adjusted.