From a196ba99a35d323e8427f7929c176b735f551147 Mon Sep 17 00:00:00 2001
From: Eric Joanis <Eric.Joanis@nrc-cnrc.gc.ca>
Date: Tue, 26 Nov 2024 13:45:31 -0500
Subject: [PATCH] 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.
---
 aligner/cli.py            | 2 +-
 aligner/tests/test_cli.py | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/aligner/cli.py b/aligner/cli.py
index 37d4268..2fb00cb 100644
--- 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
index a957c84..957c2b6 100644
--- 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):