From 3d59ba713eff5fc3f827b143d951c3d3ad3a9c96 Mon Sep 17 00:00:00 2001
From: Jeff Fry <jfry@google.com>
Date: Tue, 11 May 2010 00:46:44 +0000
Subject: [PATCH] minor edits.

git-svn-id: http://google-refine.googlecode.com/svn/trunk@706 7d457c2a-affb-35e4-300a-418c747d4874
---
 tests/windmill/js/initialize.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/tests/windmill/js/initialize.js b/tests/windmill/js/initialize.js
index a3ee2d214..32c6af72a 100644
--- a/tests/windmill/js/initialize.js
+++ b/tests/windmill/js/initialize.js
@@ -11,11 +11,13 @@ params = params || {};
     };
 }
 
+
 actions.gw_wait4ajaxend = function () {
     return action("waits.forElement", { jquery: '("body[ajax_in_progress=\'false\']")[0]' } );
 };
+
 actions.gw_wait4menuitem = function (params) {
-    return action("waits.forElement", { jquery: '(".menu-item:contains(\'' + params.name + '\')")[0]' } );
+    return action("waits.forElement", { jquery: '(".menu-item:contains(\"' + params.name + '\")")[0]' } );
 };
 
 actions.gw_click_column_header = function (params) {
@@ -32,14 +34,14 @@ function assert(what,params) {
     };
 }
     
+//TODO: is there a way to make the assert wait first, rather than putting this before each call of the assert?,
 asserts.gw_row_count = function (count) {
-    action("waits.forElement", { jquery:  '(".viewPanel-summary-row-count")[0]' } );
+    //action("waits.forElement", { jquery:  '(".viewPanel-summary-row-count")[0]' } ); // doesn't work bc waits can't go inside functions
     asserts.assertText( { jquery: '(".viewPanel-summary-row-count")[0]', validator: count } );
 };
 
 asserts.gw_expected_top_value = function (expected_value) {
     // action("waits.forElement", { jquery: '("a.facet-choice-label")[0]' } );   // doesn't work bc waits can't go inside functions
-    //TODO: is there a way to make the assert wait first, rather than putting this before each call of the assert?,
     asserts.assertEquals(expected_value, $.trim($("a.facet-choice-label")[0].text)); 
 };