data-mocks icon indicating copy to clipboard operation
data-mocks copied to clipboard

Websocket Support

Open aidenscott2016 opened this issue 4 years ago • 10 comments

We have had a need to mock a WS server at OVO using this library. This work implements that.

See tests for usage

aidenscott2016 avatar Sep 14 '21 16:09 aidenscott2016

This looks pretty solid. I'll give this a run when I'm at my desktop tomorrow. I really appreciate contributions like this out of the blue!

grug avatar Sep 14 '21 20:09 grug

Aiden I need to see a dank meme before I can approve this work.

grug avatar Sep 15 '21 13:09 grug

@grug

This looks pretty solid. I'll give this a run when I'm at my desktop tomorrow. I really appreciate contributions like this out of the blue!

You're welcome. We needed to mock websockets any way and it didn't make sense two sets of mocking infra

Aiden I need to see a dank meme before I can approve this work.

It will take a long time to find a suitably dank meme for a PR of this calibre. Now I can see there is an appetite to upstream this I will more than likely clean this up on my next 10pc day next friday

aidenscott2016 avatar Sep 15 '21 14:09 aidenscott2016

@hamishdixon morley has aske me to remind you that you are a flim-flam

aidenscott2016 avatar Sep 15 '21 14:09 aidenscott2016

@aidenscott2016 please remind Morley that I am his dad.

grug avatar Sep 15 '21 14:09 grug

@grug this is ready for merge

aidenscott2016 avatar Sep 24 '21 12:09 aidenscott2016

I'm going to have to run this stuff locally and see how it all works as I feel like people can get pretty fancy with their websocket setups so I want to see how this stands up against some weird configurations.

grug avatar Sep 24 '21 12:09 grug

thumbs up emoji

aidenscott2016 avatar Sep 24 '21 13:09 aidenscott2016

@aidenscott2016 are you still okay with all of this? I'm thinking of merging and releasing today :)

grug avatar Oct 29 '21 08:10 grug

@grug ye man merge it

aidenscott2016 avatar Oct 21 '23 17:10 aidenscott2016