WebSocket support

[meejah]

This builds on the new iosim.IOPump-based tests to add WebSocket support with proper tests.

Needs:

• [x] branch-coverage of tests in a few more places
• [x] there are two new_protocol_ws implementations; unify
• [ ] possibly more cleanup
• [ ]

[lgtm-com[bot]]

This pull request introduces 14 alerts when merging 30688f638c7f66a1ebffa160ac734ad62f30518a into de8e0f03993cb03d28da88e0cd5e17fc35fd99f5 - view on LGTM.com

new alerts:

• 5 for First parameter of a method is not named 'self'
• 5 for Unused import
• 1 for Use of the return value of a procedure
• 1 for Non-callable called
• 1 for Unreachable code
• 1 for Wrong number of arguments in a call

[lgtm-com[bot]]

This pull request introduces 13 alerts when merging 141709bf62cca0254c79110563ea528fee1114fa into de8e0f03993cb03d28da88e0cd5e17fc35fd99f5 - view on LGTM.com

new alerts:

• 5 for First parameter of a method is not named 'self'
• 5 for Unused import
• 1 for Use of the return value of a procedure
• 1 for Non-callable called
• 1 for Unreachable code

[lgtm-com[bot]]

This pull request introduces 13 alerts when merging 553158da633f167488b4d4f5f380444b6ed09c06 into de8e0f03993cb03d28da88e0cd5e17fc35fd99f5 - view on LGTM.com

new alerts:

• 5 for First parameter of a method is not named 'self'
• 5 for Unused import
• 1 for Use of the return value of a procedure
• 1 for Non-callable called
• 1 for Unreachable code

[codecov[bot]]

Codecov Report

Merging #23 (6bd063a) into master (0de4f23) will increase coverage by 0.13%. The diff coverage is 99.44%.

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
+ Coverage   99.07%   99.21%   +0.13%     
==========================================
  Files           5        7       +2     
  Lines         326      512     +186     
  Branches       48       55       +7     
==========================================
+ Hits          323      508     +185     
  Misses          1        1              
- Partials        2        3       +1     

Impacted Files	Coverage Δ
src/wormhole_transit_relay/usage.py	98.68% <98.68%> (ø)
src/wormhole_transit_relay/server_state.py	99.43% <99.43%> (ø)
src/wormhole_transit_relay/server_tap.py	100.00% <100.00%> (ø)
src/wormhole_transit_relay/transit_server.py	100.00% <100.00%> (+0.51%)	:arrow_up:

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0de4f23...6bd063a. Read the comment docs.

[lgtm-com[bot]]

This pull request introduces 9 alerts when merging aa58b85ace2fb52a0899b2ee24fd6eb933da4e47 into 0de4f23a292f7d7724a90786c5b3fcd6efdaa6cf - view on LGTM.com

new alerts:

• 5 for First parameter of a method is not named 'self'
• 2 for Use of the return value of a procedure
• 1 for Non-callable called
• 1 for Unreachable code

[lgtm-com[bot]]

This pull request introduces 9 alerts when merging 807dfc1c183337693eb2e7a45cccf401a2b6e2a5 into 0de4f23a292f7d7724a90786c5b3fcd6efdaa6cf - view on LGTM.com

new alerts:

• 5 for First parameter of a method is not named 'self'
• 2 for Use of the return value of a procedure
• 1 for Non-callable called
• 1 for Unreachable code

[lgtm-com[bot]]

This pull request introduces 9 alerts when merging 6bd063a91771b085a9a850dd257afe29de8b07e8 into 0de4f23a292f7d7724a90786c5b3fcd6efdaa6cf - view on LGTM.com

new alerts:

• 5 for First parameter of a method is not named 'self'
• 2 for Use of the return value of a procedure
• 1 for Non-callable called
• 1 for Unreachable code