Skip to content

Commit e77ae09

Browse files
committed
Python tests: Update test results to account for better handling of branches in finally blocks.
1 parent 714fecb commit e77ae09

File tree

5 files changed

+33
-31
lines changed

5 files changed

+33
-31
lines changed

python/ql/test/library-tests/ControlFlow/splitting/SuccessorCount.expected

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,7 @@
8181
| 225 | split13 | test.py:225:12:225:14 | ControlFlowNode for var | 1 |
8282
| 225 | split13 | test.py:225:12:225:14 | ControlFlowNode for var | 1 |
8383
| 225 | split13 | test.py:225:12:225:14 | ControlFlowNode for var | 1 |
84+
| 225 | split13 | test.py:225:12:225:14 | ControlFlowNode for var | 1 |
8485
| 236 | split14 | test.py:236:8:236:15 | ControlFlowNode for UnaryExpr | 1 |
8586
| 236 | split14 | test.py:236:8:236:15 | ControlFlowNode for UnaryExpr | 1 |
8687
| 241 | split15 | test.py:241:8:241:10 | ControlFlowNode for var | 2 |

python/ql/test/library-tests/ControlFlow/truefalse/ExceptionalSuccessors.expected

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
| true_false_test.py | 35 | true_false_test.py:35:18:35:26 | ControlFlowNode for range() | Entry node for Function func |
1212
| true_false_test.py | 48 | true_false_test.py:48:17:48:24 | ControlFlowNode for true12() | ControlFlowNode for ExceptStmt |
1313
| true_false_test.py | 48 | true_false_test.py:48:17:48:24 | ControlFlowNode for true12() | Entry node for Function func |
14+
| true_succ.py | 3 | true_succ.py:3:1:3:22 | ControlFlowNode for Function example | Entry node for Function example |
1415
| true_succ.py | 8 | true_succ.py:8:21:8:39 | ControlFlowNode for open() | ControlFlowNode for ExceptStmt |
1516
| true_succ.py | 8 | true_succ.py:8:21:8:39 | ControlFlowNode for open() | ControlFlowNode for f |
1617
| true_succ.py | 9 | true_succ.py:9:17:9:23 | ControlFlowNode for Attribute | ControlFlowNode for ExceptStmt |
@@ -20,6 +21,5 @@
2021
| true_succ.py | 11 | true_succ.py:11:17:11:19 | ControlFlowNode for sys | ControlFlowNode for f |
2122
| true_succ.py | 11 | true_succ.py:11:17:11:24 | ControlFlowNode for Attribute | ControlFlowNode for f |
2223
| true_succ.py | 11 | true_succ.py:11:17:11:27 | ControlFlowNode for Attribute() | ControlFlowNode for f |
23-
| true_succ.py | 13 | true_succ.py:13:16:13:28 | ControlFlowNode for Compare | Entry node for Function example |
2424
| true_succ.py | 13 | true_succ.py:13:31:13:39 | ControlFlowNode for Attribute() | Entry node for Function example |
2525
| true_succ.py | 13 | true_succ.py:13:31:13:39 | ControlFlowNode for Attribute() | Entry node for Function example |

python/ql/test/library-tests/ControlFlow/truefalse/TrueFalseSuccessors.expected

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@
8585
| true_false_test.py | 85 | true_false_test.py:85:22:85:27 | ControlFlowNode for cond24 | ControlFlowNode for cond25 | True |
8686
| true_succ.py | 4 | true_succ.py:4:8:4:15 | ControlFlowNode for filename | ControlFlowNode for Str | False |
8787
| true_succ.py | 4 | true_succ.py:4:8:4:15 | ControlFlowNode for filename | ControlFlowNode for Try | True |
88+
| true_succ.py | 13 | true_succ.py:13:16:13:28 | ControlFlowNode for Compare | ControlFlowNode for Function example | False |
8889
| true_succ.py | 13 | true_succ.py:13:16:13:28 | ControlFlowNode for Compare | ControlFlowNode for Str | False |
8990
| true_succ.py | 13 | true_succ.py:13:16:13:28 | ControlFlowNode for Compare | ControlFlowNode for f | True |
9091
| true_succ.py | 13 | true_succ.py:13:16:13:28 | ControlFlowNode for Compare | ControlFlowNode for f | True |

python/ql/test/library-tests/PointsTo/new/Dataflow.expected

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -403,14 +403,14 @@
403403
| f_finally.py:1 | Queue_0 = ClassExpr |
404404
| f_finally.py:3 | close_0 = FunctionExpr |
405405
| f_finally.py:3 | close_4 = Pi(close_0) [false] |
406-
| f_finally.py:3 | close_5 = phi(close_3, close_4) |
406+
| f_finally.py:3 | close_5 = phi(close_2, close_4) |
407407
| f_finally.py:3 | self_0 = ParameterDefinition |
408408
| f_finally.py:3 | self_3 = phi(self_1, self_2) |
409409
| f_finally.py:4 | self_1 = AttributeAssignment '_closed'(self_0) |
410410
| f_finally.py:8 | close_0 = Attribute |
411411
| f_finally.py:8 | close_1 = Attribute |
412-
| f_finally.py:9 | close_2 = SingleSuccessorGuard(close_1) [true] |
413-
| f_finally.py:10 | close_3 = Pi(close_0) [true] |
412+
| f_finally.py:10 | close_2 = Pi(close_0) [true] |
413+
| f_finally.py:10 | close_3 = Pi(close_1) [true] |
414414
| f_finally.py:10 | self_2 = AttributeAssignment '_close'(self_1) |
415415
| g_class_init.py:0 | __name___0 = ScopeEntryDefinition |
416416
| g_class_init.py:0 | __package___0 = ScopeEntryDefinition |
@@ -699,7 +699,7 @@
699699
| r_regressions.py:13 | self_2 = AttributeAssignment '_close'(self_1) |
700700
| r_regressions.py:15 | close_0 = FunctionExpr |
701701
| r_regressions.py:15 | close_4 = Pi(close_0) [false] |
702-
| r_regressions.py:15 | close_5 = phi(close_3, close_4) |
702+
| r_regressions.py:15 | close_5 = phi(close_2, close_4) |
703703
| r_regressions.py:15 | gv_8 = ScopeEntryDefinition |
704704
| r_regressions.py:15 | gv_33 = phi(gv_9, gv_11) |
705705
| r_regressions.py:15 | self_0 = ParameterDefinition |
@@ -709,8 +709,8 @@
709709
| r_regressions.py:20 | close_0 = Attribute |
710710
| r_regressions.py:20 | close_1 = Attribute |
711711
| r_regressions.py:20 | gv_10 = phi(gv_8, gv_9) |
712-
| r_regressions.py:21 | close_2 = SingleSuccessorGuard(close_1) [true] |
713-
| r_regressions.py:22 | close_3 = Pi(close_0) [true] |
712+
| r_regressions.py:22 | close_2 = Pi(close_0) [true] |
713+
| r_regressions.py:22 | close_3 = Pi(close_1) [true] |
714714
| r_regressions.py:22 | self_2 = AttributeAssignment '_close'(self_1) |
715715
| r_regressions.py:23 | gv_11 = CallsiteRefinement(gv_9) |
716716
| r_regressions.py:27 | f_0 = FunctionExpr |

python/ql/test/query-tests/Resources/Dataflow.expected

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -14,38 +14,38 @@
1414
| f6_1 = open() | open | |
1515
| f6_2 = MethodCallsiteRefinement(f6_1) | open | |
1616
| f6_3 = phi(f6_0, f6_1, f6_2) | open | |
17-
| f6_4 = SingleSuccessorGuard(f6_3) [true] | open | |
18-
| f6_5 = Pi(f6_2) [true] | open | |
19-
| f6_6 = MethodCallsiteRefinement(f6_5) | closed | |
17+
| f6_4 = Pi(f6_2) [true] | open | |
18+
| f6_5 = MethodCallsiteRefinement(f6_4) | closed | |
19+
| f6_6 = Pi(f6_3) [true] | open | |
2020
| f6_7 = Pi(f6_2) [false] | closed | |
21-
| f6_8 = phi(f6_6, f6_7) | closed | exit |
21+
| f6_8 = phi(f6_5, f6_7) | closed | exit |
2222
| f7_0 = None | closed | |
2323
| f7_1 = open() | open | |
2424
| f7_2 = MethodCallsiteRefinement(f7_1) | open | |
2525
| f7_3 = phi(f7_0, f7_1, f7_2) | open | |
26-
| f7_4 = SingleSuccessorGuard(f7_3) [true] | open | |
27-
| f7_5 = Pi(f7_2) [true] | open | |
28-
| f7_6 = MethodCallsiteRefinement(f7_5) | closed | |
26+
| f7_4 = Pi(f7_2) [true] | open | |
27+
| f7_5 = MethodCallsiteRefinement(f7_4) | closed | |
28+
| f7_6 = Pi(f7_3) [true] | open | |
2929
| f7_7 = Pi(f7_2) [false] | closed | |
30-
| f7_8 = phi(f7_6, f7_7) | closed | exit |
30+
| f7_8 = phi(f7_5, f7_7) | closed | exit |
3131
| f8_0 = None | closed | |
3232
| f8_1 = open() | open | |
3333
| f8_2 = MethodCallsiteRefinement(f8_1) | open | |
3434
| f8_3 = phi(f8_0, f8_1, f8_2) | open | |
35-
| f8_4 = SingleSuccessorGuard(f8_3) [true] | open | |
36-
| f8_5 = Pi(f8_2) [true] | closed | |
37-
| f8_6 = MethodCallsiteRefinement(f8_5) | closed | |
35+
| f8_4 = Pi(f8_2) [true] | closed | |
36+
| f8_5 = MethodCallsiteRefinement(f8_4) | closed | |
37+
| f8_6 = Pi(f8_3) [true] | closed | |
3838
| f8_7 = Pi(f8_2) [false] | open | |
39-
| f8_8 = phi(f8_6, f8_7) | open | exit |
39+
| f8_8 = phi(f8_5, f8_7) | open | exit |
4040
| f9_0 = None | closed | |
4141
| f9_1 = open() | open | |
4242
| f9_2 = MethodCallsiteRefinement(f9_1) | open | |
4343
| f9_3 = phi(f9_0, f9_1, f9_2) | open | |
44-
| f9_4 = SingleSuccessorGuard(f9_3) [true] | open | |
45-
| f9_5 = Pi(f9_2) [true] | closed | |
46-
| f9_6 = MethodCallsiteRefinement(f9_5) | closed | |
44+
| f9_4 = Pi(f9_2) [true] | closed | |
45+
| f9_5 = MethodCallsiteRefinement(f9_4) | closed | |
46+
| f9_6 = Pi(f9_3) [true] | closed | |
4747
| f9_7 = Pi(f9_2) [false] | open | |
48-
| f9_8 = phi(f9_6, f9_7) | open | exit |
48+
| f9_8 = phi(f9_5, f9_7) | open | exit |
4949
| f10_0 = open() | open | |
5050
| f10_1 = MethodCallsiteRefinement(f10_0) | open | |
5151
| f10_2 = MethodCallsiteRefinement(f10_1) | open | |
@@ -96,21 +96,21 @@
9696
| f21_2 = MethodCallsiteRefinement(f21_1) | closed | |
9797
| f21_3 = phi(f21_1, f21_2) | open | |
9898
| f21_4 = phi(f21_0, f21_1, f21_2) | open | |
99-
| f21_5 = SingleSuccessorGuard(f21_4) [true] | open | |
100-
| f21_6 = Pi(f21_3) [true] | open | |
101-
| f21_7 = MethodCallsiteRefinement(f21_6) | closed | |
99+
| f21_5 = Pi(f21_3) [true] | open | |
100+
| f21_6 = MethodCallsiteRefinement(f21_5) | closed | |
101+
| f21_7 = Pi(f21_4) [true] | open | |
102102
| f21_8 = Pi(f21_3) [false] | closed | |
103-
| f21_9 = phi(f21_7, f21_8) | closed | exit |
103+
| f21_9 = phi(f21_6, f21_8) | closed | exit |
104104
| f22_0 = open() | open | |
105105
| f22_1 = MethodCallsiteRefinement(f22_0) | open | |
106106
| f22_2 = MethodCallsiteRefinement(f22_1) | closed | |
107107
| f22_3 = phi(f22_1, f22_2) | open | |
108108
| f22_4 = phi(f22_0, f22_1, f22_2) | open | |
109-
| f22_5 = SingleSuccessorGuard(f22_4) [true] | open | |
110-
| f22_6 = Pi(f22_3) [true] | closed | |
111-
| f22_7 = MethodCallsiteRefinement(f22_6) | closed | |
109+
| f22_5 = Pi(f22_3) [true] | closed | |
110+
| f22_6 = MethodCallsiteRefinement(f22_5) | closed | |
111+
| f22_7 = Pi(f22_4) [true] | closed | |
112112
| f22_8 = Pi(f22_3) [false] | open | |
113-
| f22_9 = phi(f22_7, f22_8) | open | exit |
113+
| f22_9 = phi(f22_6, f22_8) | open | exit |
114114
| f_0 = FunctionExpr | closed | exit |
115115
| file_0 = open() | open | |
116116
| file_1 = open() | open | |

0 commit comments

Comments
 (0)