Skip to content

Commit d46467f

Browse files
committed
Python: Update tests to account for packages having locations.
1 parent c1b8f50 commit d46467f

File tree

3 files changed

+4
-0
lines changed

3 files changed

+4
-0
lines changed

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -403,6 +403,7 @@
403403
| i_imports.py:18 | ControlFlowNode for sys | Module sys | builtin-class module | 17 | import |
404404
| i_imports.py:23 | ControlFlowNode for ImportExpr | Module code | builtin-class module | 23 | import |
405405
| i_imports.py:23 | ControlFlowNode for code | Module code | builtin-class module | 23 | import |
406+
| i_imports.py:24 | ControlFlowNode for Attribute | Module code.package | builtin-class module | 0 | import |
406407
| i_imports.py:24 | ControlFlowNode for Attribute | Module code.package.x | builtin-class module | 0 | import |
407408
| i_imports.py:24 | ControlFlowNode for code | Module code | builtin-class module | 23 | import |
408409
| i_imports.py:27 | ControlFlowNode for ImportExpr | Module code.test_package | builtin-class module | 27 | import |

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -512,6 +512,7 @@
512512
| i_imports.py:18 | ControlFlowNode for sys | Module sys | builtin-class module | 17 |
513513
| i_imports.py:23 | ControlFlowNode for ImportExpr | Module code | builtin-class module | 23 |
514514
| i_imports.py:23 | ControlFlowNode for code | Module code | builtin-class module | 23 |
515+
| i_imports.py:24 | ControlFlowNode for Attribute | Module code.package | builtin-class module | 0 |
515516
| i_imports.py:24 | ControlFlowNode for Attribute | Module code.package.x | builtin-class module | 0 |
516517
| i_imports.py:24 | ControlFlowNode for code | Module code | builtin-class module | 23 |
517518
| i_imports.py:27 | ControlFlowNode for ImportExpr | Module code.test_package | builtin-class module | 27 |

python/ql/test/library-tests/PointsTo/new/Sanity.ql

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,11 @@ predicate ssa_sanity(string clsname, string problem, string what) {
9595
not exists(m.getName()) and
9696
problem = "does not have a name"
9797
or
98+
not m.isPackage() and
9899
not exists(Variable v | v.getId() = "__name__" and v.getScope() = m) and
99100
problem = "does not have a __name__ variable"
100101
or
102+
not m.isPackage() and
101103
not exists(PyNodeDefinition def |
102104
def.getDefiningNode().getScope() = m and
103105
def.getVariable().getName() = "__name__"

0 commit comments

Comments
 (0)