Add tests for instance provider

[engedaam]

Tell us about your request

As the instance provider has been moved into its own package. The package is lacking in testing, and the package should have its own unit tests.

Add tests for instance provider packages.

Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?

N/A

Are you currently working around this issue?

N/A

Additional Context

No response

Attachments

No response

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

[prabhjotsingh18]

Hey @engedaam, Can I work on this? I'm new to Golang so this will help me improve my go skills, can you guide me a little on where can I get started that would be really helpful.

[engedaam]

@prabhjotsingh18 That would be great! The main idea of the issue is to add more suite_test.go. The other providers (i.e. subnet, pricing, SecurityGroups, etc.) can be a good reference point. Function like Link, Get, Create, List, etc. all need testing. I would recommend studying the other providers tests first, as your starting point. Let me know if you need any support!

[billrayburn]

@prabhjotsingh18 should we keep you assigned to this issue?