-
Notifications
You must be signed in to change notification settings - Fork 147
Conversation
… for scripting service client
… literals for py 2 3 compat
…ered dictionary and we want to test against the contents in the input stream it was written to
…ountered, modified tests to reflect this change
…ql tools service side to finalize the event/parameter/response types
… code coverage with the test run
…s which would require another loop without invoking readchunk() which would block if there was no more data to read, leaving a message unprocessed in the buffer
@MrMeemus, |
self.assertFalse(test_client.request_thread.isAlive()) | ||
self.assertFalse(test_client.response_thread.isAlive()) | ||
self.assertEqual(threading.active_count(), 1) | ||
|
||
# Response thread should have reach EOF during test execution which should end the response thread. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing an assert after the comment? #Resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
test_client.shutdown() | ||
self.assertEqual(threading.active_count(), 1) | ||
# Disabling this test in case this scenario is valid in the future |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Disabling this test in case this scenario is valid in the future [](start = 4, length = 66)
Can we enable the test as passing with the current behavior. Then, when the scenario is valid, it will fail and we can update the test? #Resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! In reply to: 289101128 [](ancestors = 289101128) |
@@ -2,3 +2,5 @@ enum34==1.1.6 | |||
future==0.16.0 | |||
pip==9.0.1 | |||
setuptools==30.4.0 | |||
coverage==4.3.4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove extra line above #Resolved
Submitting initial PR to get the show going.
There is one test issue when ran on mac regarding the way mac treats CRLF in a text file. Investigating a fix now.