Improved browser form tests.

test/browser/form_test.rb

@@ -52,10 +52,14 @@ class FormTest < TestCase
       browser: agent,
       css: 'body div.zammad-form-modal button[type="submit"]',
     )
-    exists(
+    watch_for(
       browser: agent,
       css: 'body div.zammad-form-modal .has-error [name="body"]',
     )
+    watch_for_disappear(
+      browser: agent,
+      css: 'body div.zammad-form-modal button[type="submit"][disabled]',
+    )
     set(
       browser: agent,
       css: 'body div.zammad-form-modal [name="body"]',
@@ -70,10 +74,14 @@ class FormTest < TestCase
       browser: agent,
       css: 'body div.zammad-form-modal button[type="submit"]',
     )
-    exists(
+    watch_for(
       browser: agent,
       css: 'body div.zammad-form-modal .has-error [name="email"]',
     )
+    watch_for_disappear(
+      browser: agent,
+      css: 'body div.zammad-form-modal button[type="submit"][disabled]',
+    )
     set(
       browser: agent,
       css: 'body div.zammad-form-modal [name="email"]',
@@ -83,10 +91,14 @@ class FormTest < TestCase
       browser: agent,
       css: 'body div.zammad-form-modal button[type="submit"]',
     )
-    exists(
+    watch_for(
       browser: agent,
       css: 'body div.zammad-form-modal .has-error [name="email"]',
     )
+    watch_for_disappear(
+      browser: agent,
+      css: 'body div.zammad-form-modal button[type="submit"][disabled]',
+    )
     set(
       browser: agent,
       css: 'body div.zammad-form-modal [name="email"]',
@@ -111,8 +123,7 @@ class FormTest < TestCase
       browser: customer,
       url:     "#{browser_url}/assets/form/form.html",
     )
-    sleep 4
+    watch_for(
-    match(
       browser: customer,
       css: '.js-logDisplay',
       value: 'Faild to load form config, feature is disabled',
@@ -143,7 +154,7 @@ class FormTest < TestCase
       browser: customer,
       css: '#feedback-form-modal',
     )
-    exists(
+    watch_for(
       browser: customer,
       css: 'body div.zammad-form-modal',
     )
@@ -252,10 +263,14 @@ class FormTest < TestCase
       browser: customer,
       css: 'body div.zammad-form-modal button[type="submit"]',
     )
-    exists(
+    watch_for(
       browser: customer,
       css: 'body div.zammad-form-modal .has-error [name="name"]',
     )
+    watch_for_disappear(
+      browser: customer,
+      css: 'body div.zammad-form-modal button[type="submit"][disabled]',
+    )
     set(
       browser: customer,
       css: 'body div.zammad-form-modal [name="name"]',
@@ -270,10 +285,14 @@ class FormTest < TestCase
       browser: customer,
       css: 'body div.zammad-form-modal button[type="submit"]',
     )
-    exists(
+    watch_for(
       browser: customer,
       css: 'body div.zammad-form-modal .has-error [name="body"]',
     )
+    watch_for_disappear(
+      browser: customer,
+      css: 'body div.zammad-form-modal button[type="submit"][disabled]',
+    )
     set(
       browser: customer,
       css: 'body div.zammad-form-modal [name="body"]',
@@ -288,10 +307,14 @@ class FormTest < TestCase
       browser: customer,
       css: 'body div.zammad-form-modal button[type="submit"]',
     )
-    exists(
+    watch_for(
       browser: customer,
       css: 'body div.zammad-form-modal .has-error [name="email"]',
     )
+    watch_for_disappear(
+      browser: customer,
+      css: 'body div.zammad-form-modal button[type="submit"][disabled]',
+    )
     set(
       browser: customer,
       css: 'body div.zammad-form-modal [name="email"]',
@@ -301,10 +324,14 @@ class FormTest < TestCase
       browser: customer,
       css: 'body div.zammad-form-modal button[type="submit"]',
     )
-    exists(
+    watch_for(
       browser: customer,
       css: 'body div.zammad-form-modal .has-error [name="email"]',
     )
+    watch_for_disappear(
+      browser: customer,
+      css: 'body div.zammad-form-modal button[type="submit"][disabled]',
+    )
     set(
       browser: customer,
       css: 'body div.zammad-form-modal [name="email"]',

test/browser_test_helper.rb

@@ -1240,7 +1240,14 @@ class TestCase < Test::Unit::TestCase
       if element #&& element.displayed?
         begin
 
+          # watch for selector
+          if !params[:attribute] && !params[:value]
+            assert(true, "'#{params[:css]}' found")
+            sleep 0.5
+            return true
+
           # match pn attribute
+          else
             text = if params[:attribute]
                      element.attribute(params[:attribute])
                    elsif params[:css] =~ /(input|textarea)/i
@@ -1253,6 +1260,7 @@ class TestCase < Test::Unit::TestCase
               sleep 0.5
               return true
             end
+          end
         rescue
           # try again
         end
@@ -1260,7 +1268,10 @@ class TestCase < Test::Unit::TestCase
       sleep 0.5
     }
     screenshot(browser: instance, comment: 'watch_for_failed')
-    raise "'#{params[:value]}' found in '#{text}'"
+    if !params[:attribute] && !params[:value]
+      raise "'#{params[:css]}' not found"
+    end
+    raise "'#{params[:value]}' not found in '#{text}'"
   end
 
 =begin