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

Fix Dashboard Suites 0408, 0409, 0410 Tests and Keywords #2152

Merged
merged 8 commits into from
Dec 30, 2024

Conversation

manosnoam
Copy link
Contributor

Fix Dashboard Test Suites 0408, 0409, 0410:

  • 0408__ods_dashboard_settings_pod_tolerations.robot
  • 0409__ods_dashboard_settings_accelerator_profiles.robot
  • 0410__ods_dashboard_projects/0410__ods_dashboard_projects.robot
  • 0410__ods_dashboard_projects/0410__ods_dashboard_projects_permissions_mgmt.robot

Including fixes to numerous keywords within:

  • ODHDashboard.robot
  • ODHDashboardSettings.resource
  • DataConnections.resource
  • Permissions.resource
  • Projects.resource
  • Storages.resource
  • Workbenches.resource
  • ODHDashboardSettingsAcceleratorProfiles.resource
  • AcceleratorProfiles.resource
  • Common.robot
  • Menu.robot
  • JupyterLabLauncher.robot

image

Signed-off-by: manosnoam <nmanos@redhat.com>
- The new common keyword: `Delete All ResourceType In Namespace By Name`
is used within JupyterLabLauncher KW: Clean All Standalone Notebooks.

- Also improve KW `Begin Web Test` with option to skip Jupyter Login in UI

Signed-off-by: manosnoam <nmanos@redhat.com>
Signed-off-by: manosnoam <nmanos@redhat.com>
Signed-off-by: manosnoam <nmanos@redhat.com>
Signed-off-by: manosnoam <nmanos@redhat.com>
Signed-off-by: manosnoam <nmanos@redhat.com>
@@ -24,18 +24,20 @@
... ODH dashboard, checking that the spawner is in a ready state before
... handing control over to the test suites.
[Arguments] ${username}=${TEST_USER.USERNAME} ${password}=${TEST_USER.PASSWORD}
... ${auth_type}=${TEST_USER.AUTH_TYPE}
... ${auth_type}=${TEST_USER.AUTH_TYPE} ${jupyter_login}=${TRUE}

Check notice

Code scanning / Robocop

There is too many arguments per continuation line ({{ arguments_count }} / {{ max_arguments_count }}) Note test

There is too many arguments per continuation line (2 / 1)
ods_ci/tests/Resources/Common.robot Fixed Show fixed Hide fixed
ods_ci/tests/Resources/Common.robot Fixed Show fixed Hide fixed
${GENERIC_DELETE_BTN_XP}= xpath=//*[@role="dialog"]//button[@aria-label="Delete" or .="Delete" or starts-with(., "Delete")]
${GENERIC_STOP_BTN_XP}= xpath=//*[@role="dialog"]//button[@aria-label="Stop" or .="Stop" or starts-with(., "Stop")]
${GENERIC_CLOSE_BTN_XP}= xpath=//*[@role="dialog"]//button[@aria-label="Close" or .="Close" or starts-with(., "Close")]
${GENERIC_DISABLE_BTN_XP}= xpath=//*[@role="dialog"]//button[@aria-label="Disable" or .="Disable" or starts-with(., "Disable")]

Check warning

Code scanning / Robocop

Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test

Line is too long (130/120)
Copy link
Contributor

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
589 0 0 589 100

Including fixes to numerous keywords within:
- DataConnections.resource
- Projects.resource
- Storages.resource
- Workbenches.resource

Signed-off-by: manosnoam <nmanos@redhat.com>
@@ -18,7 +18,7 @@
${result} = Run Keyword And Return Status Should Start With ${title} Authorize service account
RETURN ${result}

Authorize jupyterhub service account
Authorize JupyterLab Service Account

Check warning

Code scanning / Robocop

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

Missing documentation in 'Authorize JupyterLab Service Account' keyword
Copy link
Member

@jiridanek jiridanek left a comment

Choose a reason for hiding this comment

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

Does not have Verified label, but I'm gonna assume that it has been tried

@manosnoam manosnoam enabled auto-merge (squash) December 30, 2024 11:06
@manosnoam manosnoam added verified This PR has been tested with Jenkins enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) labels Dec 30, 2024
@manosnoam manosnoam merged commit cfad0f2 into red-hat-data-services:master Dec 30, 2024
11 checks passed
omrirh pushed a commit to omrirh/ods-ci that referenced this pull request Dec 30, 2024
…a-services#2152)

Fix Dashboard Test Suites 0408, 0409, 0410:

- 0408__ods_dashboard_settings_pod_tolerations.robot
- 0409__ods_dashboard_settings_accelerator_profiles.robot
- 0410__ods_dashboard_projects/0410__ods_dashboard_projects.robot
- 0410__ods_dashboard_projects/0410__ods_dashboard_projects_permissions_mgmt.robot

Including fixes to numerous keywords within:
- ODHDashboard.robot
- ODHDashboardSettings.resource
- DataConnections.resource
- Permissions.resource
- Projects.resource
- Storages.resource
- Workbenches.resource
- ODHDashboardSettingsAcceleratorProfiles.resource
- AcceleratorProfiles.resource
- Common.robot
- Menu.robot
- JupyterLabLauncher.robot

Signed-off-by: manosnoam <nmanos@redhat.com>
omrirh pushed a commit to omrirh/ods-ci that referenced this pull request Dec 30, 2024
…a-services#2152)

Fix Dashboard Test Suites 0408, 0409, 0410:

- 0408__ods_dashboard_settings_pod_tolerations.robot
- 0409__ods_dashboard_settings_accelerator_profiles.robot
- 0410__ods_dashboard_projects/0410__ods_dashboard_projects.robot
- 0410__ods_dashboard_projects/0410__ods_dashboard_projects_permissions_mgmt.robot

Including fixes to numerous keywords within:
- ODHDashboard.robot
- ODHDashboardSettings.resource
- DataConnections.resource
- Permissions.resource
- Projects.resource
- Storages.resource
- Workbenches.resource
- ODHDashboardSettingsAcceleratorProfiles.resource
- AcceleratorProfiles.resource
- Common.robot
- Menu.robot
- JupyterLabLauncher.robot

Signed-off-by: manosnoam <nmanos@redhat.com>
omrirh pushed a commit to omrirh/ods-ci that referenced this pull request Dec 30, 2024
…a-services#2152)

Fix Dashboard Test Suites 0408, 0409, 0410:

- 0408__ods_dashboard_settings_pod_tolerations.robot
- 0409__ods_dashboard_settings_accelerator_profiles.robot
- 0410__ods_dashboard_projects/0410__ods_dashboard_projects.robot
- 0410__ods_dashboard_projects/0410__ods_dashboard_projects_permissions_mgmt.robot

Including fixes to numerous keywords within:
- ODHDashboard.robot
- ODHDashboardSettings.resource
- DataConnections.resource
- Permissions.resource
- Projects.resource
- Storages.resource
- Workbenches.resource
- ODHDashboardSettingsAcceleratorProfiles.resource
- AcceleratorProfiles.resource
- Common.robot
- Menu.robot
- JupyterLabLauncher.robot

Signed-off-by: manosnoam <nmanos@redhat.com>
lenahorsley pushed a commit to lenahorsley/ods-ci that referenced this pull request Jan 17, 2025
…a-services#2152)

Fix Dashboard Test Suites 0408, 0409, 0410:

- 0408__ods_dashboard_settings_pod_tolerations.robot
- 0409__ods_dashboard_settings_accelerator_profiles.robot
- 0410__ods_dashboard_projects/0410__ods_dashboard_projects.robot
- 0410__ods_dashboard_projects/0410__ods_dashboard_projects_permissions_mgmt.robot

Including fixes to numerous keywords within:
- ODHDashboard.robot
- ODHDashboardSettings.resource
- DataConnections.resource
- Permissions.resource
- Projects.resource
- Storages.resource
- Workbenches.resource
- ODHDashboardSettingsAcceleratorProfiles.resource
- AcceleratorProfiles.resource
- Common.robot
- Menu.robot
- JupyterLabLauncher.robot

Signed-off-by: manosnoam <nmanos@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants