From 6d128ca9ade783ba81538ac7becac26dd8b9f555 Mon Sep 17 00:00:00 2001 From: Ryan Johnson Date: Fri, 15 May 2026 09:28:57 -0400 Subject: [PATCH] chore: improve test error messages Replaced generic formatted messages that referenced the res variable with explicit messages like "expected resource pool, but returned nil" to make test failures easier to understand. Signed-off-by: Ryan Johnson --- builder/vsphere/driver/resource_pool_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/builder/vsphere/driver/resource_pool_test.go b/builder/vsphere/driver/resource_pool_test.go index 8ebd6a85..b4fa04c8 100644 --- a/builder/vsphere/driver/resource_pool_test.go +++ b/builder/vsphere/driver/resource_pool_test.go @@ -100,7 +100,7 @@ func TestVCenterDriver_FindResourcePoolStandaloneESX(t *testing.T) { t.Fatalf("unexpected error: '%s'", err) } if res == nil { - t.Fatalf("unexpected result: expected '%v', but returned 'nil'", res) + t.Fatalf("unexpected result: expected resource pool, but returned nil") } expectedResourcePool := "Resources" if res.pool.Name() != expectedResourcePool { @@ -113,7 +113,7 @@ func TestVCenterDriver_FindResourcePoolStandaloneESX(t *testing.T) { t.Fatalf("unexpected error: '%s'", err) } if res == nil { - t.Fatalf("unexpected result: expected '%v', but returned 'nil'", res) + t.Fatalf("unexpected result: expected non-nil resource pool, but returned nil") } if res.pool.Name() != expectedResourcePool { t.Fatalf("unexpected result: expected '%s', but returned '%s'", expectedResourcePool, res.pool.Name())