Skip to content

Commit 3a0da04

Browse files
committed
refactor: cli tests for exporter
1 parent 7f27eea commit 3a0da04

1 file changed

Lines changed: 15 additions & 15 deletions

File tree

tests/cli/test_cli.py

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
# --> Basic Call Tests <--
88

99
@pytest.mark.asyncio
10-
@patch('ytfetcher._cli.Exporter.export_as_txt')
10+
@patch('ytfetcher._cli.TXTExporter.write')
1111
@patch.object(YTFetcherCLI, '_run_fetcher')
1212
async def test_run_from_channel_called(mock_run_channel, mock_export_as_txt):
1313
parser = create_parser()
@@ -21,7 +21,7 @@ async def test_run_from_channel_called(mock_run_channel, mock_export_as_txt):
2121
mock_run_channel.assert_called_once()
2222

2323
@pytest.mark.asyncio
24-
@patch('ytfetcher._cli.Exporter.export_as_txt')
24+
@patch('ytfetcher._cli.TXTExporter.write')
2525
@patch.object(YTFetcherCLI, '_run_fetcher')
2626
async def test_run_from_video_ids_called(mock_run_from_video_ids, mock_export_as_txt):
2727
parser = create_parser()
@@ -35,7 +35,7 @@ async def test_run_from_video_ids_called(mock_run_from_video_ids, mock_export_as
3535
mock_run_from_video_ids.assert_called_once()
3636

3737
@pytest.mark.asyncio
38-
@patch('ytfetcher._cli.Exporter.export_as_txt')
38+
@patch('ytfetcher._cli.TXTExporter.write')
3939
@patch.object(YTFetcherCLI, '_run_fetcher')
4040
async def test_run_from_playlist_id_called(mock_run_from_playlist_id, mock_export_as_txt):
4141
parser = create_parser()
@@ -52,7 +52,7 @@ async def test_run_from_playlist_id_called(mock_run_from_playlist_id, mock_expor
5252

5353
@pytest.mark.asyncio
5454
@patch('ytfetcher._cli.YTFetcherCLI._initialize_http_config')
55-
@patch('ytfetcher._cli.Exporter.export_as_txt')
55+
@patch('ytfetcher._cli.TXTExporter.write')
5656
@patch('ytfetcher._cli.YTFetcher')
5757
async def test_run_from_channel_arguments_passed_correctly_to_ytfetcher(mock_ytfetcher, mock_export_as_txt, mock_initialize_http_config):
5858
mock_fetcher = AsyncMock()
@@ -83,7 +83,7 @@ async def test_run_from_channel_arguments_passed_correctly_to_ytfetcher(mock_ytf
8383

8484
@pytest.mark.asyncio
8585
@patch('ytfetcher._cli.YTFetcherCLI._initialize_http_config')
86-
@patch('ytfetcher._cli.Exporter.export_as_txt')
86+
@patch('ytfetcher._cli.TXTExporter.write')
8787
@patch('ytfetcher._cli.YTFetcher')
8888
async def test_run_from_playlist_id_arguments_passed_correctly_to_ytfetcher(mock_ytfetcher, mock_export_as_txt, mock_initialize_http_config):
8989
mock_fetcher = AsyncMock()
@@ -115,7 +115,7 @@ async def test_run_from_playlist_id_arguments_passed_correctly_to_ytfetcher(mock
115115

116116
@pytest.mark.asyncio
117117
@patch('ytfetcher._cli.YTFetcherCLI._initialize_http_config')
118-
@patch('ytfetcher._cli.Exporter.export_as_txt')
118+
@patch('ytfetcher._cli.TXTExporter.write')
119119
@patch('ytfetcher._cli.YTFetcher')
120120
async def test_run_from_video_ids_arguments_passed_correctly_to_ytfetcher(mock_ytfetcher, mock_export_as_txt, mock_initialize_http_config):
121121
mock_fetcher = AsyncMock()
@@ -147,8 +147,8 @@ async def test_run_from_video_ids_arguments_passed_correctly_to_ytfetcher(mock_y
147147
# --> Exporter Tests <--
148148

149149
@pytest.mark.asyncio
150-
@patch('ytfetcher._cli.Exporter.export_as_txt')
151-
@patch('ytfetcher._cli.Exporter')
150+
@patch('ytfetcher._cli.TXTExporter.write')
151+
@patch('ytfetcher._cli.TXTExporter')
152152
@patch('ytfetcher._cli.YTFetcher')
153153
async def test_export_method_from_video_ids(mock_ytfetcher, mock_exporter_class, mock_export_as_txt):
154154
mock_fetcher = AsyncMock()
@@ -176,11 +176,11 @@ async def test_export_method_from_video_ids(mock_ytfetcher, mock_exporter_class,
176176
timing=False #Expect timing to be false, only for this method but same for others too.
177177
)
178178

179-
mock_exporter_instance.export_as_txt.assert_called_once()
179+
mock_exporter_instance.write.assert_called_once()
180180

181181
@pytest.mark.asyncio
182-
@patch('ytfetcher._cli.Exporter.export_as_txt')
183-
@patch('ytfetcher._cli.Exporter')
182+
@patch('ytfetcher._cli.TXTExporter.write')
183+
@patch('ytfetcher._cli.TXTExporter')
184184
@patch('ytfetcher._cli.YTFetcher')
185185
async def test_export_method_from_channel(mock_ytfetcher, mock_exporter_class, mock_export_as_txt):
186186
mock_fetcher = AsyncMock()
@@ -206,11 +206,11 @@ async def test_export_method_from_channel(mock_ytfetcher, mock_exporter_class, m
206206
timing=True
207207
)
208208

209-
mock_exporter_instance.export_as_txt.assert_called_once()
209+
mock_exporter_instance.write.assert_called_once()
210210

211211
@pytest.mark.asyncio
212-
@patch('ytfetcher._cli.Exporter.export_as_txt')
213-
@patch('ytfetcher._cli.Exporter')
212+
@patch('ytfetcher._cli.TXTExporter.write')
213+
@patch('ytfetcher._cli.TXTExporter')
214214
@patch('ytfetcher._cli.YTFetcher')
215215
async def test_export_method_from_playlist_id(mock_ytfetcher, mock_exporter_class, mock_export_as_txt):
216216
mock_fetcher = AsyncMock()
@@ -236,4 +236,4 @@ async def test_export_method_from_playlist_id(mock_ytfetcher, mock_exporter_clas
236236
timing=True
237237
)
238238

239-
mock_exporter_instance.export_as_txt.assert_called_once()
239+
mock_exporter_instance.write.assert_called_once()

0 commit comments

Comments
 (0)