Skip to content

PLM-186. Add unit tests for CLI #125

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

PLM-186. Add unit tests for CLI #125

wants to merge 1 commit into from

Conversation

@sandraromanchenko
Copy link
Collaborator

@sandraromanchenko sandraromanchenko commented Aug 25, 2025 •
edited by atlassian bot
Loading

PLM-186 Powered by Pull Request Badge

github-actions[bot]

This comment was marked as resolved.

Sign in to view

@sandraromanchenko sandraromanchenko force-pushed the plm_unit_cli branch 4 times, most recently from 0f42a14 to faa4f6d Compare August 25, 2025 16:04
inelpandzic
inelpandzic reviewed Aug 28, 2025
View reviewed changes
main_test.go

func TestStartCmd_NamespaceValidation(t *testing.T) {
t.Parallel()
testCmd := &cobra.Command{
Copy link
Collaborator

@inelpandzic inelpandzic Aug 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this part of the test (between lines 132 and 179), seems redundant or I'm missing something.

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

Reviewers

@inelpandzic inelpandzic inelpandzic left review comments

@github-actions github-actions[bot] github-actions[bot] left review comments

@boris-ilijic boris-ilijic boris-ilijic approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sandraromanchenko @boris-ilijic @inelpandzic