test: added unit test for validateNetworkConfig

[smritidahal653]

added unit test coverage for validateNetworkConfig in aci_network.go

Test case break down:

Vnet name

• Provider Vnet name is set but environment variable is not: tc2
• Provider Vnet name is not set but environment variable is: tc3
• Both provider Vnet name and environment variable is set: tc6
• Neither provider Vnet name nor environment variable is set: tc1

Vnet resource group

• Provider Vnet resource group is set but environment variable is not: tc2
• Provider Vnet resource group is not set but environment variable is: tc4
• Both provider Vnet resource group and environment variable is set: tc6
• Neither provider Vnet resource group nor environment variable is set: tc3

Subnet name

• Provider Subnet name is set but environment variable is not: tc2
• Provider Subnet name is not set but environment variable is
  -- malformed: tc7
  --valid: tc8
• Both provider Subnet name and environment variable is set: tc6
• Neither provider Subnet name nor environment variable is set: tc5

[codecov]

Codecov Report

Merging #537 (0af5450) into master (4f3dc80) will increase coverage by 1.40%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #537      +/-   ##
==========================================
+ Coverage   66.04%   67.45%   +1.40%     
==========================================
  Files          15       15              
  Lines        2486     2486              
==========================================
+ Hits         1642     1677      +35     
+ Misses        728      693      -35     
  Partials      116      116              

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[helayoty]

Thanks for redesigning the test case. It hits all the corner cases now. LGTM 🚀