Skip to content

Commit

Permalink
Fix struct tag for varset relationship
Browse files Browse the repository at this point in the history
  • Loading branch information
sebasslash committed Aug 31, 2022
1 parent 16f8caa commit 49f7b93
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
2 changes: 1 addition & 1 deletion variable_set_variable.go
Expand Up @@ -49,7 +49,7 @@ type VariableSetVariable struct {
Sensitive bool `jsonapi:"attr,sensitive"`

// Relations
VariableSet *VariableSet `jsonapi:"relation,configurable"`
VariableSet *VariableSet `jsonapi:"relation,varset"`
}

type VariableSetVariableListOptions struct {
Expand Down
8 changes: 5 additions & 3 deletions variable_set_variable_test.go
Expand Up @@ -34,12 +34,14 @@ func TestVariableSetVariablesList(t *testing.T) {
t.Run("without list options", func(t *testing.T) {
vl, err := client.VariableSetVariables.List(ctx, vsTest.ID, nil)
require.NoError(t, err)
require.NotEmpty(t, vl.Items)
assert.Contains(t, vl.Items, vTest1)
assert.Contains(t, vl.Items, vTest2)

t.Skip("paging not supported yet in API")
assert.Equal(t, 1, vl.CurrentPage)
assert.Equal(t, 2, vl.TotalCount)
t.Run("variable set relationship is deserialized", func(t *testing.T) {
require.NotNil(t, vl.Items[0].VariableSet)
assert.Equal(t, vsTest.ID, vl.Items[0].VariableSet.ID)
})
})

t.Run("with list options", func(t *testing.T) {
Expand Down

0 comments on commit 49f7b93

Please sign in to comment.