expect/assert return different promises

[domenic]

expect/assert return different promises

[domenic]

.then(function () { callback(); }, function(failure) { callback(failure); });

Or just use .notify(callback)

We originally used the expect() syntax. However, this appears to return the original promise from protractor (or some variation of it)

This should not be the case. Can you produce a repro? Preferably one that does not involve external libraries like protracter?

unless an error message is specified ??? - in which case it throws an exception

This should also not be the case. Again, a repro would be quite helpful.

[networkerror]

I think I have two issues in here. Should we split them?

Here's what we found on assert's throwing exceptions when error text is supplied.

// This one throws an exception.
assert.eventually.include(promise, expState, 'Custom text!')
  .then(callback);

// This one does not
assert.eventually.include(promise, expState)
  .then(callback);

We tracked this behavior down to these lines of code.
https://github.com/domenic/chai-as-promised/blob/master/lib/chai-as-promised.js#L349-L353

[domenic]

It does not throw an exception. It passes a rejection handler that throws, which causes the returned promise to be rejected.

What type of promises are you using? Are they Promises/A+ compliant?

[networkerror]

It's webdriver (protractor).

We're trying to use chai-as-promised with cucumber.js and protractor.

[domenic]

If that throws an exception, then they are not Promises/A+ promises, and cannot be used with Chai as Promised.

[networkerror]

Sounds like protractor (web driver) promises are the root of our problems. Not sure how we'll deal with that...

Thanks for the quick responses!

[domenic]

I wonder if a new major version of Chai as Promised could bundle a promise library with it, and wrap the given promises into real Promises/A+ promises. This would allow it to work with jQuery promises as well. Hmm.

This would probably require removing the UMD wrapper though and just going CommonJS.

[domenic]

In the meantime, I should (a) test to make sure the problem is as I expected; (b) if so, blacklist webdriver promises like I do jQuery promises.

[domenic]

Realized I am already tracking this as #58.