Add some unit tests

[vishesh92]

No description provided.

[Copilot]

Pull request overview

This PR adds comprehensive unit tests for protocol handling, load balancer utilities, and instance management in the CloudStack cloud provider. The tests cover previously untested utility functions and method implementations.

• Adds table-driven tests following Go best practices for all test functions
• Tests cover edge cases including nil inputs, empty slices, invalid values, and boundary conditions
• All tests include descriptive test case names and comprehensive assertions

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
protocol_test.go	Tests for LoadBalancerProtocol methods (CSProtocol, IPProtocol, String) and protocol conversion functions (ProtocolFromLoadBalancer, ProtocolFromServicePort)
cloudstack_loadbalancer_test.go	Tests for utility functions: compareStringSlice, symmetricDifference, isFirewallSupported, isNetworkACLSupported, and annotation helper functions
cloudstack_instances_test.go	Tests for nodeAddresses method covering various VM configurations with different IP and hostname combinations

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov-commenter]

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.