Fix: Ignore bug

[sunesimonsen]

Fixes: #212

Reproduces a bug where the user is trying to ignore a text node in to satisfy and fixes it.

[coveralls]

Pull Request Test Coverage Report for Build 788

• 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 91.379%

---

Totals
Change from base Build 784:	0.0%
Covered Lines:	504
Relevant Lines:	533

---

💛 - Coveralls

[alexjeffburke]

Interesting and good catch I'd say!

Just for my understanding, does this come from text nodes being compared as plain strings and thus we end up comparing a text node to an empty object that fails in "to satisfy" when recursing?

[sunesimonsen]

The problem is that the RHS is converted into a to satisfy object structure. That would mean we would end up comparing a string to the empty object.

[papandreou]

Great catch! I guess expect.it('to be a', 'DOMNode') would also have worked, as all the unexpected-dom types except DOMNodeList inherit from that. Still, there's certainly no harm in using any, and "to be an any" sounds funny :)

[sunesimonsen]

Great catch! I guess expect.it('to be a', 'DOMNode') would also have worked, as all the unexpected-dom types except DOMNodeList inherit from that. Still, there's certainly no harm in using any, and "to be an any" sounds funny :)

Good point. I think that would actually be better and more precise.

[sunesimonsen]

Released as unexpected-dom@4.6.2