8000 Added/Updated tests\bugs\core_3553_test.py: Separated expected output… · FirebirdSQL/firebird-qa@9d1bfa7 · GitHub
[go: up one dir, main page]

Skip to content

Commit 9d1bfa7

Browse files
committed
Added/Updated tests\bugs\core_3553_test.py: Separated expected output for FB major versions prior/since 6.x. Checked on 6.0.0.876; 5.0.3.1668; 4.0.6.3214; 3.0.13.33813.
1 parent 6fffecc commit 9d1bfa7

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed

tests/bugs/core_3553_test.py

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,12 @@
1313
FB 5.0.0.455 and later: data sources with equal cardinality now present in the HASH plan in order they are specified in the query.
1414
Reversed order was used before this build. Because of this, two cases of expected stdout must be taken in account, see variables
1515
'fb3x_checked_stdout' and 'fb5x_checked_stdout'.
16+
17+
[27.06.2025] pzotov
18+
Separated expected output for FB major versions prior/since 6.x.
19+
No substitutions are used to suppress schema and quotes. Discussed with dimitr, 24.06.2025 12:39.
20+
21+
Checked on 6.0.0.876; 5.0.3.1668; 4.0.6.3214; 3.0.13.33813.
1622
"""
1723

1824
import pytest
@@ -29,17 +35,21 @@
2935

3036
act = isql_act('db', test_script)
3137

32-
fb3x_checked_stdout = """
38+
expected_stdout_4x = """
3339
PLAN HASH (D2 NATURAL, D1 NATURAL)
3440
"""
3541

36-
fb5x_checked_stdout = """
42+
expected_stdout_5x = """
3743
PLAN HASH (D1 NATURAL, D2 NATURAL)
3844
"""
3945

46+
expected_stdout_6x = """
47+
PLAN HASH ("D1" NATURAL, "D2" NATURAL)
48+
"""
49+
4050
@pytest.mark.version('>=3.0')
4151
def test_1(act: Action):
42-
act.expected_stdout = fb3x_checked_stdout if act.is_version('<5') else fb5x_checked_stdout
52+
act.expected_stdout = expected_stdout_4x if act.is_version('<5') else expected_stdout_5x if act.is_version('<6') else expected_stdout_6x
4353
act.execute()
4454
assert act.clean_stdout == act.clean_expected_stdout
4555

0 commit comments

Comments
 (0)
0