Merge pull request liammclennan#55 from cytrowski/improvement/better-suggestions

liammclennan · web-flow · commit e972a23b6f52 · 2016-09-29T09:00:56.000+10:00
Display more details in splice example
diff --git a/jskoans.htm b/jskoans.htm
@@ -1,5 +1,5 @@
-<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" 
-                    "http://www.w3.org/TR/html4/loose.dtd">
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
+                      "http://www.w3.org/TR/html4/loose.dtd">
 <html>
 <head>
   <script src="support/jquery-1.4.1.js"></script>
@@ -26,8 +26,7 @@
   <script type="text/javascript" src="topics/about_functions_and_closure.js"></script>
   <script type="text/javascript" src="topics/about_this.js"></script>
   <script type="text/javascript" src="topics/about_scope.js"></script>
-  <script type="text/javascript" src="topics/about_regular_expressions.js"></script> 
-  
+  <script type="text/javascript" src="topics/about_regular_expressions.js"></script>
 </head>
 <body>
  <h1 id="qunit-header">JavaScript Koans</h1>
diff --git a/topics/about_arrays.js b/topics/about_arrays.js
@@ -19,8 +19,10 @@ test("length", function() {
 test("splice", function() {
     var daysOfWeek = ['Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday', 'Sunday'];
     var workingWeek = daysOfWeek.splice(__, __);
-    ok(workingWeek.equalTo([__]), 'what is the value of workingWeek?');
-    ok(daysOfWeek.equalTo([__]), 'what is the value of daysOfWeek?');
+    var weekend = daysOfWeek;
+
+    deepEqual(workingWeek, ['Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday'], 'what is the value of workingWeek?');
+    deepEqual(weekend, ['Saturday', 'Sunday'], 'what is the value of weekend?');
 });
 
 test("stack methods", function() {
