Skip to content

Commit 09a2a6c

Browse files
committed
Python: Fix django re_path kwarg test
Since it was using the wrong keyword argument name before :D
1 parent b28d022 commit 09a2a6c

File tree

4 files changed

+3
-4
lines changed

4 files changed

+3
-4
lines changed

python/ql/test/experimental/library-tests/frameworks/django-v2-v3/ConceptsTest.expected

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,6 @@
1414
| routing_test.py:44:62:44:120 | Comment # $routeHandler $routedParameter=arg0 $routedParameter=arg1 | Missing result:routedParameter=arg0 |
1515
| routing_test.py:44:62:44:120 | Comment # $routeHandler $routedParameter=arg0 $routedParameter=arg1 | Missing result:routedParameter=arg1 |
1616
| routing_test.py:65:31:65:45 | Comment # $routeHandler | Missing result:routeHandler= |
17-
| routing_test.py:70:5:70:81 | ControlFlowNode for re_path() | Unexpected result: routeSetup= |
18-
| routing_test.py:70:84:70:137 | Comment # $routeSetup="^specifying-as-kwargs-is-not-a-problem" | Missing result:routeSetup="^specifying-as-kwargs-is-not-a-problem" |
1917
| routing_test.py:78:43:78:86 | Comment # $routeHandler $routedParameter=page_number | Missing result:routeHandler= |
2018
| routing_test.py:78:43:78:86 | Comment # $routeHandler $routedParameter=page_number | Missing result:routedParameter=page_number |
2119
| routing_test.py:81:43:81:120 | Comment # $routeHandler $routedParameter=foo $routedParameter=bar $routedParameter=baz | Missing result:routeHandler= |

python/ql/test/experimental/library-tests/frameworks/django-v2-v3/routing_test.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def re_path_kwargs(request): # $routeHandler
6767

6868

6969
urlpatterns = [
70-
re_path(view=re_path_kwargs, regex=r"^specifying-as-kwargs-is-not-a-problem") # $routeSetup="^specifying-as-kwargs-is-not-a-problem"
70+
re_path(view=re_path_kwargs, route=r"^specifying-as-kwargs-is-not-a-problem") # $routeSetup="^specifying-as-kwargs-is-not-a-problem"
7171
]
7272

7373
################################################################################

python/ql/test/library-tests/web/django/HttpSources.expected

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@
3939
| views_2x_3x.py:44:24:44:30 | request | django.request.HttpRequest |
4040
| views_2x_3x.py:44:33:44:36 | arg0 | externally controlled string |
4141
| views_2x_3x.py:44:39:44:42 | arg1 | externally controlled string |
42+
| views_2x_3x.py:65:20:65:26 | request | django.request.HttpRequest |
4243
| views_2x_3x.py:78:17:78:23 | request | django.request.HttpRequest |
4344
| views_2x_3x.py:78:26:78:36 | page_number | externally controlled string |
4445
| views_2x_3x.py:81:17:81:23 | request | django.request.HttpRequest |

python/ql/test/library-tests/web/django/views_2x_3x.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def re_path_kwargs(request):
6767

6868

6969
urlpatterns = [
70-
re_path(view=re_path_kwargs, regex=r'^specifying-as-kwargs-is-not-a-problem$')
70+
re_path(view=re_path_kwargs, route=r'^specifying-as-kwargs-is-not-a-problem$')
7171
]
7272

7373
################################################################################

0 commit comments

Comments
 (0)