LGTM

Some of the autobahn test keep failing locally due to tests timing out. If there is any actual issue unrelated to timeout, please let me know.

We can remove line 525, where we set the connection status, because we call #onSocketClose.

And the change in #onSocketClose with optional chaining brakes the autobahn. Because it changes the error code to 1006 instead of 1005.

Probably need to make #onSocketClose accept a parameter. If we pass a symbol kFail than we handle the onFail case and code stays at 1005. Or maybe use the wasClean variable?

The issue is that onSocketClose is not appropriate to call here. You need to handle setting the state to CLOSED and emitting a close event in a separate function that both onFail and onSocketClose can use.

Sorry for not getting to back to this earlier. After KhafkaDev said there were autobahn tests failing, I wanted to properly setup the tests locally to make sure they were working as expected. Although the test seem to be running fine on CI, it seems to fail test cases between 12.4.3 and 12.4.18 locally, while passing on CI. Does anyone have any insight on why this may be the case? And in some cases, re-running the same test locally seem to produce different fails. I have tried to do the autobahn test on two different machines (macOS 15.1 & Windows 11), both seem to fail these 16 tests locally.

│ 12.4.3  │ 'FAILED'        │ 'UNCLEAN'       │ 399      │ null            │
│ 12.4.4  │ 'FAILED'        │ 'UNCLEAN'       │ 135      │ null            │
│ 12.4.5  │ 'FAILED'        │ 'UNCLEAN'       │ 91       │ null            │
│ 12.4.6  │ 'FAILED'        │ 'UNCLEAN'       │ 54       │ null            │
│ 12.4.7  │ 'FAILED'        │ 'UNCLEAN'       │ 181      │ null            │
│ 12.4.8  │ 'FAILED'        │ 'UNCLEAN'       │ 351      │ null            │
│ 12.4.9  │ 'FAILED'        │ 'UNCLEAN'       │ 630      │ null            │
│ 12.4.10 │ 'FAILED'        │ 'UNCLEAN'       │ 1022     │ null            │
│ 12.4.11 │ 'FAILED'        │ 'UNCLEAN'       │ 61       │ null            │
│ 12.4.12 │ 'FAILED'        │ 'UNCLEAN'       │ 209      │ null            │
│ 12.4.13 │ 'FAILED'        │ 'UNCLEAN'       │ 419      │ null            │
│ 12.4.14 │ 'FAILED'        │ 'UNCLEAN'       │ 722      │ null            │
│ 12.4.15 │ 'FAILED'        │ 'UNCLEAN'       │ 1147     │ null            │
│ 12.4.16 │ 'FAILED'        │ 'UNCLEAN'       │ 1066     │ null            │
│ 12.4.17 │ 'FAILED'        │ 'UNCLEAN'       │ 1052     │ null            │
│ 12.4.18 │ 'FAILED'        │ 'UNCLEAN'       │ 1042     │ null            │

I was unable to do any reliable testing locally as a result.