unexpectedjs / should-be-unexpected

Should.js implemented with Unexpected.
3 stars 0 forks source link

should-be-unexpected

This is an attempt to implement a should.js facade using unexpected. There is some incompatabilities, but only where we believe that our behaviour is better than what is found in should.js.

Not all of the current API of should is implemented. I have taken a few projects that I know of, already using should.js, and I am using them to guide the work.

Projects whose test suites run with no errors:

Projects whose tests are not yet fully running:

tj/ejs@v1.0.0 Two tests are failing, due to .include and .match not being implemented. .match is at the point in time when these tests are written a simple string regex match - now .match is an abomination that does all sorts of magic. .include has since been removed from should.js. I'm still considering what the right course of action would be on this.

Incompatabilities

1. .eql(otherValue)

The following would be okay in should.js.

[1, 2, 3].should.eql({ '0': 1, '1': 2, '2': 3 });

You can turn that behavior off, by enabling the following configuration option in should.js.

should.config.checkProtoEql = true;

Unexpected behaves as checkProtoEql is true per default, and it cannot be changed. This is a very concious choice.

2. .containEql(otherValue)

The following would work in should.js.

({ b: 10 }).should.containEql({ b: 10 });

But it doesn't in unexpected, as it does not define the assertion 'to contain' for the object type.

3. .not.property(name, value)

The following would be allowed in should.js.

var user = { name: 'John', age: 42 };
user.should.not.have.property('age', 0);

Using the not flag with the 'to have property' assertion is not working when you provide a value. The reason this was deprectated is that we felt that the intention of a test like this is unclear. Not using this kind of assertions, will make you write better tests.

4: .length

With should.js you could:

({ length: 10}).should.have.length(10);

Unexpected has a type system, and the 'to have length' assertion is only defined for string and array like types. Thus this assertion will fail, as the subject is not an array nor an arguments-array.

5: .empty

({}).should.be.empty;

The above is valid with should. In unexpected, the 'to be empty' assertion is only implemented for values of type string or array-like.

6: NaN is not a number

NaN.should.be.a.number;

The above assertion will not throw when using should.js. It will when you are using unexpected. While javascript itself considers NaN to be a number, we could not find a single reason for why you would want your assertion framework to consider it a valid number. If you get a NaN, where you expected a number, you'll most definitely not get the behaviour that you wanted.