-
Notifications
You must be signed in to change notification settings - Fork 5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Provide Test Cases for Web3 1.0 Support #3618
Comments
Here are the tests And the specs for |
Here's a first pass at a draft of Step A. Some of the actual tests are filled in, but toward the bottom there are some empty tests just outlining what needs to be validated. https://gist.github.com/ryan-rowland/54218bde11170734acb276a3564ac06b |
@ryan-rowland , very nice! I've updated the document. At this point it should be much easier for anyone to get started. |
Running some tests locally using I'll update later when I have time. |
@ryan-rowland , i'm closing down here. For the case you still want to push this, just go on please. follow-up MetaMask/web3-provider-engine#189 & #3642 |
affects #3642 and MetaMask/web3-provider-engine#189
In order to clarify the implementation-requirements of the web3 1.0 support, test-cases should be provided.
The test-cases can be used for both web3 1.0 support implementation paths:
Step One
Provide simple js code files, which demonstrate the web3 1.0 access (which should fail)
from #3642 (comment)
Step Two
Embed those code files into the MM test system.
Draft/Starting-Point: https://gist.github.com/ryan-rowland/54218bde11170734acb276a3564ac06b
References
from: #3618 (comment)
web3 original test-cases
https://github.com/ethereum/web3.js/blob/1.0/test/helpers/test.subscription.js
https://github.com/ethereum/web3.js/blob/1.0/test/eth.subscribe.js
Subscription relevant specs:
https://web3js.readthedocs.io/en/1.0/web3-eth-subscribe.html
The text was updated successfully, but these errors were encountered: