You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think there is an opportunity to improve some of the tests in dot/rpc
TestWSConn_HandleComm is one test that contains a lot of tests. It could be structured better by using the testcases-and-for-loop pattern or by using sub-tests. (in /dot/rpc/subscription/websocket_test.go)
Test more cases in dot/rpc/websocket_test.go. For some calls, only error cases are tested.
We need tests for subscription based calls.
Other information and links
The text was updated successfully, but these errors were encountered:
Issue summary
I think there is an opportunity to improve some of the tests in
dot/rpc
TestWSConn_HandleComm
is one test that contains a lot of tests. It could be structured better by using the testcases-and-for-loop pattern or by using sub-tests. (in/dot/rpc/subscription/websocket_test.go
)dot/rpc/websocket_test.go
. For some calls, only error cases are tested.Other information and links
The text was updated successfully, but these errors were encountered: