Fix #567. Add posonlyargs to list of written arguments when extracting

rope/refactor/extract.py

@@ -947,7 +947,10 @@ def _handle_loop_context(self, node):
 
 
 def _get_argnames(arguments):
-    result = [node.arg for node in arguments.args if isinstance(node, ast.arg)]
+    result = []
+    if arguments.posonlyargs:
+        result.extend(node.arg for node in arguments.posonlyargs)
+    result.extend(node.arg for node in arguments.args if isinstance(node, ast.arg))
     if arguments.vararg:
         result.append(arguments.vararg.arg)
     if arguments.kwarg:

ropetest/refactor/extracttest.py

@@ -222,6 +222,24 @@ def new_func(a_var, another_var):
         """)
         self.assertEqual(expected, refactored)
 
+    def test_extract_function_with_posonlyargs(self):
+        code = dedent("""\
+            def a_func(a_var, /, b):
+                another_var = 20
+                third_var = a_var + another_var
+        """)
+        start, end = self._convert_line_range_to_offset(code, 3, 3)
+        refactored = self.do_extract_method(code, start, end, "new_func")
+        expected = dedent("""\
+            def a_func(a_var, /, b):
+                another_var = 20
+                new_func(a_var, another_var)
+
+            def new_func(a_var, another_var):
+                third_var = a_var + another_var
+        """)
+        self.assertEqual(expected, refactored)
+
     def test_extract_function_with_multiple_return_values(self):
         code = dedent("""\
             def a_func():