use pointer

Author: ko-da-k

folder_permissions.go

@@ -29,7 +29,7 @@ type FolderPermission struct {
 }
 
 type PermissionItems struct {
-	Items []PermissionItem `json:"items"`
+	Items []*PermissionItem `json:"items"`
 }
 
 type PermissionItem struct {
@@ -41,8 +41,8 @@ type PermissionItem struct {
 	Permission int64  `json:"permission"`
 }
 
-func (c *Client) FolderPermissions(fid string) ([]FolderPermission, error) {
-	permissions := make([]FolderPermission, 0)
+func (c *Client) FolderPermissions(fid string) ([]*FolderPermission, error) {
+	permissions := make([]*FolderPermission, 0)
 
 	req, err := c.newRequest("GET", fmt.Sprintf("/api/folders/%s/permissions", fid), nil, nil)
 	if err != nil {

folder_permissions_test.go

@@ -67,7 +67,7 @@ func TestFolderPermissions(t *testing.T) {
 
 	t.Log(pretty.PrettyFormat(resp))
 
-	expects := []FolderPermission{
+	expects := []*FolderPermission{
 		{
 			Id:             1,
 			FolderUid:      "nErXDvCkzz",
@@ -96,7 +96,7 @@ func TestFolderPermissions(t *testing.T) {
 
 	for i, expect := range expects {
 		t.Run("check data", func(t *testing.T) {
-			if resp[i] != expect {
+			if resp[i].Id != expect.Id || resp[i].Role != expect.Role {
 				t.Error("Not correctly data")
 			}
 		})
@@ -108,7 +108,7 @@ func TestUpdateFolderPermissions(t *testing.T) {
 	defer server.Close()
 
 	items := &PermissionItems{
-		Items: []PermissionItem{
+		Items: []*PermissionItem{
 			{
 				Role:       "viewer",
 				Permission: 1,