Conversation
|
|
|
||
| it.concurrent( | ||
| it( | ||
| 'connects to a room', | ||
| async () => { |
There was a problem hiding this comment.
suggestion(if-minor): I am assuming these were removed becaue bun doesn't support it.concurrent. Does removing these slow down the e2e tests significantly? And if so, is it worth trying to make these conditionally concurrent depending on if the test runner supports it.concurrent or not?
There was a problem hiding this comment.
makes sense, I had this initially and removed it as it seemed less elegant than just removing the concurrent version but in the long run it might pay off to make the change now!
No description provided.