fixed unmarshalling and added options

Author: hwatkins05-hashicorp

stack_configuration_summary.go

@@ -8,7 +8,7 @@ import (
 
 type StackConfigurationSummaries interface {
 	// List lists all the stack configuration summaries for a stack.
-	List(ctx context.Context, stackID string) (*StackConfigurationSummaryList, error)
+	List(ctx context.Context, stackID string, options *StackConfigurationSummaryListOptions) (*StackConfigurationSummaryList, error)
 }
 
 type stackConfigurationSummaries struct {
@@ -22,19 +22,27 @@ type StackConfigurationSummaryList struct {
 	Items []*StackConfigurationSummary
 }
 
+type StackConfigurationSummaryListOptions struct {
+	ListOptions
+}
+
 type StackConfigurationSummary struct {
-	ID             string
-	Type           string
-	Status         string
-	SequenceNumber int
+	ID             string `jsonapi:"primary,stack-configuration-summaries"`
+	Type           string `jsonapi:"attr,type"`
+	Status         string `jsonapi:"attr,status"`
+	SequenceNumber int    `jsonapi:"attr,sequence-number"`
 }
 
-func (s stackConfigurationSummaries) List(ctx context.Context, stackID string) (*StackConfigurationSummaryList, error) {
+func (s stackConfigurationSummaries) List(ctx context.Context, stackID string, options *StackConfigurationSummaryListOptions) (*StackConfigurationSummaryList, error) {
 	if !validStringID(&stackID) {
 		return nil, fmt.Errorf("invalid stack ID: %s", stackID)
 	}
 
-	req, err := s.client.NewRequest("GET", fmt.Sprintf("stacks/%s/stack-configuration-summaries", url.PathEscape(stackID)), nil)
+	if options == nil {
+		options = &StackConfigurationSummaryListOptions{}
+	}
+
+	req, err := s.client.NewRequest("GET", fmt.Sprintf("stacks/%s/stack-configuration-summaries", url.PathEscape(stackID)), options)
 	if err != nil {
 		return nil, err
 	}

stack_configuration_summary_integration_test.go

@@ -27,7 +27,7 @@ func TestStackConfigurationSummaryList(t *testing.T) {
 	stack, err := client.Stacks.Create(ctx, StackCreateOptions{
 		Name: "aa-test-stack",
 		VCSRepo: &StackVCSRepoOptions{
-			Identifier:   "hwatkins05-hashicorp/pet-nulls-stack",
+			Identifier:   "hashicorp-guides/pet-nulls-stack",
 			OAuthTokenID: oauthClient.OAuthTokens[0].ID,
 		},
 		Project: &Project{
@@ -39,7 +39,7 @@ func TestStackConfigurationSummaryList(t *testing.T) {
 	stack2, err := client.Stacks.Create(ctx, StackCreateOptions{
 		Name: "bb-test-stack",
 		VCSRepo: &StackVCSRepoOptions{
-			Identifier:   "hwatkins05-hashicorp/pet-nulls-stack",
+			Identifier:   "hashicorp-guides/pet-nulls-stack",
 			OAuthTokenID: oauthClient.OAuthTokens[0].ID,
 		},
 		Project: &Project{
@@ -59,21 +59,21 @@ func TestStackConfigurationSummaryList(t *testing.T) {
 	require.NoError(t, err)
 
 	t.Run("Successful empty list", func(t *testing.T) {
-		stackConfigSummaryList, err := client.StackConfigurationSummaries.List(ctx, stack.ID)
+		stackConfigSummaryList, err := client.StackConfigurationSummaries.List(ctx, stack.ID, nil)
 		require.NoError(t, err)
 
 		assert.Len(t, stackConfigSummaryList.Items, 0)
 	})
 
 	t.Run("Successful multiple config summary list", func(t *testing.T) {
-		stackConfigSummaryList, err := client.StackConfigurationSummaries.List(ctx, stack2.ID)
+		stackConfigSummaryList, err := client.StackConfigurationSummaries.List(ctx, stack2.ID, nil)
 		require.NoError(t, err)
 
 		assert.Len(t, stackConfigSummaryList.Items, 2)
 	})
 
 	t.Run("Unsuccessful list", func(t *testing.T) {
-		_, err := client.StackConfigurationSummaries.List(ctx, "")
+		_, err := client.StackConfigurationSummaries.List(ctx, "", nil)
 		require.Error(t, err)
 	})
 }