Skip to content

Commit 8d99d33

Browse files
authored
🐛 Changes for ODC 4.19+ (#3713)
* Changes for ODC 4.19+ * Enable dev prespective * Drop for support for <4.14 * Selector changes for >=4.19 * Bring back support for 4.14
1 parent a16087f commit 8d99d33

File tree

4 files changed

+45
-6
lines changed

4 files changed

+45
-6
lines changed

test/ui-e2e-tests.sh

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,29 @@ function archive_cypress_artifacts {
3131
popd >/dev/null
3232
}
3333

34+
function enable_dev_perspective() {
35+
local ocpversion
36+
ocpversion="$(oc get clusterversion/version -o jsonpath='{.status.desired.version}')"
37+
if versions.lt "$ocpversion" '4.19.0'; then
38+
logger.info 'Dev Console is always enabled for OCP <4.19. Skipping the enablement.'
39+
return
40+
fi
41+
local patch='{"spec":{"customization":{"perspectives":[{"id":"dev","visibility":{"state":"Enabled"}}]}}}'
42+
43+
if LANG=C oc patch console.operator.openshift.io/cluster \
44+
--type='merge' \
45+
--dry-run='server' \
46+
-p "$patch" | grep -q 'no change'; then
47+
logger.success 'Dev Perspective already enabled'
48+
return
49+
fi
50+
51+
oc patch console.operator.openshift.io/cluster \
52+
--type='merge' \
53+
-p "$patch"
54+
logger.success 'Dev Perspective enabled'
55+
}
56+
3457
OCP_VERSION="$(oc get clusterversion version -o jsonpath='{.status.desired.version}')"
3558
OCP_USERNAME="${OCP_USERNAME:-uitesting}"
3659
OCP_PASSWORD="${OCP_PASSWORD:-$(echo "$OCP_USERNAME" | sha1sum - | awk '{print $1}')}"
@@ -51,6 +74,7 @@ export OCP_VERSION OCP_USERNAME OCP_PASSWORD OCP_LOGIN_PROVIDER CYPRESS_BASE_URL
5174

5275
add_user "$OCP_USERNAME" "$OCP_PASSWORD"
5376
check_node
77+
enable_dev_perspective
5478
archive_cypress_artifacts
5579
logger.success '🚀 Cluster prepared for testing.'
5680

test/ui/cypress/code/knative/serving/showcase.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,10 @@ class ShowcaseKservice {
2424
*/
2525
url() {
2626
if (this.clusterLocal) {
27-
let selector = '.overview__sidebar-pane .pf-v5-c-clipboard-copy input[type=text]'
27+
let selector = '.overview__sidebar-pane .pf-v6-c-clipboard-copy input[type=text]'
28+
if (environment.ocpVersion().satisfies('<=4.18')) {
29+
selector = '.overview__sidebar-pane .pf-v5-c-clipboard-copy input[type=text]'
30+
}
2831
if (environment.ocpVersion().satisfies('<=4.14')) {
2932
selector = '.overview__sidebar-pane .pf-c-clipboard-copy input[type=text]'
3033
}
@@ -35,9 +38,6 @@ class ShowcaseKservice {
3538
.and('include', 'showcase')
3639
}
3740
let selector = '.co-external-link--block a'
38-
if (environment.ocpVersion().satisfies('<=4.13')) {
39-
selector = 'a.co-external-link'
40-
}
4141
return cy.get(selector)
4242
.last()
4343
.scrollIntoView()

test/ui/cypress/code/openshift/openshiftConsole.js

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,18 @@ class OpenshiftConsole {
8585
drawer: '.odc-topology .pf-c-drawer',
8686
}
8787
}
88+
if (environment.ocpVersion().satisfies('<=4.18')) {
89+
return {
90+
/**
91+
* @param drawer {JQuery<HTMLElement>}
92+
* @returns {boolean}
93+
*/
94+
checkIsOpen: function (drawer) {
95+
return drawer.find('.pf-topology-resizable-side-bar').length > 0
96+
},
97+
drawer: '.pf-v5-c-drawer__panel.ocs-sidebar-index',
98+
}
99+
}
88100
return {
89101
/**
90102
* @param drawer {JQuery<HTMLElement>}
@@ -93,7 +105,7 @@ class OpenshiftConsole {
93105
checkIsOpen: function (drawer) {
94106
return drawer.find('.pf-topology-resizable-side-bar').length > 0
95107
},
96-
drawer: '.pf-v5-c-drawer__panel.ocs-sidebar-index',
108+
drawer: '.pf-v6-c-drawer__panel.ocs-sidebar-index',
97109
}
98110
}
99111
}

test/ui/cypress/e2e/serving/multiple-revisions.cy.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,10 @@ describe('OCP UI for Serverless Serving', () => {
4343
cy.get('input[name="trafficSplitting.1.tag"]')
4444
.type('v1')
4545
cy.contains('Select a Revision', {matchCase: false}).click()
46-
let selector = `ul.pf-v5-c-dropdown__menu button`
46+
let selector = `.pf-v6-c-dropdown.pf-m-expanded .pf-v6-c-menu button`
47+
if (environment.ocpVersion().satisfies('<=4.18')) {
48+
selector = `ul.pf-v5-c-dropdown__menu button`
49+
}
4750
if (environment.ocpVersion().satisfies('<=4.14')) {
4851
selector = `ul.pf-c-dropdown__menu button`
4952
}

0 commit comments

Comments
 (0)