Skip to content

Commit 0bd0c44

Browse files
committed
format
1 parent cde7fe0 commit 0bd0c44

File tree

2 files changed

+18
-22
lines changed

2 files changed

+18
-22
lines changed

unit_tests/manifest_server/command_processor/test_utils.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,7 @@ def test_build_catalog_creates_correct_structure(self):
3232
assert configured_stream.destination_sync_mode == DestinationSyncMode.overwrite
3333

3434
@patch("airbyte_cdk.manifest_server.command_processor.utils.ConcurrentDeclarativeSource")
35-
def test_build_source_creates_manifest_declarative_source(
36-
self, mock_source_class
37-
):
35+
def test_build_source_creates_manifest_declarative_source(self, mock_source_class):
3836
"""Test that build_source creates a ConcurrentDeclarativeSource with correct parameters."""
3937
# Setup mocks
4038
mock_source = Mock()
@@ -85,9 +83,7 @@ def test_build_source_creates_manifest_declarative_source(
8583
assert result == mock_source
8684

8785
@patch("airbyte_cdk.manifest_server.command_processor.utils.ConcurrentDeclarativeSource")
88-
def test_build_source_with_normalize_flag(
89-
self, mock_source_class
90-
):
86+
def test_build_source_with_normalize_flag(self, mock_source_class):
9187
"""Test build_source when normalize flag is set."""
9288
mock_source = Mock()
9389
mock_source_class.return_value = mock_source

unit_tests/manifest_server/routers/test_manifest.py

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -110,13 +110,13 @@ def test_test_read_endpoint_success(
110110
assert response.status_code == 200
111111
# Verify build_source was called with correct arguments
112112
mock_build_source.assert_called_once_with(
113-
sample_manifest,
114-
mock_build_catalog.return_value,
115-
sample_config,
116-
[],
113+
sample_manifest,
114+
mock_build_catalog.return_value,
115+
sample_config,
116+
[],
117117
100, # record_limit
118-
5, # page_limit
119-
5 # slice_limit
118+
5, # page_limit
119+
5, # slice_limit
120120
)
121121
mock_build_catalog.assert_called_once_with("products")
122122
mock_runner.test_read.assert_called_once()
@@ -167,16 +167,16 @@ def test_test_read_with_custom_components(
167167
assert config_arg["__injected_components_py"] == custom_code
168168
assert "__injected_components_py_checksums" in config_arg
169169
assert config_arg["__injected_components_py_checksums"]["md5"] == expected_checksum
170-
170+
171171
# Verify other arguments
172172
mock_build_source.assert_called_once_with(
173173
sample_manifest,
174174
mock_build_catalog.return_value,
175175
config_arg,
176176
[],
177177
50, # record_limit
178-
3, # page_limit
179-
2 # slice_limit
178+
3, # page_limit
179+
2, # slice_limit
180180
)
181181

182182
@patch("airbyte_cdk.manifest_server.routers.manifest.AirbyteStateMessageSerializer")
@@ -245,13 +245,13 @@ def test_resolve_endpoint_success(self, sample_manifest, mock_source):
245245
assert "manifest" in data
246246
assert data["manifest"] == mock_source.resolved_manifest
247247
mock_build_source.assert_called_once_with(
248-
sample_manifest,
248+
sample_manifest,
249249
None, # catalog
250-
{}, # config
250+
{}, # config
251251
None, # state
252252
None, # record_limit
253253
None, # page_limit
254-
None # slice_limit
254+
None, # slice_limit
255255
)
256256

257257
def test_resolve_invalid_manifest(self):
@@ -417,7 +417,7 @@ def test_check_endpoint_success(
417417
None, # state
418418
None, # record_limit
419419
None, # page_limit
420-
None # slice_limit
420+
None, # slice_limit
421421
)
422422
mock_runner_class.assert_called_once_with(mock_source)
423423
mock_runner.check_connection.assert_called_once_with(sample_config)
@@ -445,15 +445,15 @@ def test_check_endpoint_failure(
445445
data = response.json()
446446
assert data["success"] is False
447447
assert data["message"] == "Invalid API key"
448-
448+
449449
mock_build_source.assert_called_once_with(
450450
sample_manifest,
451451
None, # catalog
452452
sample_config,
453453
None, # state
454454
None, # record_limit
455455
None, # page_limit
456-
None # slice_limit
456+
None, # slice_limit
457457
)
458458

459459
@patch("airbyte_cdk.manifest_server.routers.manifest.ManifestCommandProcessor")
@@ -500,7 +500,7 @@ def test_discover_endpoint_success(
500500
None, # state
501501
None, # record_limit
502502
None, # page_limit
503-
None # slice_limit
503+
None, # slice_limit
504504
)
505505
mock_runner_class.assert_called_once_with(mock_source)
506506
mock_runner.discover.assert_called_once_with(sample_config)

0 commit comments

Comments
 (0)