chore: add status code to error message

docs/resources/project.md

@@ -0,0 +1,36 @@
+---
+# generated by https://github.com/hashicorp/terraform-plugin-docs
+page_title: "supabase_project Resource - terraform-provider-supabase"
+subcategory: ""
+description: |-
+  Project resource
+---
+
+# supabase_project (Resource)
+
+Project resource
+
+## Example Usage
+
+```terraform
+resource "supabase_project" "test" {
+  organization_id   = "continued-brown-smelt"
+  name              = "foo"
+  database_password = "bar"
+  region            = "us-east-1"
+}
+```
+
+<!-- schema generated by tfplugindocs -->
+## Schema
+
+### Required
+
+- `database_password` (String, Sensitive) Password for the project database
+- `name` (String) Name of the project
+- `organization_id` (String) Reference to the organization
+- `region` (String) Region where the project is located
+
+### Read-Only
+
+- `id` (String) Project identifier

internal/provider/branch_data_source.go

@@ -106,11 +106,13 @@ func (d *BranchDataSource) Read(ctx context.Context, req datasource.ReadRequest,
 	// provider client data and make a call using it.
 	httpResp, err := d.client.GetBranchesWithResponse(ctx, projectRef.ValueString())
 	if err != nil {
-		resp.Diagnostics.AddError("Client Error", fmt.Sprintf("Unable to read branch, got error: %s", err))
+		msg := fmt.Sprintf("Unable to read branch, got error: %s", err)
+		resp.Diagnostics.AddError("Client Error", msg)
 		return
 	}
 	if httpResp.JSON200 == nil {
-		resp.Diagnostics.AddError("Client Error", fmt.Sprintf("Unable to read branch, got error: %s", httpResp.Body))
+		msg := fmt.Sprintf("Unable to read branch, got status %d: %s", httpResp.StatusCode(), httpResp.Body)
+		resp.Diagnostics.AddError("Client Error", msg)
 		return
 	}
 

internal/provider/project_resource.go

@@ -152,6 +152,7 @@ func (r *ProjectResource) Update(ctx context.Context, req resource.UpdateRequest
 		return
 	}
 
+	// TODO: allow api to update project resource
 	msg := fmt.Sprintf("Update is not supported for project resource: %s", data.Id.ValueString())
 	resp.Diagnostics.Append(diag.NewErrorDiagnostic("Client Error", msg))
 }

internal/provider/settings_resource.go

@@ -219,18 +219,21 @@ func readApiConfig(ctx context.Context, data *SettingsResourceModel, client *api
 		return diag.Diagnostics{diag.NewErrorDiagnostic("Client Error", msg)}
 	}
 	// Deleted project is an orphan resource, not returning error so it can be destroyed.
-	if httpResp.StatusCode() == http.StatusNotFound {
+	switch httpResp.StatusCode() {
+	case http.StatusNotFound, http.StatusNotAcceptable:
 		return nil
+	default:
+		break
 	}
 	if httpResp.JSON200 == nil {
-		msg := fmt.Sprintf("Unable to read api settings, got error: %s", httpResp.Body)
+		msg := fmt.Sprintf("Unable to read api settings, got status %d: %s", httpResp.StatusCode(), httpResp.Body)
 		return diag.Diagnostics{diag.NewErrorDiagnostic("Client Error", msg)}
 	}
 
 	httpResp.JSON200.JwtSecret = nil
 	value, err := json.Marshal(*httpResp.JSON200)
 	if err != nil {
-		msg := fmt.Sprintf("Unable to read api settings, got error: %s", err)
+		msg := fmt.Sprintf("Unable to read api settings, got marshal error: %s", err)
 		return diag.Diagnostics{diag.NewErrorDiagnostic("Client Error", msg)}
 	}
 
@@ -250,13 +253,13 @@ func updateApiConfig(ctx context.Context, data *SettingsResourceModel, client *a
 		return diag.Diagnostics{diag.NewErrorDiagnostic("Client Error", msg)}
 	}
 	if httpResp.JSON200 == nil {
-		msg := fmt.Sprintf("Unable to update api settings, got error: %s", httpResp.Body)
+		msg := fmt.Sprintf("Unable to update api settings, got status %d: %s", httpResp.StatusCode(), httpResp.Body)
 		return diag.Diagnostics{diag.NewErrorDiagnostic("Client Error", msg)}
 	}
 
 	value, err := json.Marshal(*httpResp.JSON200)
 	if err != nil {
-		msg := fmt.Sprintf("Unable to update api settings, got error: %s", err)
+		msg := fmt.Sprintf("Unable to update api settings, got marshal error: %s", err)
 		return diag.Diagnostics{diag.NewErrorDiagnostic("Client Error", msg)}
 	}