fix: unit tests for sync volume#3881
Conversation
fa61844 to
ed7ea33
Compare
tenthirtyam
left a comment
There was a problem hiding this comment.
@nikhilbarge - could you sign-off using your @broadcom.com email instead of @vmware.com?
3691ef9 to
1a9bcb2
Compare
done |
There was a problem hiding this comment.
Pull Request Overview
This PR fixes unit tests for sync volume functionality by relocating the CnsSyncVolume API test to an earlier position in the test sequence and adding environment variable setup for test configuration.
- Moved CnsSyncVolume API test from the end of TestClient to an earlier position after volume creation
- Added environment variable setup for CNS configuration including datastore, datacenter, VC URL, and resource pool path
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
1a9bcb2 to
12219dd
Compare
Signed-off-by: Nikhil Barge <nikhil.barge@broadcom.com>
12219dd to
e00e51c
Compare
|
@nikhilbarge A couple tests are failing because you don't have SSH keys. |
thanks for pointing out, i have added ssh key. tests are passed |
Description
Fix unit tests for sync volume
Please include a summary of the change.
updated unit tests
Closes: #(issue-number)
How Has This Been Tested?
Unit tests
logs:
client_test.log
Please describe any manual tests done to verify your changes.
Guidelines
Please read and follow the
CONTRIBUTIONguidelines of this project.