From 619f17f2b8ab3b418fa92f96bcb44b3203a0c580 Mon Sep 17 00:00:00 2001
From: Ken <kchau@microsoft.com>
Date: Thu, 28 Feb 2019 22:37:47 -0800
Subject: [PATCH] some exercise for 2.6

---
 .../src/reducers/pureFunctions.spec.ts        |  2 +-
 .../exercise/src/reducers/pureFunctions.ts    | 25 ++++++++++---------
 2 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/step2-05/exercise/src/reducers/pureFunctions.spec.ts b/step2-05/exercise/src/reducers/pureFunctions.spec.ts
index c8e202a..aa7018e 100644
--- a/step2-05/exercise/src/reducers/pureFunctions.spec.ts
+++ b/step2-05/exercise/src/reducers/pureFunctions.spec.ts
@@ -21,5 +21,5 @@ describe('TodoApp reducers', () => {
     */
   });
 
-  // TODO: test remove, complete and clear
+  // TODO: add a test for remove()
 });
diff --git a/step2-05/exercise/src/reducers/pureFunctions.ts b/step2-05/exercise/src/reducers/pureFunctions.ts
index 12ca8ef..4f2b931 100644
--- a/step2-05/exercise/src/reducers/pureFunctions.ts
+++ b/step2-05/exercise/src/reducers/pureFunctions.ts
@@ -19,20 +19,21 @@ export function remove(state: Store['todos'], id: string) {
 }
 
 export function complete(state: Store['todos'], id: string) {
-  // Write code:
-  // - to clone the state[id] object into new todo object, using the spread syntax
-  // - in the spread syntax, also override the value of the completed key like this: {...foo, [id]: !foo[id].completed}
-  // - modify new state and set the id key to the value of the new item object
-
-  return state;
+  // Clone the todo, overriding
+  const newTodo = { ...state[id], completed: !state[id].completed };
+  return { ...state, [id]: newTodo };
 }
 
 export function clear(state: Store['todos']) {
-  // Write code:
-  // - to clone the state object into new state object
-  // - loop through the keys of the new state object
-  // - remove those items inside that new state if the item is completed using the "delete" keyword
-  // - return the new state
+  // Clone the todos
+  const newTodos = { ...state };
 
-  return state;
+  // Delete all todos based on the completed flag, looping over the keys of the todos
+  Object.keys(state).forEach(key => {
+    if (state[key].completed) {
+      delete newTodos[key];
+    }
+  });
+
+  return newTodos;
 }