Fixed issue with undefined being wrapped as nodes while constructing …

…ReactWrapper (fixes #554) (#576)
enzymejs · Sep 6, 2016 · 5049574 · 5049574
1 parent f40dae8
commit 5049574
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/src/ReactWrapper.jsx b/src/ReactWrapper.jsx
@@ -87,7 +87,9 @@ export default class ReactWrapper {
     } else {
       this.component = null;
       this.root = root;
-      if (!Array.isArray(nodes)) {
+      if (!nodes) {
+        this.nodes = [];
+      } else if (!Array.isArray(nodes)) {
         this.node = nodes;
         this.nodes = [nodes];
       } else {

diff --git a/test/ReactWrapper-spec.jsx b/test/ReactWrapper-spec.jsx
@@ -3005,5 +3005,18 @@ describeWithDOM('mount', () => {
         expect(wrapper.name()).to.equal('div');
       });
     });
+
+    describe('.ref()', () => {
+      it('unavailable ref should return empty nodes', () => {
+        class WithoutRef extends React.Component {
+          render() { return <div />; }
+        }
+        const wrapper = mount(<WithoutRef />);
+        const ref = wrapper.ref('not-a-ref');
+
+        expect(ref.length).to.equal(0);
+        expect(ref.isEmpty()).to.equal(true);
+      });
+    });
   });
 });