Skip to content

Add CLI entrypoint test and mark slow tests #23#95

Open
AdityaGupta716 wants to merge 2 commits intomllam:mainfrom
AdityaGupta716:add-cli-entrypoint-test
Open

Add CLI entrypoint test and mark slow tests #23#95
AdityaGupta716 wants to merge 2 commits intomllam:mainfrom
AdityaGupta716:add-cli-entrypoint-test

Conversation

@AdityaGupta716
Copy link

@AdityaGupta716 AdityaGupta716 commented Feb 21, 2026

Describe your changes

Added a test for the actual installed CLI entry point by running it as a subprocess with --help. This catches import/dependency errors (like a missing psutil dependency) that would not be caught by calling the Python function directly.

Marked the existing test_call as @pytest.mark.slow since it requires network access and AWS credentials to run. Registered the slow mark in pyproject.toml to avoid pytest warnings about unknown marks.

Issue Link

Closes #23

Type of change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change
  • 📖 Documentation

Checklist before requesting a review

  • My branch is up-to-date with the target branch
  • I have performed a self-review of my code
  • For any new/modified functions/classes I have added docstrings that clearly describe its purpose, expected inputs and returned values
  • I have placed in-line comments to clarify the intent of any hard-to-understand passages of my code
  • I have updated the documentation to cover introduced code changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have given the PR a name that clearly describes the change, written in imperative form
  • I have requested a reviewer and an assignee

Author checklist after completed review

  • I have added a line to the CHANGELOG describing this change

@AdityaGupta716
Copy link
Author

hi @leifdenby plz review — let me know if the approach works for you!

@leifdenby
Copy link
Member

Please revert to using the default PR template here. Once you've done that you can tag me for a review :) thank you!

@AdityaGupta716 AdityaGupta716 force-pushed the add-cli-entrypoint-test branch from d4bc422 to c9c4dd2 Compare February 28, 2026 00:38
@AdityaGupta716
Copy link
Author

@leifdenby Plz review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Include testing of CLI in ci/cd

2 participants