matcher for is

[ljharb]

I tried expect(foo).to.be.a('bar') to be a nicer form of expect(foo.is('bar')).to.equal(true), but it didn't work. What do you all think about a matcher for this?

[keithamus]

👍 from me

[ahstro]

Shouldn't match do this? That is expect(foo).to.match('bar')?

[ljharb]

"match" implies a selector/regex, and possibly multiple elements. is/be imply one exact identity.

[jeffcarbs]

The .match() matcher seems to actually just be a wrapper around .is(). I just submitted a PR to make it more consistent and support any valid EnzymeSelector (see: #121).

Assuming that PR is merged, you'll be able to make assertions like:

expect(foo).to.match('div')
expect(foo).to.match('div.some-class')
expect(foo).to.match('MyComponent')
expect(foo).to.match(MyComponent)

That being said, I do agree that something like expect(foo).to.be.a(MyComponent) would be more clear. Perhaps they'd be open to a rename or you could alias is somehow in your codebase. Not too familiar with chai, tbh.

[ayrton]

@ljharb - @ahstro is correct, the behaviour you want is currently what match does, that being said we could have done a better job at naming this and with @jcarbo's PR merged I think we should add support for .to.be.a 👯