From d6ee4ce0180ac332d6e4af4432b8e5d336e90795 Mon Sep 17 00:00:00 2001 From: Luis Perez Date: Thu, 1 Feb 2024 09:04:12 -0800 Subject: [PATCH] Remove unnecessary mocks This is what I get for trusting GHA rather than also running the tests internally :) These mocks are no longer required and are actually causing test failures internally. --- tests/utils/scribereader_test.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/utils/scribereader_test.py b/tests/utils/scribereader_test.py index 58a0848fa..65e0f4863 100644 --- a/tests/utils/scribereader_test.py +++ b/tests/utils/scribereader_test.py @@ -21,7 +21,7 @@ def test_read_log_stream_for_action_run_min_date_and_max_date_today(): "tron.utils.scribereader.get_scribereader_host_and_port", autospec=True, return_value=("host", 1234), - ), mock.patch("tron.utils.scribereader.scribereader", autospec=True,), mock.patch( + ), mock.patch( "tron.utils.scribereader.scribereader.get_stream_reader", autospec=True, ) as mock_stream_reader, mock.patch( @@ -97,7 +97,7 @@ def test_read_log_stream_for_action_run_min_date_and_max_date_different_days(): "tron.utils.scribereader.get_scribereader_host_and_port", autospec=True, return_value=("host", 1234), - ), mock.patch("tron.utils.scribereader.scribereader", autospec=True,), mock.patch( + ), mock.patch( "tron.utils.scribereader.scribereader.get_stream_reader", autospec=True, ) as mock_stream_reader, mock.patch( @@ -195,7 +195,7 @@ def test_read_log_stream_for_action_run_min_date_and_max_date_in_past(): "tron.utils.scribereader.get_scribereader_host_and_port", autospec=True, return_value=("host", 1234), - ), mock.patch("tron.utils.scribereader.scribereader", autospec=True,), mock.patch( + ), mock.patch( "tron.utils.scribereader.scribereader.get_stream_reader", autospec=True, ) as mock_stream_reader, mock.patch( @@ -261,7 +261,7 @@ def test_read_log_stream_for_action_run_min_date_and_max_date_for_long_output(): "tron.utils.scribereader.get_scribereader_host_and_port", autospec=True, return_value=("host", 1234), - ), mock.patch("tron.utils.scribereader.scribereader", autospec=True,), mock.patch( + ), mock.patch( "tron.utils.scribereader.scribereader.get_stream_reader", autospec=True, ) as mock_stream_reader, mock.patch(