feat(saml_connection): add field

clerk · Sep 26, 2024 · 741670a · 741670a
1 parent 1f2556d
commit 741670a
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/saml_connection.go b/saml_connection.go
@@ -7,6 +7,7 @@ type SAMLConnection struct {
 	Name                             string                         `json:"name"`
 	Domain                           string                         `json:"domain"`
 	IdpEntityID                      *string                        `json:"idp_entity_id"`
+	OrganizationID                   *string                        `json:"organization_id"`
 	IdpSsoURL                        *string                        `json:"idp_sso_url"`
 	IdpCertificate                   *string                        `json:"idp_certificate"`
 	IdpMetadataURL                   *string                        `json:"idp_metadata_url"`

diff --git a/samlconnection/client.go b/samlconnection/client.go
@@ -34,6 +34,7 @@ type AttributeMappingParams struct {
 type CreateParams struct {
 	clerk.APIParams
 	Name             *string                 `json:"name,omitempty"`
+	OrganizationID   *string                 `json:"organization_id,omitempty"`
 	Domain           *string                 `json:"domain,omitempty"`
 	Provider         *string                 `json:"provider,omitempty"`
 	IdpEntityID      *string                 `json:"idp_entity_id,omitempty"`
@@ -67,9 +68,14 @@ func (c *Client) Get(ctx context.Context, id string) (*clerk.SAMLConnection, err
 
 type UpdateParams struct {
 	clerk.APIParams
-	Name                             *string                 `json:"name,omitempty"`
-	Domain                           *string                 `json:"domain,omitempty"`
-	IdpEntityID                      *string                 `json:"idp_entity_id,omitempty"`
+	Name        *string `json:"name,omitempty"`
+	Domain      *string `json:"domain,omitempty"`
+	IdpEntityID *string `json:"idp_entity_id,omitempty"`
+	// OrganizationID is a nullable optional field.
+	// - If nil or unset, the backend will not take any action.
+	// - If an empty value (""), the backend will unset the organization_id.
+	// - If a valid ID is provided, the backend will update the organization_id.
+	OrganizationID                   *string                 `json:"organization_id,omitempty"`
 	IdpSsoURL                        *string                 `json:"idp_sso_url,omitempty"`
 	IdpCertificate                   *string                 `json:"idp_certificate,omitempty"`
 	IdpMetadataURL                   *string                 `json:"idp_metadata_url,omitempty"`

diff --git a/samlconnection/client_test.go b/samlconnection/client_test.go
@@ -103,7 +103,7 @@ func TestSAMLConnectionClientUpdate(t *testing.T) {
 	config.HTTPClient = &http.Client{
 		Transport: &clerktest.RoundTripper{
 			T:      t,
-			In:     json.RawMessage(fmt.Sprintf(`{"name":"%s", "disable_additional_identifications": %t}`, name, disableAdditionalIdentifications)),
+			In:     json.RawMessage(fmt.Sprintf(`{"name":"%s", "disable_additional_identifications": %t, "organization_id": ""}`, name, disableAdditionalIdentifications)),
 			Out:    json.RawMessage(fmt.Sprintf(`{"id":"%s","name":"%s","domain":"%s","provider":"%s","disable_additional_identifications": %t}`, id, name, domain, provider, disableAdditionalIdentifications)),
 			Method: http.MethodPatch,
 			Path:   "/v1/saml_connections/" + id,
@@ -113,6 +113,7 @@ func TestSAMLConnectionClientUpdate(t *testing.T) {
 	samlConnection, err := client.Update(context.Background(), id, &UpdateParams{
 		Name:                             clerk.String(name),
 		DisableAdditionalIdentifications: clerk.Bool(disableAdditionalIdentifications),
+		OrganizationID:                   clerk.String(""),
 	})
 	require.NoError(t, err)
 	require.Equal(t, id, samlConnection.ID)