Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Automate Validate ServiceMeshControlPlaneAlreadyCreated Test Case #1762
Automate Validate ServiceMeshControlPlaneAlreadyCreated Test Case #1762
Changes from 3 commits
b8bbe5d
601222a
da00f98
3f99bc5
3b085ac
2296ab1
6c04140
7b4abdf
8f1f664
4cd24cb
38b1dbf
c8cd1fa
6d36f21
cf1f1c5
7c48f75
4af25ad
4b76708
31205d5
cb4618e
8db0156
28dc83d
cc3559c
df37b0a
54e611f
dec7e54
17c986c
1be5592
1171fce
3474104
ac4fccf
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Check warning
Code scanning / Robocop
The assignment sign is not consistent within the file. Expected '{{ expected_sign }}' but got '{{ actual_sign }}' instead Warning
Check notice
Code scanning / Robocop
Variable '{{ name }}' is assigned but not used Note test
Check notice
Code scanning / Robocop
Variable '{{ name }}' is assigned but not used Note test
Check warning
Code scanning / Robocop
Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test
Check warning
Code scanning / Robocop
Test case '{{ test_name }}' has too many keywords inside ({{ keyword_count }}/{{ max_allowed_count }}) Warning test
Check warning
Code scanning / Robocop
Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test
Check warning
Code scanning / Robocop
Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need a teardown method for the test just in case some step fails, so we are back to initial state, with ODH/RHOAI installed and working
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is the suite teardown
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with Cesar, we need a teardown which restore the correct RHOAI status in the cluster
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok I can move it to the Suite Teardown
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added Test Case Teardown
Check warning
Code scanning / Robocop
Keyword '{{ keyword_name }}' has too many keywords inside ({{ keyword_count }}/{{ max_allowed_count }}) Warning test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if we test a catalog source that is not created by automations and the user includes manually another name, this line will fail.
Lets see if we can get a generic way to retrieve the used catalogsource, maybe by tags or by image
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
try with some label selector
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added workaround to get CatalogSource from subscription
Check notice
Code scanning / Robocop
{{ set_variable_keyword }} can be replaced with VAR Note test
Check notice
Code scanning / Robocop
{{ set_variable_keyword }} can be replaced with VAR Note test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the same here, the subscription can have a different name if we dont use automations and we deploy the catalog source by hand
Check notice
Code scanning / Robocop
{{ set_variable_keyword }} can be replaced with VAR Note test
Check notice
Code scanning / Robocop
{{ set_variable_keyword }} can be replaced with VAR Note test
Check warning
Code scanning / Robocop
Keyword name '{{ keyword_name }}' does not follow case convention Warning test
Check notice
Code scanning / Robocop
'{{ block_name }}' condition can be simplified Note test
Check warning
Code scanning / Robocop
Keyword name '{{ keyword_name }}' does not follow case convention Warning test