Skip to content

Commit

Permalink
Add Test and Kw for authorino
Browse files Browse the repository at this point in the history
Signed-off-by: Milind Waykole <mwaykole@mwaykole-thinkpadp1gen4i.bengluru.csb>
  • Loading branch information
Milind Waykole committed Apr 20, 2024
1 parent 9d815d5 commit 381658c
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 11 deletions.
7 changes: 0 additions & 7 deletions ods_ci/tests/Resources/CLI/ModelServing/llm.resource
Original file line number Diff line number Diff line change
Expand Up @@ -344,7 +344,6 @@ Query Model Multiple Times
... query_text=${EXP_RESPONSES}[queries][${query_idx}][query_text]
IF "${token}" != "${None}"
${header}= Set Variable "Authorization: Bearer ${token}" -H ${header}
Log To Console ${header}
END
${runtime_details}= Set Variable ${RUNTIME_FORMATS}[${runtime}][${inference_type}][${protocol}]
${endpoint}= Set Variable ${runtime_details}[endpoint]
Expand Down Expand Up @@ -782,12 +781,6 @@ Start Port-forwarding
Process Should Be Running ${process}
sleep 7s

Create Inference Access Token
[Arguments] ${test_namespace} ${DEFAULT_BUCKET_SA_NAME}
${rc} ${out}= Run And Return Rc And Output oc create token -n ${test_namespace} ${DEFAULT_BUCKET_SA_NAME}
Should Be Equal As Strings ${rc} 0
[Return] ${out}

Create Role Binding For Authorino

Check warning

Code scanning / Robocop

Missing documentation in '{{ name }}' keyword Warning test

Missing documentation in 'Create Role Binding For Authorino' keyword
[Arguments] ${name} ${namespace}
Set Test Variable ${name}

Check notice

Code scanning / Robocop

{{ set_variable_keyword }} can be replaced with VAR Note test

Set Test Variable can be replaced with VAR

Check warning

Code scanning / Robocop

Test, suite and global variables should be uppercase Warning test

Test, suite and global variables should be uppercase
Expand Down
6 changes: 6 additions & 0 deletions ods_ci/tests/Resources/RHOSi.resource
Original file line number Diff line number Diff line change
Expand Up @@ -97,3 +97,9 @@ Assign Vars According To Product
Set Suite Variable ${OPERATOR_DEPLOYMENT_NAME} opendatahub-operator-controller-manager
Set Suite Variable ${OPERATOR_LABEL_SELECTOR} control-plane=controller-manager
END

Create Inference Access Token

Check warning

Code scanning / Robocop

Missing documentation in '{{ name }}' keyword Warning test

Missing documentation in 'Create Inference Access Token' keyword
[Arguments] ${test_namespace} ${bucket_sa_name}
${rc} ${out}= Run And Return Rc And Output oc create token -n ${test_namespace} ${bucket_sa_name}
Should Be Equal As Strings ${rc} 0
[Return] ${out}

Check warning

Code scanning / Robocop

Missing trailing blank line at the end of file Warning test

Missing trailing blank line at the end of file

Check warning

Code scanning / Robocop

'{{ statement_name }}' is deprecated since Robot Framework version {{ version }}, use '{{ alternative }}' instead Warning test

'[Return]' is deprecated since Robot Framework version 5.*, use 'RETURN' instead
Original file line number Diff line number Diff line change
Expand Up @@ -552,8 +552,9 @@ Verify User Can Query A Model Using HTTP Calls

Verify User Can Serve And Query A Model With Token

Check warning

Code scanning / Robocop

Test case '{{ test_name }}' is too long ({{ test_length }}/{{ allowed_length }}) Warning test

Test case 'Verify User Can Serve And Query A Model With Token' is too long (29/20)

Check warning

Code scanning / Robocop

Test case '{{ test_name }}' has too many keywords inside ({{ keyword_count }}/{{ max_allowed_count }}) Warning test

Test case 'Verify User Can Serve And Query A Model With Token' has too many keywords inside (11/10)
[Documentation] Basic tests for preparing, deploying and querying a LLM model
... using Kserve and Caikit+TGIS runtime
[Tags] authz-2
... With Token using Kserve and Caikit+TGIS runtime
[Tags] RHOAIENG-6333
... Tier1
[Setup] Set Project And Runtime namespace=${TEST_NS}-cli
${test_namespace}= Set Variable ${TEST_NS}-cli
${flan_model_name}= Set Variable flan-t5-small-caikit
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -628,8 +628,9 @@ Verify User Can Query A Model Using HTTP Calls
Verify User Can Serve And Query A Model With Token

Check warning

Code scanning / Robocop

Test case '{{ test_name }}' is too long ({{ test_length }}/{{ allowed_length }}) Warning test

Test case 'Verify User Can Serve And Query A Model With Token' is too long (45/20)

Check warning

Code scanning / Robocop

Test case '{{ test_name }}' has too many keywords inside ({{ keyword_count }}/{{ max_allowed_count }}) Warning test

Test case 'Verify User Can Serve And Query A Model With Token' has too many keywords inside (15/10)
[Documentation] Basic tests for preparing, deploying and querying a LLM model

Check warning

Code scanning / Robocop

{{ bad_indent_msg }} Warning test

Line is over-indented
... with token using Kserve and TGIS runtime
[Tags] Tier1 ODS-authz
... With Token using Kserve and Caikit+TGIS runtime
[Tags] RHOAIENG-6306

Check warning

Code scanning / Robocop

{{ bad_indent_msg }} Warning test

Line is over-indented
... Tier1
[Setup] Set Project And Runtime runtime=${TGIS_RUNTIME_NAME} namespace=${TEST_NS}-cli

Check warning

Code scanning / Robocop

{{ bad_indent_msg }} Warning test

Line is over-indented
${test_namespace}= Set Variable ${TEST_NS}-cli

Check warning

Code scanning / Robocop

{{ bad_indent_msg }} Warning test

Line is over-indented
${flan_model_name}= Set Variable flan-t5-small-caikit

Check warning

Code scanning / Robocop

{{ bad_indent_msg }} Warning test

Line is over-indented
Expand Down

0 comments on commit 381658c

Please sign in to comment.