DontPanicO / fastapi-distributed-websocket

A library to implement websocket for distibuted system based on FastAPI.
MIT License
52 stars 10 forks source link

Switch to tests that don't require Docker #4

Closed DontPanicO closed 1 year ago

DontPanicO commented 1 year ago

Rationale

At the moment, tests requires Docker and docker-compose to run. This make it difficult to rely on github actions workflows for testing. Even if those tests strictly reproduce the most common use case in a real world manner, it should be better to have a test suite that can be run with pytest only (using starlette.testclient.TestClient where needed). Also we need to improve coverage, which will require a lot of functional tests (e.g. test matches, untag_broker_message and so on). Using Docker will become a unreasonable overhead when srtictly required only by a very small segment of the test suite.