code review

cycledash/static/js/examine/RecordStore.js

@@ -231,7 +231,7 @@ function createRecordStore(run, dispatcher, opt_testDataSource) {
     }
   }
 
-  // no need to debounce this update -- make it so now!
+  // There's no need to debounce this update -- make it so now!
   _updateGenotypes({append: false});
 
   function notifyChange() {

cycledash/static/js/examine/components/QueryBox.js

@@ -113,8 +113,10 @@ var QueryBox = React.createClass({
     }
   },
 
-  // Set the visible query in the typeahead box to `str`.
-  // This is factored out for easy intercepting while testing.
+  /**
+   * Set the visible query in the typeahead box to `str`.
+   * This is factored out for easy intercepting while testing.
+   */
   setDisplayedQuery: function(str) {
     var $input = $(this.refs.input.getDOMNode());
     $input.typeahead('val', str);

tests/js/Utils-test.js

@@ -22,30 +22,32 @@ describe('Utils', function() {
   describe('stubReactMethod', function() {
     it('should stub a React class method, then restore it', function() {
       var TestUtils = React.addons.TestUtils;
-      var captured;
+      var captured = [];
       var RC = React.createClass({
         render: function() {
           this.method();
           return null;
         },
         method: function() {
-          captured = 'original';
+          captured.push('original');
         }
       });
 
-      var stubFn = function() { captured = 'stubbed'; };
+      var stubFn = function() { captured.push('stubbed'); };
 
       var el = React.createElement(RC, null);
       TestUtils.renderIntoDocument(el);
-      assert.equal('original', captured);
+      assert.deepEqual(['original'], captured);
 
       var stub = Utils.stubReactMethod(RC, 'method', stubFn);
+      captured = [];
       TestUtils.renderIntoDocument(el);
-      assert.equal('stubbed', captured);
+      assert.deepEqual(['stubbed'], captured);
 
       stub.restore();
+      captured = [];
       TestUtils.renderIntoDocument(el);
-      assert.equal('original', captured);
+      assert.deepEqual(['original'], captured);
     });
   });
 });