Merge pull request #145 from opf/fix/rails3-cucumber-wait-for-ajax

merged anyhow. Maybe we want to add a sleep later but it is definitly not necessary.

app/assets/javascripts/application.js

@@ -549,6 +549,16 @@ jQuery.viewportHeight = function() {
 
 /* TODO: integrate with existing code and/or refactor */
 jQuery(document).ready(function($) {
+  document.ajaxActive = false;
+
+
+  $(document).ajaxStart(function () {
+    document.ajaxActive = true;
+  });
+
+  $(document).ajaxStop(function () {
+    document.ajaxActive = false;
+  });
 
     var propagateOpenClose = function () {
       if ($(this).is(":visible")) {

features/step_definitions/web_steps.rb

@@ -273,24 +273,27 @@ end
 
 When /^I wait(?: (\d+) seconds)? for(?: the)? [Aa][Jj][Aa][Xx](?: requests?(?: to finish)?)?$/ do |timeout|
   ajax_done = lambda do
-    page.evaluate_script(%Q{
-      (function (){
-        var done = true;
-
-        if (window.jQuery) {
-          if (!window.jQuery.isReady || window.jQuery.active != 0) {
-            done = false;
+    is_done = false
+    while (!is_done) do
+      is_done = page.evaluate_script(%Q{
+        (function (){
+          var done = true;
+
+          if (window.jQuery) {
+            if (document.ajaxActive) {
+              done = false;
+            }
           }
-        }
-        if (window.Prototype && window.Ajax) {
-          if (window.Ajax.activeRequestCount != 0) {
-            done = false;
+          if (window.Prototype && window.Ajax) {
+            if (window.Ajax.activeRequestCount != 0) {
+              done = false;
+            }
           }
-        }
 
-        return done;
-      }())
-    }.gsub("\n", ''))
+          return done;
+        }())
+      }.gsub("\n", ''))
+    end
   end
 
   timeout = timeout.present? ?