From ea7ff8c57de65185f6a61f1b8bcaf708b6a749db Mon Sep 17 00:00:00 2001 From: Punya Shree S Date: Wed, 18 Feb 2026 00:09:36 +0530 Subject: [PATCH] Replaced sys.exit() with ClickException in test_connection decorator. Established initial exception handling pattern for CLI failures as discussed in #3676. --- augur/application/cli/__init__.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/augur/application/cli/__init__.py b/augur/application/cli/__init__.py index e68af307bb..e5546dd7db 100644 --- a/augur/application/cli/__init__.py +++ b/augur/application/cli/__init__.py @@ -33,14 +33,14 @@ def new_func(ctx, *args, **kwargs): print(traceback.format_exc()) if not success: - print( + raise click.ClickException( f""" - \n\n{usage} command setup failed. + {usage} command setup failed. There was an error while testing for network connectivity Please check your connection to the internet to run Augur Consider setting http_proxy variables for limited access installations.""" ) - sys.exit(-1) + return ctx.invoke(function_internet_connection, *args, **kwargs)