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

Failing test #24

Open
BlackTurtle123 opened this issue Jun 23, 2019 · 4 comments
Open

Failing test #24

BlackTurtle123 opened this issue Jun 23, 2019 · 4 comments

Comments

@BlackTurtle123
Copy link
Contributor

BlackTurtle123 commented Jun 23, 2019

When doing a clone from the project and running the tests as they are, there is one failing

ERROR: test_inject (tests.common.test_injector.TestInjector)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Users\bramV\Documents\PycharmProjects\WavesGatewayFrameworkQuart\tests\common\test_injector.py", line 53, in test_inject
    other_class_with_deps_instance = self._global_injector.get(other_class_with_deps_token)
  File "C:\Users\bramV\Documents\PycharmProjects\WavesGatewayFrameworkQuart\waves_gateway\common\injector.py", line 114, in get
    if self._dependency_map[token].instance is not None:
AttributeError: 'NoneType' object has no attribute 'instance'
@henning96
Copy link
Collaborator

Hello @BlackTurtle123,

I cannot confirm that. I did a clean install on my machine and the tests are still working. Are you using Python 3.5?

@BlackTurtle123
Copy link
Contributor Author

I am using Python 3.7

@henning96
Copy link
Collaborator

henning96 commented Jun 25, 2019 via email

@BlackTurtle123
Copy link
Contributor Author

BlackTurtle123 commented Jun 25, 2019

Ow okay, the git makes it sound like it's recommended to have python3.5, but not fixed on python3.5, so that was why I was thinking at least 3.5
My bad then
From readme

A framework to connect other cryptocurrencies to the Waves platform. Requires Python 3.5 or newer.

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