Merge pull request #101 from joinpursuit/start-with-all-tests-red

add check for empty method
joinpursuit · Oct 26, 2020 · 74b289f · 74b289f
2 parents d76be59 + 61e15fe
commit 74b289f
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/__tests__/loops.test.js b/__tests__/loops.test.js
@@ -287,9 +287,10 @@ describe("containsWhileLoop", () => {
     expect(funcString.includes("for")).toBe(false);
   });
 
-  test("does not use built in includes method", () => {
+  test("is implemented and does not use built in includes method", () => {
     const funcString = containsWhileLoop.toString();
     expect(funcString.includes("includes")).toBe(false);
+    expect(funcString !== "function containsWhileLoop() {}").toBe(true)
   });
 });
 
@@ -309,9 +310,10 @@ describe("containsForLoop", () => {
     expect(funcString.includes("for")).toBe(true);
   });
 
-  test("does not use built in includes method", () => {
+  test("is implemented and does not use built in includes method", () => {
     const funcString = containsForLoop.toString();
     expect(funcString.includes("includes")).toBe(false);
+    expect(funcString !== "function containsForLoop() {}").toBe(true)
   });
 });