chore: test debt

Signed-off-by: Ryan Johnson <[email protected]>

builder/vsphere/common/testing/utility.go

@@ -13,17 +13,21 @@ import (
 	"github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/driver"
 )
 
+const (
+	testDefaultUsername    = "[email protected]"
+	testDefaultPassword    = "VMw@re1!"
+	testDefaultApiEndpoint = "vcenter.example.com"
+)
+
 func NewVMName() string {
-	rand.Seed(time.Now().UnixNano())
-	return fmt.Sprintf("test-%v", rand.Intn(1000))
+	r := rand.New(rand.NewSource(time.Now().UnixNano()))
+	return fmt.Sprintf("test-%v", r.Intn(1000))
 }
 
 func RenderConfig(builderType string, config map[string]interface{}) string {
 	t := map[string][]map[string]interface{}{
 		"builders": {
-			map[string]interface{}{
-				"type": builderType,
-			},
+			{"type": builderType},
 		},
 	}
 	for k, v := range config {
@@ -37,21 +41,27 @@ func RenderConfig(builderType string, config map[string]interface{}) string {
 func TestConn() (driver.Driver, error) {
 	username := os.Getenv("VSPHERE_USERNAME")
 	if username == "" {
-		username = "root"
+		username = testDefaultUsername
 	}
+
 	password := os.Getenv("VSPHERE_PASSWORD")
 	if password == "" {
-		password = "jetbrains"
+		password = testDefaultPassword
+	}
+
+	vcenter := os.Getenv("VSPHERE_VCENTER_SERVER")
+	if vcenter == "" {
+		vcenter = testDefaultApiEndpoint
 	}
 
 	d, err := driver.NewDriver(&driver.ConnectConfig{
-		VCenterServer:      "vcenter.example.com",
+		VCenterServer:      vcenter,
 		Username:           username,
 		Password:           password,
 		InsecureConnection: true,
 	})
 	if err != nil {
-		return nil, fmt.Errorf("error connecting to vCenter Server instance: %v", err)
+		return nil, fmt.Errorf("error connecting to endpoint: %v", err)
 	}
 	return d, nil
 }
@@ -61,7 +71,6 @@ func GetVM(d driver.Driver, name string) (driver.VirtualMachine, error) {
 	if err != nil {
 		return nil, fmt.Errorf("error finding virtual machine: %v", err)
 	}
-
 	return vm, nil
 }
 

builder/vsphere/driver/driver_test.go

@@ -23,21 +23,27 @@ import (
 	"github.com/vmware/govmomi/vim25/soap"
 )
 
-// Defines whether acceptance tests should be run
-const TestHostName = "esxi-01.example.com"
+const (
+	testDefaultUsername    = "[email protected]"
+	testDefaultPassword    = "VMw@re1!"
+	testDefaultApiEndpoint = "vcenter.example.com"
+	testDefaultHost        = "esxi-01.example.com"
+)
 
-func newTestDriver(t *testing.T) Driver {
-	username := os.Getenv("VSPHERE_USERNAME")
-	if username == "" {
-		username = "root"
-	}
-	password := os.Getenv("VSPHERE_PASSWORD")
-	if password == "" {
-		password = "jetbrains"
+func getEnv(key, fallback string) string {
+	if value := os.Getenv(key); value != "" {
+		return value
 	}
+	return fallback
+}
+
+func newTestDriver(t *testing.T) Driver {
+	username := getEnv("VSPHERE_USERNAME", testDefaultUsername)
+	password := getEnv("VSPHERE_PASSWORD", testDefaultPassword)
+	vcenter := getEnv("VSPHERE_VCENTER_SERVER", testDefaultApiEndpoint)
 
 	d, err := NewDriver(&ConnectConfig{
-		VCenterServer:      "vcenter.example.com",
+		VCenterServer:      vcenter,
 		Username:           username,
 		Password:           password,
 		InsecureConnection: true,
@@ -49,8 +55,8 @@ func newTestDriver(t *testing.T) Driver {
 }
 
 func newVMName() string {
-	rand.Seed(time.Now().UTC().UnixNano())
-	return fmt.Sprintf("test-%v", rand.Intn(1000))
+	r := rand.New(rand.NewSource(time.Now().UTC().UnixNano()))
+	return fmt.Sprintf("test-%v", r.Intn(1000))
 }
 
 type VCenterSimulator struct {
@@ -94,23 +100,23 @@ func (s *VCenterSimulator) Close() {
 	}
 }
 
-// Simulator shortcut to choose any pre created VM.
+// Simulator shortcut to choose any pre-created virtual machine.
 func (s *VCenterSimulator) ChooseSimulatorPreCreatedVM() (VirtualMachine, *simulator.VirtualMachine) {
 	machine := simulator.Map.Any("VirtualMachine").(*simulator.VirtualMachine)
 	ref := machine.Reference()
 	vm := s.driver.NewVM(&ref)
 	return vm, machine
 }
 
-// Simulator shortcut to choose any pre created Datastore.
+// Simulator shortcut to choose any pre-created datastore.
 func (s *VCenterSimulator) ChooseSimulatorPreCreatedDatastore() (Datastore, *simulator.Datastore) {
 	ds := simulator.Map.Any("Datastore").(*simulator.Datastore)
 	ref := ds.Reference()
 	datastore := s.driver.NewDatastore(&ref)
 	return datastore, ds
 }
 
-// Simulator shortcut to choose any pre created Host.
+// Simulator shortcut to choose any pre-created EXSi host.
 func (s *VCenterSimulator) ChooseSimulatorPreCreatedHost() (*Host, *simulator.HostSystem) {
 	h := simulator.Map.Any("HostSystem").(*simulator.HostSystem)
 	ref := h.Reference()

builder/vsphere/driver/host_acc_test.go

@@ -10,7 +10,7 @@ import (
 func TestHostAcc(t *testing.T) {
 	t.Skip("Acceptance tests not configured yet.")
 	d := newTestDriver(t)
-	host, err := d.FindHost(TestHostName)
+	host, err := d.FindHost(testDefaultHost)
 	if err != nil {
 		t.Fatalf("unexpected error: '%s'", err)
 	}
@@ -19,7 +19,7 @@ func TestHostAcc(t *testing.T) {
 	if err != nil {
 		t.Fatalf("unexpected error: '%s'", err)
 	}
-	if info.Name != TestHostName {
-		t.Errorf("unexpected result: expected '%s', but returned '%s'", TestHostName, info.Name)
+	if info.Name != testDefaultHost {
+		t.Errorf("unexpected result: expected '%s', but returned '%s'", testDefaultHost, info.Name)
 	}
 }

builder/vsphere/driver/vm_clone_acc_test.go

@@ -32,7 +32,7 @@ func TestVMAcc_clone(t *testing.T) {
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			tc.config.Host = TestHostName
+			tc.config.Host = testDefaultHost
 			tc.config.Name = newVMName()
 
 			templateName := "alpine"
@@ -88,8 +88,8 @@ func cloneDefaultCheck(t *testing.T, vm VirtualMachine, config *CloneConfig) {
 	if err != nil {
 		t.Fatal("Cannot read host properties: ", err)
 	}
-	if hostInfo.Name != TestHostName {
-		t.Errorf("Invalid host name: expected '%v', got '%v'", TestHostName, hostInfo.Name)
+	if hostInfo.Name != testDefaultHost {
+		t.Errorf("Invalid host name: expected '%v', got '%v'", testDefaultHost, hostInfo.Name)
 	}
 
 	p := d.NewResourcePool(vmInfo.ResourcePool)

builder/vsphere/driver/vm_create_acc_test.go

@@ -20,7 +20,7 @@ func TestVMAcc_create(t *testing.T) {
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			tc.config.Host = TestHostName
+			tc.config.Host = testDefaultHost
 			tc.config.Name = newVMName()
 
 			d := newTestDriver(t)
@@ -71,8 +71,8 @@ func createDefaultCheck(t *testing.T, vm VirtualMachine, config *CreateConfig) {
 	if err != nil {
 		t.Fatal("Cannot read host properties: ", err)
 	}
-	if hostInfo.Name != TestHostName {
-		t.Errorf("Invalid host name: expected '%v', got '%v'", TestHostName, hostInfo.Name)
+	if hostInfo.Name != testDefaultHost {
+		t.Errorf("Invalid host name: expected '%v', got '%v'", testDefaultHost, hostInfo.Name)
 	}
 
 	p := d.NewResourcePool(vmInfo.ResourcePool)

builder/vsphere/iso/builder_acc_test.go

@@ -14,6 +14,13 @@ import (
 	"github.com/vmware/govmomi/vim25/types"
 )
 
+const (
+	defaultUsername    = "[email protected]"
+	defaultPassword    = "VMw@re1!"
+	defaultApiEndpoint = "vcenter.example.com"
+	defaultHost        = "esxi-01.example.com"
+)
+
 func TestAccISOBuilderAcc_default(t *testing.T) {
 	config := defaultConfig()
 	testCase := &acctest.PluginTestCase{
@@ -38,25 +45,18 @@ func TestAccISOBuilderAcc_default(t *testing.T) {
 	acctest.TestPlugin(t, testCase)
 }
 
-func defaultConfig() map[string]interface{} {
-	username := os.Getenv("VSPHERE_USERNAME")
-	if username == "" {
-		username = "root"
-	}
-	password := os.Getenv("VSPHERE_PASSWORD")
-	if password == "" {
-		password = "jetbrains"
-	}
-
-	vcenter := os.Getenv("VSPHERE_VCENTER_SERVER")
-	if vcenter == "" {
-		vcenter = "vcenter.example.com"
+func getEnv(key, fallback string) string {
+	if value := os.Getenv(key); value != "" {
+		return value
 	}
+	return fallback
+}
 
-	host := os.Getenv("VSPHERE_HOST")
-	if host == "" {
-		host = "esxi-01.example.com"
-	}
+func defaultConfig() map[string]interface{} {
+	username := getEnv("VSPHERE_USERNAME", defaultUsername)
+	password := getEnv("VSPHERE_PASSWORD", defaultPassword)
+	vcenter := getEnv("VSPHERE_VCENTER_SERVER", defaultApiEndpoint)
+	host := getEnv("VSPHERE_HOST", defaultHost)
 
 	config := map[string]interface{}{
 		"vcenter_server":      vcenter,
@@ -65,8 +65,8 @@ func defaultConfig() map[string]interface{} {
 		"host":                host,
 		"insecure_connection": true,
 
-		"ssh_username": "root",
-		"ssh_password": "jetbrains",
+		"ssh_username": "packer",
+		"ssh_password": "VMw@re1!",
 
 		"vm_name": commonT.NewVMName(),
 		"storage": map[string]interface{}{
@@ -546,21 +546,32 @@ func TestAccISOBuilderAcc_full(t *testing.T) {
 func fullConfig() map[string]interface{} {
 	username := os.Getenv("VSPHERE_USERNAME")
 	if username == "" {
-		username = "root"
+		username = "[email protected]"
 	}
+
 	password := os.Getenv("VSPHERE_PASSWORD")
 	if password == "" {
-		password = "jetbrains"
+		password = "VMw@re1!"
+	}
+
+	vcenter := os.Getenv("VSPHERE_VCENTER_SERVER")
+	if vcenter == "" {
+		vcenter = "vcenter.example.com"
+	}
+
+	host := os.Getenv("VSPHERE_HOST")
+	if host == "" {
+		host = "esxi-01.example.com"
 	}
 
 	config := map[string]interface{}{
-		"vcenter_server":      "vcenter.example.com",
+		"vcenter_server":      vcenter,
 		"username":            username,
 		"password":            password,
+		"host":                host,
 		"insecure_connection": true,
 
 		"vm_name": commonT.NewVMName(),
-		"host":    "esxi-01.example.com",
 
 		"RAM": 512,
 		"disk_controller_type": []string{
@@ -602,8 +613,8 @@ func fullConfig() map[string]interface{} {
 			"/media/floppy/SETUP.SH<enter>",
 		},
 
-		"ssh_username": "root",
-		"ssh_password": "jetbrains",
+		"ssh_username": "packer",
+		"ssh_password": "VMw@re1!",
 	}
 
 	return config

examples/builder/vsphere-clone/alpine/alpine.json

@@ -4,8 +4,8 @@
       "type": "vsphere-clone",
 
       "vcenter_server": "vcenter.example.com",
-      "username":            "root",
-      "password":            "jetbrains",
+      "username": "[email protected]",
+      "password": "VMw@re1!",
       "insecure_connection": "true",
 
       "template": "alpine",

examples/builder/vsphere-clone/alpine/alpine.pkr.hcl

@@ -12,9 +12,9 @@ source "vsphere-clone" "example_clone" {
   communicator        = "none"
   host                = "esxi-01.example.com"
   insecure_connection = "true"
-  password            = "jetbrains"
+  password            = "VMw@re1!"
   template            = "alpine"
-  username            = "root"
+  username            = "[email protected]"
   vcenter_server      = "vcenter.example.com"
   vm_name             = "alpine-clone-${local.timestamp}"
 }

examples/builder/vsphere-iso/alpine/alpine-3.8.json

@@ -39,16 +39,16 @@
           "network_card": "vmxnet3"
         }
       ],
-      "password": "jetbrains",
-      "ssh_password": "jetbrains",
-      "ssh_username": "root",
+      "password": "VMw@re1!",
+      "ssh_password": "VMw@re1!",
+      "ssh_username": "packer",
       "storage": [
         {
           "disk_size": 1024,
           "disk_thin_provisioned": true
         }
       ],
-      "username": "root",
+      "username": "[email protected]",
       "vcenter_server": "vcenter.example.com",
       "vm_name": "alpine-{{timestamp}}"
     }