fix: shell_complete and autocompletion cannot share the same fns

Trying to write complete_path so it works for either is futile, because typer checks that the functions has the mandated signature.
EveryVoiceTTS · Nov 26, 2024 · a196ba9 · a196ba9
1 parent 6d813bd
commit a196ba9
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/aligner/cli.py b/aligner/cli.py
@@ -26,7 +26,7 @@
 )
 
 
-def complete_path(ctx=None, param=None, incomplete=None):
+def complete_path(ctx, param, incomplete):
     return []
 
 

diff --git a/aligner/tests/test_cli.py b/aligner/tests/test_cli.py
@@ -105,7 +105,6 @@ def test_align_something(self):
 
 class MiscTests(TestCase):
     def test_shell_complete(self):
-        self.assertEqual(complete_path(), [])
         self.assertEqual(complete_path(None, None, None), [])
 
     def test_segment(self):
-Original file line number
+Diff line change
@@ Expand Up / @@ -26,7 +26,7 @@ @@
     )
-    def complete_path(ctx=None, param=None, incomplete=None):
+    def complete_path(ctx, param, incomplete):
         return []
@@ Expand Down @@