Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Fix tests for web3-provider-ws@1.0.0-beta.33 #94

Merged
merged 2 commits into from
Apr 12, 2018

Conversation

AusIV
Copy link
Contributor

@AusIV AusIV commented Mar 31, 2018

In web3-provider-ws v1.0.0-beta.33 they changed the callback behavior
from cb(err, response) to cb(response), but it seems some instances
make the callback without passing the error while other instances make
the callback with an error and response. This change deals with both
possible inputs to the callback function.

An alternative might be to pin web3-provider-ws to 1.0.0-beta.30.

In web3-provider-ws v1.0.0-beta.33 they changed the callback behavior
from cb(err, response) to cb(response), but it seems some instances
make the callback without passing the error while other instances make
the callback with an error and response. This change deals with both
possible inputs to the callback function.

An alternative might be to pin web3-provider-ws to 1.0.0-beta.30.
@AusIV
Copy link
Contributor Author

AusIV commented Mar 31, 2018

This addresses the build failures in #92 & #93.

@AusIV
Copy link
Contributor Author

AusIV commented Apr 4, 2018

Also addresses the build failures in #95

@benjamincburns
Copy link
Contributor

@AusIV Thanks for this! I was just about to go track this down!

@gnidan @cgewecke are you guys aware of/impacted by this change in truffle-contract?

@benjamincburns benjamincburns merged commit d8028b4 into trufflesuite:develop Apr 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants