From 884c694d967d9edd5f54bc7559e228ee9fcf0a86 Mon Sep 17 00:00:00 2001 From: jakearchibald Date: Mon, 3 Aug 2009 16:39:40 +0100 Subject: [PATCH] NodeList#html: Added test to show failing case[#17] --- test/glow/dom/dom.js | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/test/glow/dom/dom.js b/test/glow/dom/dom.js index 5551111..c50bc34 100644 --- a/test/glow/dom/dom.js +++ b/test/glow/dom/dom.js @@ -1077,7 +1077,7 @@ t.test("glow.dom.NodeList.clone()", function () { }); t.test("glow.dom.NodeList.html()", function () { - t.expect(2); + t.expect(4); var nodes = glow.dom.create("
first
second
"); @@ -1087,6 +1087,17 @@ t.test("glow.dom.NodeList.html()", function () { t.equals(nodes.html().toLowerCase(), "
first
", "html sets inner html for first node"); + var emptyNodeList = new glow.dom.NodeList(); + // this shouldn't error + emptyNodeList.html("
"); + t.equals(emptyNodeList.length, 0, "No error on empty nodelist"); + + // http://glow-project.lighthouseapp.com/projects/33663/tickets/17-empty-nodelists-html-method-causes-error-if-passed-undefined + emptyNodeList = new glow.dom.NodeList(); + // this shouldn't error + emptyNodeList.html(undefined); + t.equals(emptyNodeList.length, 0, "No error on empty nodelist with undefined param"); + }); t.test("glow.dom.NodeList#append()", function () {