fails to test with the revert reason string with the Ganache GUI v2.0.1

[changhwan0813]

with the Ganache GUI v2.0.1 - which is the latest stable release - shouldFail.reverting.withMessage in v0.3.2 or expectRevert in master both fails to test the reason string.

Because a regex to test a nodeInfo cannot match a pre-release or a meta-data specified in a semver documentation - https://semver.org/#spec-item-9 and Ganache GUI v2.0.1 uses ganache-core v2.5.5-beta.0 - nodeInfo is EthereumJS TestRPC/v2.5.5-beta.0/ethereum-js

openzeppelin-test-helpers/src/shouldFail.js

Line 38 in 3eff5bc

const matches = /TestRPC\/v([0-9.]+)\/ethereum-js/.exec(nodeInfo);

openzeppelin-test-helpers/src/expectRevert.js

Line 29 in 34868ca

const matches = /TestRPC\/v([0-9.]+)\/ethereum-js/.exec(nodeInfo);

expectRevert output:

openzeppelin-test-helpers WARN       expectRevert: revert reason checking only supported on Ganache v2.2.0 or newer.

[frangio]

Thanks for reporting @bebeerd! There's a fix for this in #46. Would you mind testing it?

[changhwan0813]

@frangio It's working now 👍