-
Notifications
You must be signed in to change notification settings - Fork 27
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
Missing anvil-zksync
inside devContainer to run just test-z
#216
Comments
- add tests for install command - add tests for impacted commands - run - test - deploy - compile - using helper utility to handle moccasin toml scenarios - add auto-install lib/github and lib/pypi folders when install (fix issue Cyfrin#209) - fix issues with tests using COMPLEX_PROJECT path and not temporary one - update test constants WARNING: just test-z and live test KO since anvil-zksync is not installed (reported Cyfrin#216)
I also have a script for #bin/bash
# Install ZKVyper
echo "Installing ZKVyper"
# Get arg from command line or set stable one
VERSION="${1:-"1.5.7"}"
# Download and install ZKVyper
wget https://github.com/matter-labs/era-compiler-vyper/releases/download/$VERSION/zkvyper-linux-amd64-gnu-v$VERSION -O zkvyper
chmod u+x zkvyper
sudo rm -rf /usr/local/bin/zkvyper
sudo mv zkvyper /usr/local/bin
# Check installation
zkvyper --version |
Next PR will come soon with stable
An example, after running a few |
Potential issue coming from # boa_zksync/__init__.py
def set_zksync_test_env(node_args=(), nickname=None):
return boa.set_env(
ZksyncEnv(rpc=AnvilZKsync(node_args=node_args), nickname=nickname)
)
# boa_zksync/node.py
class AnvilZKsync(EthereumRPC):
...
def __del__(self):
stop_subprocess(self._test_node) The del is not triggered properly I think. |
* feat: auto-install dependencies * feat: implement tests for install command - add tests for install command - add tests for impacted commands - run - test - deploy - compile - using helper utility to handle moccasin toml scenarios - add auto-install lib/github and lib/pypi folders when install (fix issue #209) - fix issues with tests using COMPLEX_PROJECT path and not temporary one - update test constants WARNING: just test-z and live test KO since anvil-zksync is not installed (reported #216) * fix: refactor auto-install tests * fix: lint --------- Co-authored-by: s3bc40 <s3bc40@gmail.com>
Is it something to add inside
DockerFile
, or should it be the dev to install it on his side?The text was updated successfully, but these errors were encountered: