-
Notifications
You must be signed in to change notification settings - Fork 61
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
Test: Adding integration tests for cv_task_v3 #631
Conversation
35e5544
to
335a040
Compare
Can we follow the same format as the other molecule test and put the tests in a separate playbook and import them in |
- "{{MOVE_DEVICE_DATA_EXECUTE.devices_moved.taskIds[0]}}" | ||
register: EXECUTE_TASK_DATA | ||
|
||
- name: Testing Execute Task |
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.
Can we get the task facts from CVP and check if they got executed? and same for cancelled example
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.
Will complete this once PR #632 gets merged
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.
Fixed!
Fixed! |
5f5f921
to
12bcc1a
Compare
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.
tested in ATD, lgtm!
This reverts commit 080bc78.
Change Summary
Adding integration tests for cv_task_v3
Related Issue(s)
Fixes #
Component(s) name
arista.cvp.cv_task_v3
How to test
molecule converge -s cv_task_v3
Checklist
User Checklist
Repository Checklist