feat: add compile time checks for config validator interface

juju · Sep 12, 2024 · 902d587 · 902d587
1 parent 635571d
commit 902d587
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 0 deletions.
diff --git a/internal/provider/resource_access_jaas_model.go b/internal/provider/resource_access_jaas_model.go
@@ -19,6 +19,7 @@ import (
 // Ensure provider defined types fully satisfy framework interfaces.
 var _ resource.Resource = &jaasAccessModelResource{}
 var _ resource.ResourceWithConfigure = &jaasAccessModelResource{}
+var _ resource.ResourceWithConfigValidators = &jaasAccessModelResource{}
 
 // NewJAASAccessModelResource returns a new resource for JAAS model access.
 func NewJAASAccessModelResource() resource.Resource {

diff --git a/internal/provider/resource_access_model.go b/internal/provider/resource_access_model.go
@@ -27,6 +27,7 @@ import (
 var _ resource.Resource = &accessModelResource{}
 var _ resource.ResourceWithConfigure = &accessModelResource{}
 var _ resource.ResourceWithImportState = &accessModelResource{}
+var _ resource.ResourceWithConfigValidators = &accessModelResource{}
 
 func NewAccessModelResource() resource.Resource {
 	return &accessModelResource{}

diff --git a/internal/provider/resource_user.go b/internal/provider/resource_user.go
@@ -24,6 +24,7 @@ import (
 var _ resource.Resource = &userResource{}
 var _ resource.ResourceWithConfigure = &userResource{}
 var _ resource.ResourceWithImportState = &userResource{}
+var _ resource.ResourceWithConfigValidators = &userResource{}
 
 func NewUserResource() resource.Resource {
 	return &userResource{}

diff --git a/internal/provider/validator_require_jaas.go b/internal/provider/validator_require_jaas.go
@@ -21,6 +21,8 @@ type RequiresJAASValidator struct {
 	client *juju.Client
 }
 
+// NewRequiresJAASValidator returns a new validator that enforces a resource can
+// only be created against JAAS.
 func NewRequiresJAASValidator(client *juju.Client) RequiresJAASValidator {
 	return RequiresJAASValidator{
 		client: client,