Skip to content
This repository was archived by the owner on May 17, 2024. It is now read-only.

Commit 98d0de5

Browse files
committed
fix tests
1 parent 948ffc0 commit 98d0de5

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

data_diff/info_tree.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
from typing import List, Dict, Optional, Any, Tuple
1+
from typing import List, Dict, Optional, Any, Tuple, Union
22

33
from runtype import dataclass
44

@@ -9,15 +9,15 @@
99
class SegmentInfo:
1010
tables: List[TableSegment]
1111

12-
diff: List[Tuple[Any, ...]] = None
12+
diff: List[Union[Tuple[Any, ...], List[Any]]] = None
1313
diff_schema: Tuple[Tuple[str, type], ...] = None
1414
is_diff: bool = None
1515
diff_count: int = None
1616

1717
rowcounts: Dict[int, int] = {}
1818
max_rows: int = None
1919

20-
def set_diff(self, diff: List[List[Any]], schema: Optional[Tuple[Tuple[str, type]]] = None):
20+
def set_diff(self, diff: List[Union[Tuple[Any, ...], List[Any]]], schema: Optional[Tuple[Tuple[str, type]]] = None):
2121
self.diff_schema = schema
2222
self.diff = diff
2323
self.diff_count = len(diff)

tests/test_dbt.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -835,7 +835,7 @@ def test_diff_is_not_cloud(self, mock_print, mock_dbt_parser, mock_cloud_diff, m
835835
mock_dbt_parser_inst.get_models.assert_called_once()
836836
mock_dbt_parser_inst.set_connection.assert_called_once()
837837
mock_cloud_diff.assert_not_called()
838-
mock_local_diff.assert_called_once_with(diff_vars)
838+
mock_local_diff.assert_called_once_with(diff_vars, False)
839839
mock_print.assert_not_called()
840840

841841
@patch("data_diff.dbt._get_diff_vars")
@@ -909,7 +909,7 @@ def test_diff_only_prod_db(self, mock_print, mock_dbt_parser, mock_cloud_diff, m
909909
mock_dbt_parser_inst.get_models.assert_called_once()
910910
mock_dbt_parser_inst.set_connection.assert_called_once()
911911
mock_cloud_diff.assert_not_called()
912-
mock_local_diff.assert_called_once_with(diff_vars)
912+
mock_local_diff.assert_called_once_with(diff_vars, False)
913913
mock_print.assert_not_called()
914914

915915
@patch("data_diff.dbt._get_diff_vars")
@@ -946,7 +946,7 @@ def test_diff_only_prod_schema(
946946
mock_dbt_parser_inst.get_models.assert_called_once()
947947
mock_dbt_parser_inst.set_connection.assert_called_once()
948948
mock_cloud_diff.assert_not_called()
949-
mock_local_diff.assert_called_once_with(diff_vars)
949+
mock_local_diff.assert_called_once_with(diff_vars, False)
950950
mock_print.assert_not_called()
951951

952952
@patch("data_diff.dbt._initialize_api")

0 commit comments

Comments
 (0)