fix: Allow for case-insensitve matching of products

warrensbox · Jun 8, 2024 · 7d2a673 · 7d2a673
1 parent e25a6bc
commit 7d2a673
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/lib/product_test.go b/lib/product_test.go
@@ -24,6 +24,16 @@ func Test_GetProductById(t *testing.T) {
 		}
 	}
 
+	// Test case-insensitve match
+	product = GetProductById("oPeNtOfU")
+	if product == nil {
+		t.Errorf("Terraform product returned nil")
+	} else {
+		if expected := "opentofu"; product.GetId() != expected {
+			t.Errorf("Product ID does not match expected Id. Expected: %q, actual: %q", expected, product.GetId())
+		}
+	}
+
 	product = GetProductById("doesnotexist")
 	if product != nil {
 		t.Errorf("Unknown product returned non-nil response")

diff --git a/lib/products.go b/lib/products.go
@@ -149,7 +149,7 @@ var products = []Product{
 
 func GetProductById(id string) Product {
 	for _, product := range products {
-		if product.GetId() == id {
+		if strings.EqualFold(product.GetId(), id) {
 			return product
 		}
 	}