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.

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

Tests for odo --help #1045

Closed
cdrage opened this issue Nov 26, 2018 · 0 comments · Fixed by #2759
Closed

Tests for odo --help #1045

cdrage opened this issue Nov 26, 2018 · 0 comments · Fixed by #2759
Assignees
@cdrage
Labels
area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering estimated-size/M (10-20) Rough sizing for Epics. About 1 sprint of work for one person priority/Low Nice to have issue. It's not immediately on the project roadmap to get it done.

Comments

@cdrage
Copy link
Member

cdrage commented Nov 26, 2018

We should add tests for odo --help and odo -h

There's been a few PR's where we've modified help commands / CLI but odo --help isn't tested and thus we don't realize it's stopped working until much later.
@kadel kadel added the area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering label Nov 27, 2018
@kadel kadel added this to To do in Sprint 159 via automation Dec 3, 2018
@kadel kadel added state/Ready priority/Low Nice to have issue. It's not immediately on the project roadmap to get it done. labels Dec 3, 2018
@cdrage cdrage removed this from To do in Sprint 159 Jan 16, 2019
@cdrage cdrage self-assigned this Mar 25, 2020
cdrage added a commit to cdrage/odo that referenced this issue Mar 25, 2020
@cdrage
Add tests for --help / -h / help
355c078 
**What type of PR is this?**
/kind test

**What does does this PR do / why we need it**:

Adds some simple help integration tests

**Which issue(s) this PR fixes**:

Fixes redhat-developer#1045

**How to test changes / Special notes to the reviewer**:

N/A

Signed-off-by: Charlie Drage <charlie@charliedrage.com>
@cdrage cdrage mentioned this issue Mar 25, 2020
Merged
openshift-merge-robot pushed a commit that referenced this issue Mar 27, 2020
@cdrage
Add tests for --help / -h / help (#2759)
e87ee69 
**What type of PR is this?**
/kind test

**What does does this PR do / why we need it**:

Adds some simple help integration tests

**Which issue(s) this PR fixes**:

Fixes #1045

**How to test changes / Special notes to the reviewer**:

N/A

Signed-off-by: Charlie Drage <charlie@charliedrage.com>
@rm3l rm3l added the estimated-size/M (10-20) Rough sizing for Epics. About 1 sprint of work for one person label Jun 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cdrage cdrage

Labels
area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering estimated-size/M (10-20) Rough sizing for Epics. About 1 sprint of work for one person priority/Low Nice to have issue. It's not immediately on the project roadmap to get it done.
Projects
odo Project
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add tests for --help / -h / help cdrage/odo
4 participants
@kadel @rm3l @cdrage @girishramnani