From b52f2278397fc0c7d999bcf80156e03493e92c64 Mon Sep 17 00:00:00 2001 From: impinball Date: Fri, 6 Nov 2015 15:59:30 -0500 Subject: [PATCH] Re-enable some failing tests --- test/mithril.deferred.js | 2 +- test/mithril.render.js | 2 +- test/mithril.route.js | 9 ++++----- test/mithril.trust.js | 2 +- 4 files changed, 7 insertions(+), 8 deletions(-) diff --git a/test/mithril.deferred.js b/test/mithril.deferred.js index 1f31eb1c..ea7a8184 100644 --- a/test/mithril.deferred.js +++ b/test/mithril.deferred.js @@ -118,7 +118,7 @@ describe("m.deferred()", function () { }) // FIXME: this is a bug. - xit("synchronously throws subclasses of Errors on creation", function () { + it("synchronously throws subclasses of Errors on creation", function () { expect(function () { m.deferred().reject(new TypeError()) }).to.throw() diff --git a/test/mithril.render.js b/test/mithril.render.js index 34faa178..27e3c154 100644 --- a/test/mithril.render.js +++ b/test/mithril.render.js @@ -1318,7 +1318,7 @@ describe("m.render()", function () { // FIXME: implement document.createRange().createContextualFragment() in the // mock document to fix this test - xit("keeps unkeyed identity if mixed with elements/trusted text and identity can be inferred", function () { // eslint-disable-line + it("keeps unkeyed identity if mixed with elements/trusted text and identity can be inferred", function () { // eslint-disable-line var root = mock.document.createElement("div") m.render(root, m("div", [ diff --git a/test/mithril.route.js b/test/mithril.route.js index 7317d268..b82642ea 100644 --- a/test/mithril.route.js +++ b/test/mithril.route.js @@ -39,7 +39,6 @@ describe("m.route()", function () { // Use these instead of `it` and `xit` in this set of tests if you need a // root element. var dit = makeIt(it) - var xdit = makeIt(xit) // Wraps the `it` function for dependency injection that doesn't require // `this` @@ -103,7 +102,7 @@ describe("m.route()", function () { }) // FIXME: this causes others to fail - xdit("skips route change if component ctrl.onunload calls preventDefault", function (root) { // eslint-disable-line + dit("skips route change if component ctrl.onunload calls preventDefault", function (root) { // eslint-disable-line mode("search") var spy = sinon.spy() @@ -131,7 +130,7 @@ describe("m.route()", function () { }) // FIXME: this causes others to fail - xdit("skips route change if subcomponent ctrl.onunload calls preventDefault", function (root) { // eslint-disable-line + dit("skips route change if subcomponent ctrl.onunload calls preventDefault", function (root) { // eslint-disable-line mode("search") var spy = sinon.spy() @@ -162,7 +161,7 @@ describe("m.route()", function () { }) // FIXME: this causes others to fail - xdit("skips route change if non-curried component ctrl.onunload calls preventDefault", function (root) { // eslint-disable-line + dit("skips route change if non-curried component ctrl.onunload calls preventDefault", function (root) { // eslint-disable-line mode("search") var spy = sinon.spy() @@ -191,7 +190,7 @@ describe("m.route()", function () { }) // FIXME: this causes others to fail - xdit("skips route change if non-curried subcomponent ctrl.onunload calls preventDefault", function (root) { // eslint-disable-line + dit("skips route change if non-curried subcomponent ctrl.onunload calls preventDefault", function (root) { // eslint-disable-line mode("search") var spy = sinon.spy() diff --git a/test/mithril.trust.js b/test/mithril.trust.js index 748fa75e..41881b31 100644 --- a/test/mithril.trust.js +++ b/test/mithril.trust.js @@ -43,7 +43,7 @@ describe("m.trust()", function () { // FIXME: this is a bug (trusted string's contents rendered as just // textual contents) - xit("works with mixed trusted content in td", function () { + it("works with mixed trusted content in td", function () { var root = document.createElement("table") root.appendChild(root = document.createElement("tr"))