fix(org): make the organisation parameter optional

docs/index.md

@@ -15,12 +15,9 @@ CleverCloud provider allow you to interact with CleverCloud platform.
 <!-- schema generated by tfplugindocs -->
 ## Schema
 
-### Required
-
-- `organisation` (String, Sensitive) CleverCloud organisation, can be either orga_xxx, or user_xxx for personal spaces. This parameter can also be provided via CC_ORGANISATION environment variable.
-
 ### Optional
 
 - `endpoint` (String) CleverCloud API endpoint, default to https://api.clever-cloud.com
+- `organisation` (String, Sensitive) CleverCloud organisation, can be either orga_xxx, or user_xxx for personal spaces. This parameter can also be provided via CC_ORGANISATION environment variable.
 - `secret` (String, Sensitive) CleverCloud OAuth1 secret, can be took from clever-tools after login. This parameter can also be provided via CC_OAUTH_SECRET environment variable.
 - `token` (String, Sensitive) CleverCloud OAuth1 token, can be took from clever-tools after login. This parameter can also be provided via CC_OAUTH_TOKEN environment variable.

pkg/provider/impl/provider_configure.go

@@ -26,6 +26,10 @@ func (p *Provider) Configure(ctx context.Context, req provider.ConfigureRequest,
 	} else {
 		p.organization = config.Organisation.ValueString()
 	}
+	if p.organization == "" {
+		resp.Diagnostics.AddError("Invalid provider configuration", "Organisation should be set by either the organisation parameter or by the CC_ORGANISATION environment variable")
+		return
+	}
 
 	// Allow to get creds from CLI config directory or by injected variables
 	if config.Secret.IsUnknown() ||

pkg/provider/impl/provider_configure_test.go

@@ -0,0 +1,37 @@
+package impl
+
+import (
+	"github.com/hashicorp/terraform-plugin-framework/providerserver"
+	"github.com/hashicorp/terraform-plugin-go/tfprotov6"
+	"github.com/hashicorp/terraform-plugin-testing/helper/resource"
+	"regexp"
+	"testing"
+)
+
+var protoV6Provider = map[string]func() (tfprotov6.ProviderServer, error){
+	"clevercloud": providerserver.NewProtocol6WithError(New("test")()),
+}
+
+func TestProvider_ConfigureEmptyOrganisation(t *testing.T) {
+	providerBlock := `
+provider "clevercloud" {}
+
+// dummy resource to trigger provider configuration
+resource "clevercloud_cellar" "test" {
+  name = "test"
+}
+`
+	expectedError := regexp.MustCompile("Organisation should be set")
+
+	resource.Test(t, resource.TestCase{
+		IsUnitTest:               true,
+		ProtoV6ProviderFactories: protoV6Provider,
+		Steps: []resource.TestStep{{
+			Config: providerBlock,
+			SkipFunc: func() (bool, error) {
+				return false, nil
+			},
+			ExpectError: expectedError,
+		}},
+	})
+}

pkg/provider/impl/provider_schema.go

@@ -44,7 +44,7 @@ func (p *Provider) Schema(_ context.Context, req provider.SchemaRequest, res *pr
 			},
 			"organisation": schema.StringAttribute{
 				Sensitive:           true,
-				Required:            true,
+				Optional:            true, // can be read from environment variable
 				MarkdownDescription: "CleverCloud organisation, can be either orga_xxx, or user_xxx for personal spaces. This parameter can also be provided via CC_ORGANISATION environment variable.",
 				Validators: []validator.String{
 					pkg.NewValidatorRegex("valid owner name", regexp.MustCompile(`^(user|orga)_.{36}`)),