-
Notifications
You must be signed in to change notification settings - Fork 33
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
Issue using getAccountAddress #234
Comments
@Luze26 Does this error still persist after upgrading your Flow CLI? |
Seems like I still have the error. |
Hmmm ok, looking into this... |
I'm having trouble reproducing the issue. Can you post the node.js version that you are using? Wondering if this could be related. Also is there any chance you could post a code snippet of the failing test? |
Thx for looking into it.
I then updated the dependencies to use the stable cadence version of flow-js-testing. And I get the same error, I also get this error with version 0.5.0. I'll try to understand more the issue. |
So Line 140 in 1b630d2
So it fails, if I change the config in flow.json to directly include the key it works well. |
Oh, nice catch :). Yeah this can be worked around in the library, I can make a PR. |
awesome, thx a lot |
When using flow-js-testing 0.6.0-stable-cadence.3 and the CLI v1.15.0-cadence-v1.0.0-preview.13.
When I run:
I get the following error:
The text was updated successfully, but these errors were encountered: