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
Bug 1801142: Vmi dashboard tests #4265
Bug 1801142: Vmi dashboard tests #4265
Conversation
fe2bb0c
to
be48fe8
Compare
/test analyze |
/test e2e-gcp-console |
be48fe8
to
e824568
Compare
import { TAB, VM_STATUS, NOT_AVAILABLE } from './utils/consts'; | ||
import { NodePortService } from './utils/types'; | ||
|
||
describe('Test VM overview', () => { |
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.
VMI
).not.toEqual(NOT_AVAILABLE); | ||
}); | ||
|
||
it('Check vm services', async () => { |
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.
VMI
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.
thanks !
fixed 👍
ac77087
to
2e7023e
Compare
/test frontend |
@yaacov: This pull request references Bugzilla bug 1801142, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@yaacov: This pull request references Bugzilla bug 1801142, which is valid. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
2e7023e
to
1982807
Compare
/test e2e-gcp-console |
1982807
to
324558e
Compare
/test e2e-gcp-console |
import { KubevirtDetailView } from './kubevirtDetailView'; | ||
import { ImportWizard } from './importWizard'; | ||
|
||
const confirmDialogActions = [ | ||
VM_ACTION.Clone, | ||
VM_ACTION.Start, |
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.
Start is not to be confirmed anymore
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.
👍
/test e2e-gcp-console |
/test analyze |
import { TAB, VM_STATUS, NOT_AVAILABLE } from './utils/consts'; | ||
import { NodePortService } from './utils/types'; | ||
|
||
describe('Test VMI overview', () => { |
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.
Probably would be better to change overview to Details, so that it's not confusing
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.
dnoe
p.s.
do we need to rename the files ?
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.
@yaacov I think it would be better to align file names with the current naming, if you don't mind :)
await isLoaded(); | ||
}); | ||
|
||
it('Check VMI details in overview', async () => { |
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.
'Check VMI data in Details'?
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.
done
/test e2e-gcp-console |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rhrazdil, yaacov 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 |
/test e2e-gcp-console |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
@yaacov: All pull requests linked via external trackers have merged. Bugzilla bug 1801142 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
depend on #4209A test to check VMI dashboard page.