[test] rosbridge v2 protocol testing: advertise/unadvertise op #38
Conversation
|
Please don't merge as some test cases will not pass as spec not completely implemented. |
qiuzhong
force-pushed
the
protocol
branch
2 times, most recently
from
8953958
to
f7ef9c7
Compare
qiuzhong
changed the title
qiuzhong
force-pushed
the
protocol
branch
2 times, most recently
from
aa6fb87
to
940ba8a
Compare
|
I'm blocked by the fact that the CI on Windows failed while succeeded on Linux. I'm fixing it. |
|
I suppose you have more than one WebSockets server instances, which will be listening the same port. This may lead to the TCP error, please try use different port or guarantee there is only one server instance at any time. |
advertise/unadvertise operation test cases added (+11).
|
Using different ports for multiple server instances doesn't help. I'm trying to avoid creating/destroying multiple bridge servers in my test cases to see if it can help. |
advertise/unadvertise operation test cases added (+11).
|
It seems the windows ci doesn't complain :) |
|
Finally it passed on Windows CI! |
|
So can I merge this PR now, if you will not change it any more? |
|
I think you can merge it now. |
|
OK, merging... |
advertise/unadvertise operation test cases added.