Revert unrelated provider_test.go changes to keep PR focused

   Remove the OPENAI_ORGANIZATION_ID optional changes from provider_test.go
   as they are not related to the retry logic fix for issue #35.
   This change should be addressed in a separate PR if needed.

   Addresses feedback from @Fodoj in PR #37

Author: pabloinigo

internal/provider/provider_test.go

@@ -29,11 +29,8 @@ func testAccPreCheck(t *testing.T) {
 	if v := os.Getenv("OPENAI_API_KEY"); v == "" {
 		t.Fatal("OPENAI_API_KEY must be set for acceptance tests")
 	}
-	// OPENAI_ORGANIZATION_ID is optional - some users only have personal accounts
-	if v := os.Getenv("OPENAI_ORGANIZATION_ID"); v != "" {
-		t.Logf("Using organization ID: %s", v)
-	} else {
-		t.Log("No organization ID set - using personal account")
+	if v := os.Getenv("OPENAI_ORGANIZATION_ID"); v == "" {
+		t.Fatal("OPENAI_ORGANIZATION_ID must be set for acceptance tests")
 	}
 }