Explicitly return TimeoutError on WebSocketResponse.close() (#1084)

[achimnol]

What do these changes do?

Revises WebSocketResponse.close() patch by explicitly returning TimeoutError with close code 1006 in #1084 for #1002.

Are there changes in behavior for the user?

Nothing.

Related issue number

#1002

Checklist

• I think the code is well written
• Unit tests for the changes exist
• Documentation reflects the changes

[codecov-io]

Current coverage is 97.62% (diff: 100%)

Merging #1089 into master will decrease coverage by 0.08%

@@             master      #1089   diff @@
==========================================
  Files            28         28          
  Lines          6407       6432    +25   
  Methods           0          0          
  Messages          0          0          
  Branches       1078       1094    +16   
==========================================
+ Hits           6260       6279    +19   
- Misses           80         81     +1   
- Partials         67         72     +5   

Powered by Codecov. Last update 4167156...6a07366

[asvetlov]

Just assert (yield from ws.close()) would be fine.

[achimnol]

I have fixed the branch according to your comments and rebased!
...ough... forced push has made your comments invisible... 😢

[asvetlov]

Thanks!

[lock]

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.