Merge pull request #18 from jaavier/feature/delete-key-list

Add functions to delete keys and lists

Author: jaavier

keys.go

@@ -25,3 +25,12 @@ func Get(key string) (string, error) {
 	}
 	return "", fmt.Errorf("key '%s' not found", key)
 }
+
+func DeleteKey(key string) error {
+	value, _ := Get(key)
+	if len(value) > 0 {
+		delete(keys, key)
+		return nil
+	}
+	return fmt.Errorf("key '%s' doesn't exist", key)
+}	

lists.go

@@ -144,4 +144,14 @@ func ReplaceList(listName string, index int, newElement string) (bool, error) {
 func CountList(listName string) (int, error) {
 	list, err := GetList(listName)
 	return len(list), err
+}
+
+func DeleteList(listName string) error {
+	for key, _ := range lists {
+		if key == listName {
+			delete(lists, listName)
+			return nil
+		}
+	}
+	return fmt.Errorf("list '%s' doesn't exist", listName)
 }

tests/keys_test.go

@@ -23,4 +23,11 @@ func TestKeys(t *testing.T) {
 		}
 	})
 
+	t.Run("Delete Key", func(t *testing.T) {
+		var newKey string = "delete-key"
+		sider.Set(newKey, "value")
+		if err := sider.DeleteKey(newKey); err != nil {
+			t.Errorf("Error deleting key '%s'", newKey)
+		}
+	})
 }

tests/lists_test.go

@@ -168,4 +168,12 @@ func TestLists(t *testing.T) {
 			}
 		}
 	})
+
+	t.Run("Delete List", func(t *testing.T) {
+		var newList string = "my-list-to-delete"
+		sider.LPush(newList, "element")
+		if err := sider.DeleteList(newList); err != nil {
+			t.Errorf("error: %v", err)
+		}
+	})
 }