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

chore(cli): Add unit tests and restructure how cmds are invoked #92

Merged
merged 39 commits into from
Mar 23, 2021

Conversation

raulb
Copy link
Member

@raulb raulb commented Mar 22, 2021

Description of change

https://meroxa.atlassian.net/browse/PLATFORM-297

Type of change

  • New feature
  • Bug fix
  • Refactor
  • Documentation

How was this tested?

  • Unit Tests
  • Deployed to staging

Other tests that will be added in a separate PR

  • Output once all arguments and flags are valid.
  • Login/logout
  • Usage is shown when cmd is not run with valid args or flags.
  • Listing commands (this is partially tested in display_test.
  • Billing (opening browser)
  • api

@raulb raulb changed the title Raul/add tests chore(cli): Add unit tests and restructure how cmds are invoked Mar 22, 2021
@raulb raulb marked this pull request as ready for review March 23, 2021 10:53
@raulb raulb self-assigned this Mar 23, 2021
@raulb raulb requested review from ahmeroxa, owenthereal, lyuboxa, dianadoherty and mwoods79 and removed request for ahmeroxa March 23, 2021 10:53
@raulb raulb merged commit a32c4f0 into master Mar 23, 2021
@raulb raulb deleted the raul/add-tests branch March 23, 2021 14:54
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.

2 participants