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
Add E2E tests for Installation flow #4803
Add E2E tests for Installation flow #4803
Conversation
frontend/packages/ceph-storage-plugin/integration-tests/tests/ocp-tests/installFlow.scenario.ts
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/integration-tests/tests/ocp-tests/installFlow.scenario.ts
Outdated
Show resolved
Hide resolved
c390689
to
de836be
Compare
frontend/packages/ceph-storage-plugin/integration-tests/tests/ocp-tests/installFlow.scenario.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/integration-tests/tests/ocp-tests/installFlow.scenario.ts
Outdated
Show resolved
Hide resolved
|
||
const Installer = new InstallCluster(NS); | ||
|
||
describe('Testing OCS Cluster Creation', () => { |
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.
describe('Testing OCS Cluster Creation', () => { | |
describe('Testing OpenShift Container Storage Installation, () => { |
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 are testing cluster creation there are e2e-tests for operators-hub that test installation of an operator.
frontend/packages/ceph-storage-plugin/integration-tests/tests/ocp-tests/installFlow.scenario.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/integration-tests/tests/ocp-tests/installFlow.scenario.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/integration-tests/tests/ocp-tests/installFlow.scenario.ts
Outdated
Show resolved
Hide resolved
de836be
to
9096996
Compare
/retest |
1 similar comment
/retest |
9096996
to
bbbea01
Compare
/retest |
export const hasOCSTaint = (node) => { | ||
const taints = node?.spec?.taints || []; | ||
return taints.some((taint) => _.isEqual(taint, ocsTaint)); | ||
}; |
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.
Why you have defined the functions again here? You can reuse the function from node-list.tsx
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.
You can't pull that code into e2e tests and vice versa,
/retest |
fc03bca
to
c2542c6
Compare
/retest |
c2542c6
to
cfcf303
Compare
/approve |
/retest |
2 similar comments
/retest |
/retest |
cfcf303
to
0fbc4a1
Compare
/approve |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bipuladh, gnehapk, umangachapagain The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
/hold |
/hold cancel |
/hold |
/retest |
/hold cancel |
These tests are targetted to run on OCP CI (e2e-gcp-console)