-
Notifications
You must be signed in to change notification settings - Fork 10
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
Review Nodemanager unit tests with Affix integration. #91
Comments
Curiously, a test that fails is the ut_nm_simple.r2py because the overloading of openconnection does not seem to work properly. However adding the simple line: "dy_import_module_symbols('time.r2py')" right above 'dy_import_module_symbols('nmclient.r2py')" fixes the issue. That is, if we import time.r2py, the overloading of the openconnection seems to work and the test passes. The same behavior can be seen in some of the other tests that are failing and are not importing time.r2py. |
This ticket description is missing useful information. Why do you think overloading is the root cause?
Is this what I'm supposed to see? Why or why not? |
Alternatively, is this what is expected:
|
I changed the nm_client default timeout from 15 to 150. And here is what I got
|
I am attaching a tarball with all the latest nodemanager unit tests along with the latest version of the nodemanager. Both the nodemanager and the unit tests use the Affix framework to communicate. Someone will need to review the changes and test out the unit tests on different platform. Out of all the unit tests, about ~7 of the unit tests fail. (Some of them due to us not being able to overload the openconnection call properly with affix.openconnection)
To run all the tests properly, check out a fresh version of all files from svn. Untar the file nodemanager_and_unit_tests.tar.gz. Copy nmmain.py into repy_v2/nodemanager/. Untar ut_nm_tests.tar.gz into the directory repy_v2/nodemanager/tests/
The text was updated successfully, but these errors were encountered: