Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Functional Test Gaps for CLI #7453

Closed
3 tasks
kachawla opened this issue Apr 4, 2024 · 3 comments
Closed
3 tasks

Functional Test Gaps for CLI #7453

kachawla opened this issue Apr 4, 2024 · 3 comments
Labels
maintenance Issue is a non-user-facing task like updating tests, improving automation, etc.. triaged This issue has been reviewed and triaged

Comments

@kachawla
Copy link
Contributor

kachawla commented Apr 4, 2024

Our CLI functional test coverage is currently inconsistent, with several commands lacking any tests, including env show, list, app switch, among others. Although we introduced helper functions (removed in this PR) within the test framework for these commands, they were never utilized to implement actual tests. This issue aims to perform a review of our CLI functional test coverage and identify commands that are missing tests.

Next steps:

  • Review the overall test coverage of our CLI commands.
  • Identify commands that lack sufficient test coverage.
  • Implement tests for commands lacking coverage.

AB#11714

@kachawla kachawla added the maintenance Issue is a non-user-facing task like updating tests, improving automation, etc.. label Apr 4, 2024
@radius-triage-bot
Copy link

👋 @kachawla Thanks for filing this issue.

A project maintainer will review this issue and get back to you soon.

We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

For more information on our triage process please visit our triage overview

@kachawla kachawla changed the title Functional Test Coverage Gaps for CLI Functional Test Gaps for CLI Apr 5, 2024
@sylvainsf sylvainsf added the triaged This issue has been reviewed and triaged label Apr 11, 2024
@radius-triage-bot
Copy link

👍 We've reviewed this issue and have agreed to add it to our backlog. Please subscribe to this issue for notifications, we'll provide updates when we pick it up.

We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

For more information on our triage process please visit our triage overview

@sylvainsf
Copy link
Contributor

Due to integration test coverage of the CLI most all of the code paths are covered, particularly in complex areas. Going forward we will take a leave it better than we found it approach and add functional tests if we modify functionality or move the CLI to its own repo.

@sylvainsf sylvainsf closed this as not planned Won't fix, can't repro, duplicate, stale Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Issue is a non-user-facing task like updating tests, improving automation, etc.. triaged This issue has been reviewed and triaged
Projects
None yet
Development

No branches or pull requests

2 participants