Skip to content
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

"npm test" failed #183

Open
gerisse opened this issue Oct 30, 2020 · 6 comments
Open

"npm test" failed #183

gerisse opened this issue Oct 30, 2020 · 6 comments

Comments

@gerisse
Copy link

gerisse commented Oct 30, 2020

hi
After npm install, i run npm test . It failed , with this message :
Contract: RocketNodeManager admin: can set a node's trusted status: Error: value out-of-bounds (argument="_index", value="-1", code=INVALID_ARGUMENT, version=abi/5.0.0-beta.153)
an idea ? thanks

@darcius
Copy link
Member

darcius commented Oct 31, 2020

Hey @gerisse! Are you getting this error on master branch? I just ran the tests on master and they all passed for me.

What version of node are you running? You can find that out by typing node -v in terminal.

@gerisse
Copy link
Author

gerisse commented Oct 31, 2020

yes, Darcius, i 'm on master branch. node -v gives v15.0.1 . Thanks

@darcius
Copy link
Member

darcius commented Nov 3, 2020

@gerisse Ah ok, found the issue! We hadn't tested on > Node 13 LTS before, so I upgraded to Node 14 LTS which caused issues with our versions of Ganache / Truffle. Updated those and then encountered your error which it seems was due to a bug in that unit test which was being ignored in previous node versions.

If you do a fresh pull + npm install && npm run test, it should all work for you now 👍

@gerisse
Copy link
Author

gerisse commented Nov 4, 2020

ok , so i did the update , and choosed the node 14 version. But the same issue appears :-(

@darcius
Copy link
Member

darcius commented Nov 4, 2020

That is odd, it appears fixed for me. Did you run a git pull before running the tests again?

@gerisse
Copy link
Author

gerisse commented Nov 11, 2020

yes, i pulled it before

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants