Fix collect command requires duplication of collect word during invocation

.gitignore

@@ -159,4 +159,6 @@ cython_debug/
 # option (not recommended) you can uncomment the following to ignore the entire idea folder.
 .idea/
 .fleet/
-.vscode/
+.vscode/
+
+*.DS_Store

odd_cli/apps/metadata.py

@@ -8,21 +8,16 @@
 from odd_cli.logger import logger
 from odd_cli.reader.reader import read
 
-app = typer.Typer(
- short_help="Collect and ingest metadata for local files from folder",
- pretty_exceptions_show_locals=False,
-)
 
-
-@app.callback()
 def collect(
  folder: Path = typer.Argument(..., exists=True, resolve_path=True),
  platform_host: str = typer.Option(..., "--host", "-h", envvar="ODD_PLATFORM_HOST"),
  platform_token: str = typer.Option(
  ..., "--token", "-t", envvar="ODD_PLATFORM_TOKEN"
  ),
 ):
- "Collect and ingest metadata for local files from folder"
+ """Collect and ingest metadata for local files from folder"""
+
  client = Client(host=platform_host, token=platform_token)
 
  generator = FilesystemGenerator(host_settings="local")

odd_cli/main.py

@@ -3,25 +3,26 @@
 from odd_dbt.app import app as dbt_test_app
 
 from odd_cli.__version__ import VERSION
-from odd_cli.apps.metadata import app as collect_app
+from odd_cli.apps.metadata import collect
 from odd_cli.apps.tokens import app as tokens_app
 
+
 app = typer.Typer(pretty_exceptions_show_locals=False)
 
-app.add_typer(
- collect_app,
-)
-app.add_typer(
- tokens_app,
- name="tokens",
-)
+app.add_typer(tokens_app, name="tokens")
 app.add_typer(dbt_test_app, name="dbt")
 
+# such solution was made to prevent subcommand nesting (2 collect words in a row)
+# and save the modularity of commands
+app.command()(collect)
+
+
 @app.command()
 def version(
  short: bool = typer.Option(False, "--short", "-s", help="Print only version number")
 ):
- "Print version"
+ """Print version"""
+
  if short:
  print(VERSION)
  else:

tests/test_collect.py

@@ -10,7 +10,7 @@
 runner = CliRunner()
 
 
-@mock.patch("odd_cli.main.Client")
+@mock.patch("odd_cli.client.Client")
 class CollectCommandTestCase(unittest.TestCase):
  def setUp(self) -> None:
  self.folder_path = str(Path(__file__).parent / "data")