Fix extract_function when the extracted code defines no variable

If the extracted lines define no name, the new function ended with a
bare "return " and the call site became " = func(v1)". Both are
invalid Python.

Only add the final return line and the assignment when there is at
least one variable to return. Otherwise call the function as a
statement.

AUTHORS.txt

@@ -71,6 +71,7 @@ Code Contributors
 - Dhanraj E (@Dhanu-dynamic) <dhanraj@labs.digiscrypt.com>
 - Charan Rathore (@charan-rathore)
 - Abdullah Ahmed (@abdull-ah-med)
+- spicysharkco (@spicysharkco)
 
 And a few more "anonymous" contributors.
 

jedi/api/refactoring/extract.py

@@ -247,6 +247,7 @@ def extract_function(inference_state, path, module_context, name, pos, until_pos
         code_block = dedent(code_block)
         if not has_ending_return_stmt:
             output_var_str = ', '.join(return_variables)
+            if return_variables:
                 code_block += 'return ' + output_var_str + '\n'
 
     # Check if we have to raise RefactoringError
@@ -282,8 +283,10 @@ def extract_function(inference_state, path, module_context, name, pos, until_pos
     else:
         if has_ending_return_stmt:
             replacement = 'return ' + function_call + '\n'
-        else:
+        elif return_variables:
             replacement = output_var_str + ' = ' + function_call + '\n'
+        else:
+            replacement = function_call + '\n'
 
     replacement_dct = _replace(nodes, replacement, function_code, pos,
                                insert_before_leaf, remaining_prefix)

test/refactor/extract_function.py

@@ -314,6 +314,22 @@ def x(v1):
     #bar
     return x
 x
+# -------------------------------------------------- with-range-func-no-output
+def x(v1):
+    #? 2 text {'new_name': 'func', 'until_line': 4}
+    print(v1)
+    print(v1 + 1)
+    return v1
+# ++++++++++++++++++++++++++++++++++++++++++++++++++
+def func(v1):
+    print(v1)
+    print(v1 + 1)
+
+
+def x(v1):
+    #? 2 text {'new_name': 'func', 'until_line': 4}
+    func(v1)
+    return v1
 # -------------------------------------------------- in-class-range-1
 class X1:
     #? 9 text {'new_name': 'f', 'until_line': 4}