fix .html(str)

- add tests for setting html string

index.js

@@ -224,7 +224,11 @@ proto.html = function(val){
   var el = this[0];
 
   if (val) {
-    el.innerHTML = str;
+    if (typeof(val) !== 'string') {
+      throw new Error('.html() requires a string');
+    }
+
+    el.innerHTML = val;
     return this;
   }
 

test/dom.js

@@ -20,11 +20,13 @@ test('trim selector', function() {
     assert(' foo ' == li.text());
 });
 
+// create from html
 test('html', function() {
     var list = dom('<em>Hello</em>');
     assert('Hello' == list[0].textContent);
 });
 
+// create multiple html elements
 test('multiple html', function() {
     var list = dom('<em>Hello</em><em>World</em>');
     assert.equal(2, list.length);
@@ -159,4 +161,3 @@ test('.attr(key)', function() {
 });
 
 
-// setting html by: string, array of string? array of List

test/html.js

@@ -0,0 +1,25 @@
+var assert = require('assert');
+var dom = require('../');
+
+suite('.html');
+
+test('.html(str)', function() {
+    var div = dom('<div>');
+
+    // setting html should work
+    div.html('<p>foo</p>');
+    assert.equal(div.html(), '<p>foo</p>');
+
+    // make sure that we can find in new html
+    assert.equal(div.find('p').text(), 'foo');
+});
+
+test('.html(str) - override', function() {
+    var div = dom('<div><p>foo</div>');
+
+    // override old html
+    div.html('bar');
+    assert.equal(div.html(), 'bar');
+    assert.equal(div.text(), 'bar');
+});
+