Always send output to flake callback

[mike-eason]

This change ensures that the protractor flake callback will always receive the output log even if the first test pass succeeds with no failures.

[NickTomlin]

Apologies for the very delayed response.

I think this sounds fine. I'm curious what is your use case?

Also, if possible could you include a unit test for this?

Thanks!

[mike-eason]

Hi there,

Our use case is that we want to send the log in an email at the end of the test pass. Currently the log will only be passed to the callback function if there was an error with any of the specs, we want the log regardless.

I'll add a unit test when I am back at work tomorrow.

Thanks.

[NickTomlin]

Great thanks for that detail makes sense!

[mike-eason]

Hi again Nick,

I have added a unit test as requested.

[ntomlinnf]

I'll troubleshoot that failure (looks like a typing issue for node 6.x) and merge this in tomorrow.

[NickTomlin]

🎉 This PR is included in version 3.4.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[jayvdb]

This appears to have caused #102

[NickTomlin]

This was reverted because we thought it might have been the root of #102 but it was in fact not. I've opened #105 to restore it.