cleanup .unbind() code

[mkws-moved-to-github.git] / test / spec / mkws-pazpar2.js

diff --git a/test/spec/mkws-pazpar2.js b/test/spec/mkws-pazpar2.js
index fe1e746..8772996 100644 (file)
--- a/test/spec/mkws-pazpar2.js
+++ b/test/spec/mkws-pazpar2.js
@@ -289,6 +289,13 @@ describe("Check Termlist", function () {
 
         $("div.mkwsFacetSources div.term:nth-child(" + source_number + ") a").trigger("click");
 
+        // wait for a stat response
+        var waitcount = 0;
+        $(".mkwsPager").bind("DOMNodeInserted DOMNodeRemoved propertychange", function () {
+            waitcount++;
+            debug("DOM wait for stat: " + waitcount);
+        });
+
         waitsFor(function () {
             if ($("div.mkwsNavi").length && $("div.mkwsNavi").text().match(/(Source|datenquelle|kilder): /i)) {
                 return true;
@@ -300,34 +307,45 @@ describe("Check Termlist", function () {
         // Note: it may happens that limited source search returns the same number of hits
         // as before. Thats not really an error, but unfortunate
         waitsFor(function () {
-            return get_hit_counter() <= hits_all_targets ? true : false;
+            return waitcount >= 2 && get_hit_counter() <= hits_all_targets ? true : false;
         }, "Limited source search for less than " + hits_all_targets + " hits", 5 * jasmine_config.second);
 
         runs(function () {
             var hits_single_target = get_hit_counter();
             debug("get less hits for sources: " + hits_all_targets + " > " + hits_single_target);
             expect(hits_all_targets).not.toBeLessThan(hits_single_target);
+
+            $(".mkwsPager").unbind("DOMNodeInserted DOMNodeRemoved propertychange");
         });
     });
 });
 
+
 describe("Check record list", function () {
     it("got a record", function () {
-        // make sure we have a link.
         var linkaddr = "div.mkwsRecords div.record:nth-child(1) a";
-        var waitcount = 0;
+        var waitcount = 2;
 
-        waitsFor(function () {
+        /*
+        // wait for new records, propertychange is for IE8
+        $("div.mkwsRecords").bind("DOMNodeInserted propertychange", function () {
             waitcount++;
-            debug("waiting for the link " + waitcount + " " + $(linkaddr) + " =" + $(linkaddr).length + " " + $(linkaddr).text());
-            return ($(linkaddr).length > 0);
-        }, "wait until we see a link", 1 * jasmine_config.second);
+            debug("DOM DOMNodeInserted:" + waitcount + " " + $("div.mkwsRecords div.record").length);
+        });
+        $("div.mkwsRecords").bind("DOMNodeRemoved", function () {
+            waitcount++;
+            debug("DOM DOMNodeRemoved:" + waitcount + " " + $("div.mkwsRecords div.record").length);
+        });
+        */
+
+        waitsFor(function () {
+            // remove + insert node: must be at least 2
+            return waitcount >= 2 && $(linkaddr).length > 0;
+        }, "wait until we see a new record: " + waitcount, 2.2 * jasmine_config.second);
 
         runs(function () {
-            var link = $(linkaddr);
-            debug("== waited (" + waitcount + ") for the link..." + $(linkaddr) + " =" + $(linkaddr).length + " " + $(linkaddr).text());
-            expect(link.length).toBe(1);
-            // link.trigger("click");
+            expect(waitcount).toBeGreaterThan(0);
+            // $("div.mkwsRecords").unbind("DOMNodeInserted DOMNodeRemoved");
         });
     });
 });
@@ -342,8 +360,10 @@ describe("Show record", function () {
         // wait until the record pops up
         waitsFor(function () {
             var show = $("div.mkwsRecords div.record:nth-child(" + record_number + ") div");
+            debug($("div.mkwsRecords div.record").text());
+
             return show != null && show.length ? true : false;
-        }, "wait some miliseconds to show up a record", 2 * jasmine_config.second);
+        }, "wait some miliseconds to show up a record", 3 * jasmine_config.second);
 
         runs(function () {
             debug("show record pop up");
@@ -365,7 +385,7 @@ describe("Show record", function () {
             debug("URL: " + url.attr('href') + " text: " + url.text());
 
             expect(url.attr('href')).not.toBe(null);
-            expect(url.attr('href')).toMatch(/^https?:\/\/[a-z0-9]+\.[0-9a-z].*\//i);
+            expect(url.attr('href')).toMatch(/^https?:\/\/[a-z0-9\-]+\.[0-9a-z].*\//i);
             expect(url.text()).not.toBe("");
         }
     });