More error info (#49)

Author: AlexYaroshenko

api/lists.go

@@ -5,6 +5,7 @@ package api
 // See LICENSE file for license details
 
 import (
+	"fmt"
 	"strconv"
 	"time"
 
@@ -347,13 +348,12 @@ func (c *ListsController) Clone(ctx *app.CloneListsContext) error {
 		opts...,
 	)
 	if err != nil {
-		return err
+		return fmt.Errorf("failed to create new list: %w", err)
 	}
 
 	// Add to manager
 	db.Lists.Add(list)
 
 	// ListsController_Create: end_implement
 	return ctx.OKFull(morph.List{In: list}.Full(ctx, db.Storage, true))
-
 }

list.go

@@ -147,11 +147,12 @@ func (l ListOption) Cache(cache Cache) ListOption {
 // Use WithListOption to access additional options.
 func NewList(ctx context.Context, id ListID, set string, bs blobstore.Store, opts ...ListOption) (*List, error) {
 	ctx = log.WithFn(ctx, "list_id", id)
+	log.Info(ctx, "Creating new list", "set", set, "list_id", id)
 	options := defaultListOptions()
 	for _, opt := range opts {
 		err := opt(&options)
 		if err != nil {
-			return nil, err
+			return nil, fmt.Errorf("applying option: %v", err)
 		}
 	}
 	if set == "" {
@@ -174,13 +175,13 @@ func NewList(ctx context.Context, id ListID, set string, bs blobstore.Store, opt
 		e.UpdateTime(time.Now())
 		err := l.Populate(ctx, bs, e)
 		if err != nil {
-			return nil, err
+			return nil, fmt.Errorf("populating list: %v", err)
 		}
 	}
 	if options.clone != nil {
 		err := l.cloneElements(ctx, bs, options.clone)
 		if err != nil {
-			return nil, err
+			return nil, fmt.Errorf("cloning list: %v", err)
 		}
 	}
 	if l.Scores.Unset() {

list_test.go

@@ -1,4 +1,4 @@
-//nolint
+// nolint
 package rankdb_test
 
 // Copyright 2019 Vivino. All rights reserved