Merge pull request #334 from sorvell/master

Fix insertBefore on IE and minor test issue with new Option; merge based on previous LGTM.

Author: Steve Orvell

src/wrappers/Node.js

@@ -306,6 +306,7 @@
         }
       } else {
         refWrapper = null;
+        refNode = null;
       }
 
       refWrapper && assert(refWrapper.parentNode === this);

test/js/HTMLOptionElement.js

@@ -38,7 +38,8 @@ suite('HTMLOptionElement', function() {
 
     var option = new Option(' more  text  ');
     assert.equal(option.text, 'more text');
-    assert.equal(option.value, 'more text');
+    // on IE10, the value includes the surrounding spaces; trim to workaround
+    assert.equal(option.value.trim(), 'more text');
     assert.isFalse(option.defaultSelected);
     assert.isFalse(option.selected);
 

test/js/Node.js

@@ -290,4 +290,18 @@ suite('Node', function() {
     assert.equal(unwrap(clone).innerHTML, '<a></a>');
   });
 
+  test('insertBefore', function() {
+    var parent = document.createElement('div');
+    var c1 = document.createElement('div');
+    var c2 = document.createElement('div');
+    var c3 = document.createElement('div');
+    parent.insertBefore(c3);
+    parent.insertBefore(c2, c3);
+    parent.insertBefore(c1, c2);
+    
+    assert.equal(parent.firstChild, c1);
+    assert.equal(c1.nextElementSibling, c2);
+    assert.equal(c2.nextElementSibling, c3);
+  });
+
 });