Skip to content

Remove Autobahn Test suite #117

@nhooyr

Description

@nhooyr

Old issue was #59 but I closed cause it wasn't worth spending time on at the time.

Now at the library is very stable and used I think it's a good idea to do this to make the dev experience better.

Summary

We currently use the autobahn test suite which is shipped as a python binary that runs a server and client that we use to test against a client and server ran by this library. This makes the tests fairly slow, a poor experience for debugging and adds more dependencies.

The tests ran by the suite are not very complicated and you can view all of them by checking the reports generated by the suite in ci/out/wstestServerReports and ci/out/wstestClientReports after running the tests or see the artifacts in test job in CI. The description of what they do and how it looks on the wire is included in the report.

We need to convert each of these tests into pure Go tests. We should follow the exact behaviour for now and add extra tests as necessary for more coverage of this library.

It'll be a very repetitive job but writing thoughtful helpers should alleviate the pains.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions