[BUG #9080] Return undefined after test is skipped

qooxdoo-archive · Apr 23, 2015 · 75dece4 · 75dece4
1 parent 7794840
commit 75dece4
Showing 1 changed file with 13 additions and 10 deletions.
diff --git a/framework/source/test/tests/bom/client/Pdfjs.js b/framework/source/test/tests/bom/client/Pdfjs.js
@@ -25,14 +25,17 @@
  */
 describe("bom.client.Pdfjs", function() {
 
-  if (qx.core.Environment.get("browser.name") === "firefox") {
-    it("is PDF.js available", function (done) {
-      qx.core.Environment.getAsync("plugin.pdfjs", function (result) {
-        setTimeout(function () {
-          assert.isTrue(result);
-          done();
-        }, 0);
-      }, this);
-    });
-  }
+  it("is PDF.js available", function(done) {
+    if (qx.core.Environment.get("browser.name") !== "firefox") {
+      this.test.skipped = true;
+      done();
+      return;
+    }
+    qx.core.Environment.getAsync("plugin.pdfjs", function(result) {
+      setTimeout(function() {
+        assert.isTrue(result);
+        done();
+      }, 0);
+    }, this);
+  });
 });