Fix renaming in function definition

kam193 · Oct 13, 2024 · aab4320 · aab4320
1 parent ea1476a
commit aab4320
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 2 deletions.
diff --git a/ASTGrep/Makefile b/ASTGrep/Makefile
@@ -6,3 +6,6 @@ AL_SERVICE_NAME=ASTGrep
 test-dependencies:
 	pip install tox
 	sudo apt-get install -y libfuzzy-dev
+
+test-quick:
+	WORK_DIR=$$(pwd) tox -c ../tox.ini -- -m 'not slow'
diff --git a/ASTGrep/rules/templates/python-rename-identifier.yml.j2 b/ASTGrep/rules/templates/python-rename-identifier.yml.j2
@@ -32,4 +32,6 @@ rule:
             has:
               stopBy: end
               pattern: {{RENAMED}}
+      - inside:
+          kind: function_definition
 fix: {{ORIGINAL}}
diff --git a/ASTGrep/tests/simple_examples/python/python-renaming-imports.in b/ASTGrep/tests/simple_examples/python/python-renaming-imports.in
@@ -31,4 +31,7 @@ script.something("aaaa")
 invoke("aaaa")
 
 def aaa():
-    invoke("aaaa")
+    invoke("aaaa")
+
+def script(other: str):
+    pass
diff --git a/ASTGrep/tests/simple_examples/python/python-renaming-imports.out b/ASTGrep/tests/simple_examples/python/python-renaming-imports.out
@@ -31,4 +31,7 @@ bitcoin.core.script.something("aaaa")
 something("aaaa")
 
 def aaa():
-    something("aaaa")
+    something("aaaa")
+
+def script(other: str):
+    pass