Websocket is undefined about sockette HOT 8 CLOSED

No problem~!

You shouldn't modify any node_modules/** code -- all your changes will be immediately lost anytime you install another dependency or if/when a coworker tries to replicate your setup.

Instead, you should do this in your testing setup file, or at the top of test files (repetitive):

// test/setup.js
global.WebSocket = require('ws');

from sockette.

Hey @tinchoz49 👋

No I would not, sorry. It would be meaningless to Sockette. You should simply do this instead:

global.WebSocket = require('ws');

This would only be set in your Node.js environment, leaving window.WebSocket intact for your browser bundle(s). If that's not the case, then you will need to take a closer look at your build config because you should never have server code/deps in your front-end bundles. I can maybe help with this part if you have questions or issues.

The above will give you full support 👍

from sockette.

Hi, this isn't to do with Sockette.

There's no WebSocket class in node.js, so you have to add one to the global scope.

Not sure if mock-socket does that or if you have to do it manually. However, I did notice in the first paragraph it says it's"unstable" so maybe you should find another solution?

I've answered elsewhere in these issues my recommendation. Eventually I'll include tests in this repo so that there's an example to follow, but no time now =\

Good luck!

from sockette.

Thanks Lukeed,
Just info for those who have the same problem. I solve it by installing ws and import it (see below) in the first line of node_modules/sockette/dist/sockette.js
const WebSocket = require('ws');

from sockette.

it works! thanks again ^^

from sockette.

Hi @lukeed how are you?

There's no WebSocket class in node.js, so you have to add one to the global scope.

Would you accept a PR to change the behavior so we can do something like this:

const ws = new Sockette('ws://localhost:3000', {
  ws: require('ws')
});

By default would be the window.WebSocket if is the browser and throw an error if you don't set a ws for nodejs

from sockette.

Don't worry 😃 I understand. Thanks for the quick response!

from sockette.

Of course! Feel free to ping me if you have issues with setting that up~

from sockette.