Ensure return code non-zero on cli error

[hwine]

After inspection, only the shell_completion did not use click methods to report errors. Fixed and added/updated tests for both internally detected errors.

📝 Summary

Fixes #3476

🔍 Description of Changes

After code inspection, only the shell_completion did not use click methods to report errors. Fixed and added/updated tests for both internally detected errors.

❯ pytest -p no:warnings tests/_cli/test_cli.py::test_shell_completion
===================================================== test session starts =====================================================
platform win32 -- Python 3.13.0, pytest-8.3.4, pluggy-1.5.0
rootdir: C:\Users\hwine\marimo
configfile: pyproject.toml
plugins: anyio-4.8.0
collected 6 items

tests\_cli\test_cli.py ......                                                                                            [100%]

===================================================== 6 passed in 14.19s ======================================================

📋 Checklist

• I have read the contributor guidelines.
• For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
• I have added tests for the changes made.
• I have run the code and verified that it works as expected.

📜 Reviewers

@mscolnick

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
marimo-storybook	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 18, 2025 7:10pm

[vercel]

@hwine is attempting to deploy a commit to the marimo Team on Vercel.

A member of the Team first needs to authorize it.

[mscolnick]

nice work! thank you

[github-actions]

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.10.15-dev6