generate input objects, start using Nullable

.changes/unreleased/Refactor-20241227-103359.yaml

@@ -0,0 +1,3 @@
+kind: Refactor
+body: 'BREAKING CHANGE: "Nullable" type now wraps optional "string", "ID", "bool", and timestamp struct fields of API input objects. This "Nullable" type enables fields to be set to the JSON "null" value.'
+time: 2024-12-27T10:33:59.906375-06:00

actions.go

@@ -165,10 +165,10 @@ func (client *Client) CreateTriggerDefinition(input CustomActionsTriggerDefiniti
 		} `graphql:"customActionsTriggerDefinitionCreate(input: $input)"`
 	}
 	if input.AccessControl == nil {
-		input.AccessControl = RefOf(CustomActionsTriggerDefinitionAccessControlEnumEveryone)
+		input.AccessControl = &CustomActionsTriggerDefinitionAccessControlEnumEveryone
 	}
 	if input.EntityType == nil {
-		input.EntityType = RefOf(CustomActionsEntityTypeEnumService)
+		input.EntityType = &CustomActionsEntityTypeEnumService
 	}
 	v := PayloadVariables{
 		"input": input,

actions_test.go

@@ -20,10 +20,12 @@ func TestCreateWebhookAction(t *testing.T) {
 	client := BestTestClient(t, "custom_actions/create_action", testRequest)
 
 	// Act
+	jsonHeaders, err := ol.NewJSON(`{"Content-Type": "application/json"}`)
+	autopilot.Ok(t, err)
 	action, err := client.CreateWebhookAction(ol.CustomActionsWebhookActionCreateInput{
 		Name:           "Deploy Rollback",
-		LiquidTemplate: ol.RefOf("{\"token\": \"XXX\", \"ref\":\"main\", \"action\": \"rollback\"}"),
-		Headers:        &ol.JSON{"Content-Type": "application/json"},
+		LiquidTemplate: ol.NewNullableFrom("{\"token\": \"XXX\", \"ref\":\"main\", \"action\": \"rollback\"}"),
+		Headers:        jsonHeaders,
 		HttpMethod:     ol.CustomActionsHttpMethodEnumPost,
 		WebhookUrl:     "https://gitlab.com/api/v4/projects/1/trigger/pipeline",
 	})
@@ -69,9 +71,10 @@ func TestUpdateWebhookAction(t *testing.T) {
 	client := BestTestClient(t, "custom_actions/update_action", testRequest)
 
 	// Act
+	putHttpMethod := ol.CustomActionsHttpMethodEnumPut
 	action, err := client.UpdateWebhookAction(ol.CustomActionsWebhookActionUpdateInput{
 		Id:         *newID,
-		HttpMethod: ol.RefOf(ol.CustomActionsHttpMethodEnumPut),
+		HttpMethod: &putHttpMethod,
 	})
 
 	// Assert
@@ -90,10 +93,12 @@ func TestUpdateWebhookAction2(t *testing.T) {
 	client := BestTestClient(t, "custom_actions/update_action2", testRequest)
 
 	// Act
+	jsonHeaders, err := ol.NewJSON(`{"Accept": "application/json"}`)
+	autopilot.Ok(t, err)
 	action, err := client.UpdateWebhookAction(ol.CustomActionsWebhookActionUpdateInput{
 		Id:          *newID,
-		Description: ol.RefOf(""),
-		Headers:     &ol.JSON{"Accept": "application/json"},
+		Description: ol.NewNullableFrom(""),
+		Headers:     jsonHeaders,
 	})
 
 	// Assert
@@ -130,13 +135,13 @@ func TestCreateTriggerDefinition(t *testing.T) {
 	// Act
 	trigger, err := client.CreateTriggerDefinition(ol.CustomActionsTriggerDefinitionCreateInput{
 		Name:                   "Deploy Rollback",
-		AccessControl:          ol.RefOf(ol.CustomActionsTriggerDefinitionAccessControlEnumEveryone),
-		Description:            ol.RefOf("Disables the Deploy Freeze"),
-		ResponseTemplate:       ol.RefOf(""),
-		ManualInputsDefinition: ol.RefOf(""),
-		ActionId:               newID,
+		AccessControl:          &ol.CustomActionsTriggerDefinitionAccessControlEnumEveryone,
+		Description:            ol.NewNullableFrom("Disables the Deploy Freeze"),
+		ResponseTemplate:       ol.NewNullableFrom(""),
+		ManualInputsDefinition: ol.NewNullableFrom(""),
+		ActionId:               ol.NewNullableFrom(*newID),
 		OwnerId:                *newID,
-		FilterId:               ol.NewID("987654321"),
+		FilterId:               ol.NewNullableFrom(ol.ID("987654321")),
 	})
 	// Assert
 	autopilot.Ok(t, err)
@@ -155,14 +160,14 @@ func TestCreateTriggerDefinitionWithGlobalEntityType(t *testing.T) {
 	// Act
 	trigger, err := client.CreateTriggerDefinition(ol.CustomActionsTriggerDefinitionCreateInput{
 		Name:                   "Deploy Rollback",
-		AccessControl:          ol.RefOf(ol.CustomActionsTriggerDefinitionAccessControlEnumEveryone),
-		Description:            ol.RefOf("Disables the Deploy Freeze"),
-		ActionId:               newID,
-		ManualInputsDefinition: ol.RefOf(""),
-		ResponseTemplate:       ol.RefOf(""),
+		AccessControl:          &ol.CustomActionsTriggerDefinitionAccessControlEnumEveryone,
+		Description:            ol.NewNullableFrom("Disables the Deploy Freeze"),
+		ActionId:               ol.NewNullableFrom(*newID),
+		ManualInputsDefinition: ol.NewNullableFrom(""),
+		ResponseTemplate:       ol.NewNullableFrom(""),
 		OwnerId:                *newID,
-		FilterId:               ol.NewID("987654321"),
-		EntityType:             ol.RefOf(ol.CustomActionsEntityTypeEnumGlobal),
+		FilterId:               ol.NewNullableFrom(ol.ID("987654321")),
+		EntityType:             &ol.CustomActionsEntityTypeEnumGlobal,
 		ExtendedTeamAccess: &[]ol.IdentifierInput{
 			*ol.NewIdentifier("example_1"),
 			*ol.NewIdentifier("example_1"),
@@ -185,13 +190,13 @@ func TestCreateTriggerDefinitionWithNullExtendedTeams(t *testing.T) {
 	// Act
 	trigger, err := client.CreateTriggerDefinition(ol.CustomActionsTriggerDefinitionCreateInput{
 		Name:                   "Deploy Rollback",
-		Description:            ol.RefOf("Disables the Deploy Freeze"),
-		AccessControl:          ol.RefOf(ol.CustomActionsTriggerDefinitionAccessControlEnumEveryone),
-		ManualInputsDefinition: ol.RefOf(""),
-		ResponseTemplate:       ol.RefOf(""),
-		ActionId:               newID,
+		Description:            ol.NewNullableFrom("Disables the Deploy Freeze"),
+		AccessControl:          &ol.CustomActionsTriggerDefinitionAccessControlEnumEveryone,
+		ManualInputsDefinition: ol.NewNullableFrom(""),
+		ResponseTemplate:       ol.NewNullableFrom(""),
+		ActionId:               ol.NewNullableFrom(*newID),
 		OwnerId:                *newID,
-		FilterId:               ol.NewID("987654321"),
+		FilterId:               ol.NewNullableFrom(ol.ID("987654321")),
 		ExtendedTeamAccess:     &[]ol.IdentifierInput{},
 	})
 	// Assert
@@ -258,7 +263,7 @@ func TestUpdateTriggerDefinition(t *testing.T) {
 	// Act
 	trigger, err := client.UpdateTriggerDefinition(ol.CustomActionsTriggerDefinitionUpdateInput{
 		Id:       *newID,
-		FilterId: ol.NewID(),
+		FilterId: ol.NewNullOf[ol.ID](),
 	})
 	// Assert
 	autopilot.Ok(t, err)
@@ -277,8 +282,8 @@ func TestUpdateTriggerDefinition2(t *testing.T) {
 	// Act
 	trigger, err := client.UpdateTriggerDefinition(ol.CustomActionsTriggerDefinitionUpdateInput{
 		Id:                 *newID,
-		Name:               ol.RefOf("test"),
-		Description:        ol.RefOf(""),
+		Name:               ol.NewNullableFrom("test"),
+		Description:        ol.NewNullableFrom(""),
 		ExtendedTeamAccess: &[]ol.IdentifierInput{},
 	})
 	// Assert
@@ -298,8 +303,8 @@ func TestUpdateTriggerDefinition3(t *testing.T) {
 	// Act
 	trigger, err := client.UpdateTriggerDefinition(ol.CustomActionsTriggerDefinitionUpdateInput{
 		Id:          *newID,
-		Name:        ol.RefOf("test"),
-		Description: ol.RefOf(""),
+		Name:        ol.NewNullableFrom("test"),
+		Description: ol.NewNullableFrom(""),
 		ExtendedTeamAccess: &[]ol.IdentifierInput{
 			*ol.NewIdentifier(string(*newID)),
 			*ol.NewIdentifier(string(id1)),

aliases.go

@@ -49,7 +49,7 @@ func (client *Client) CreateAliases(ownerId ID, aliases []string) ([]string, err
 	for _, alias := range aliases {
 		input := AliasCreateInput{
 			Alias:   alias,
-			OwnerId: ownerId,
+			OwnerId: ID(ownerId),
 		}
 		result, err := client.CreateAlias(input)
 		allErrors = errors.Join(allErrors, err)

category_test.go

@@ -93,7 +93,7 @@ func TestUpdateRubricCategory(t *testing.T) {
 	// Act
 	result, err := client.UpdateCategory(ol.CategoryUpdateInput{
 		Id:   id4,
-		Name: ol.RefOf("Emily"),
+		Name: ol.NewNullableFrom("Emily"),
 	})
 	// Assert
 	autopilot.Ok(t, err)

check.go

@@ -90,14 +90,14 @@ type CheckCreateInputProvider interface {
 }
 
 type CheckCreateInput struct {
-	Name     string        `json:"name" yaml:"name" mapstructure:"name"`
-	Enabled  *bool         `json:"enabled" yaml:"enabled" mapstructure:"enabled"`
-	EnableOn *iso8601.Time `json:"enableOn,omitempty" yaml:"enableOn,omitempty" mapstructure:"enabledOn,omitempty"`
-	Category ID            `json:"categoryId" yaml:"categoryId" mapstructure:"categoryId"`
-	Level    ID            `json:"levelId" yaml:"levelId" mapstructure:"levelId"`
-	Owner    *ID           `json:"ownerId,omitempty" yaml:"ownerId,omitempty" mapstructure:"ownerId,omitempty"`
-	Filter   *ID           `json:"filterId,omitempty" yaml:"filterId,omitempty" mapstructure:"filterId,omitempty"`
-	Notes    *string       `json:"notes" yaml:"notes" default:"Notes on Example Check" mapstructure:"notes"`
+	Category ID                      `json:"categoryId" yaml:"categoryId" mapstructure:"categoryId"`
+	EnableOn *Nullable[iso8601.Time] `json:"enableOn,omitempty" yaml:"enableOn,omitempty" mapstructure:"enabledOn,omitempty"`
+	Enabled  *Nullable[bool]         `json:"enabled,omitempty" yaml:"enabled,omitempty" mapstructure:"enabled"`
+	Filter   *Nullable[ID]           `json:"filterId,omitempty" yaml:"filterId,omitempty" mapstructure:"filterId,omitempty"`
+	Level    ID                      `json:"levelId" yaml:"levelId" mapstructure:"levelId"`
+	Name     string                  `json:"name" yaml:"name" mapstructure:"name"`
+	Notes    *Nullable[string]       `json:"notes,omitempty" yaml:"notes,omitempty" mapstructure:"notes,omitempty"`
+	Owner    *Nullable[ID]           `json:"ownerId,omitempty" yaml:"ownerId,omitempty" mapstructure:"ownerId,omitempty"`
 }
 
 func NewCheckCreateInputTypeOf[T any](checkCreateInput CheckCreateInput) *T {
@@ -113,15 +113,15 @@ type CheckUpdateInputProvider interface {
 }
 
 type CheckUpdateInput struct {
-	Id       ID            `json:"id" mapstructure:"id"`
-	Name     string        `json:"name,omitempty" mapstructure:"name,omitempty"`
-	Enabled  *bool         `json:"enabled,omitempty" mapstructure:"enabled,omitempty"`
-	EnableOn *iso8601.Time `json:"enableOn,omitempty" mapstructure:"enabledOn,omitempty"`
-	Category ID            `json:"categoryId,omitempty" mapstructure:"categoryId,omitempty"`
-	Level    ID            `json:"levelId,omitempty" mapstructure:"levelId,omitempty"`
-	Owner    *ID           `json:"ownerId,omitempty" yaml:"ownerId,omitempty" mapstructure:"ownerId,omitempty"`
-	Filter   *ID           `json:"filterId,omitempty" yaml:"filterId,omitempty" mapstructure:"filterId,omitempty"`
-	Notes    *string       `json:"notes,omitempty" mapstructure:"notes,omitempty"`
+	Category *Nullable[ID]           `json:"categoryId,omitempty" mapstructure:"categoryId,omitempty"`
+	EnableOn *Nullable[iso8601.Time] `json:"enableOn,omitempty" mapstructure:"enabledOn,omitempty"`
+	Enabled  *Nullable[bool]         `json:"enabled,omitempty" mapstructure:"enabled,omitempty"`
+	Filter   *Nullable[ID]           `json:"filterId,omitempty" yaml:"filterId,omitempty" mapstructure:"filterId,omitempty"`
+	Id       ID                      `json:"id" mapstructure:"id"`
+	Level    *Nullable[ID]           `json:"levelId,omitempty" mapstructure:"levelId,omitempty"`
+	Name     *Nullable[string]       `json:"name,omitempty" mapstructure:"name,omitempty"`
+	Notes    *Nullable[string]       `json:"notes,omitempty" mapstructure:"notes,omitempty"`
+	Owner    *Nullable[ID]           `json:"ownerId,omitempty" yaml:"ownerId,omitempty" mapstructure:"ownerId,omitempty"`
 }
 
 func NewCheckUpdateInputTypeOf[T any](checkUpdateInput CheckUpdateInput) *T {
@@ -275,7 +275,7 @@ func (client *Client) DeleteCheck(id ID) error {
 		} `graphql:"checkDelete(input: $input)"`
 	}
 	v := PayloadVariables{
-		"input": CheckDeleteInput{Id: &id},
+		"input": CheckDeleteInput{Id: NewNullableFrom(id)},
 	}
 	err := client.Mutate(&m, v, WithName("CheckDelete"))
 	return HandleErrors(err, m.Payload.Errors)

check_manual.go

@@ -24,9 +24,9 @@ func NewManualCheckFrequencyInput(startingDate string, timeScale FrequencyTimeSc
 func NewManualCheckFrequencyUpdateInput(startingDate string, timeScale FrequencyTimeScale, value int) *ManualCheckFrequencyUpdateInput {
 	startingDateIso := NewISO8601Date(startingDate)
 	return &ManualCheckFrequencyUpdateInput{
-		StartingDate:       &startingDateIso,
+		StartingDate:       NewNullableFrom(startingDateIso),
 		FrequencyTimeScale: &timeScale,
-		FrequencyValue:     &value,
+		FrequencyValue:     NewNullableFrom(value),
 	}
 }