Skip to content

Commit

Permalink
Fix CI (#184)
Browse files Browse the repository at this point in the history
* Fix tox config, and add newer pythons to CI

* Pacify mypy

* typing is hard

* isort was feeling left out

* mypy again

* :shaking fist: isorrrrrrtttt!!!

* Run autobahn tests on a version of ubuntu that exists

* install py2 for autobahn tests

* newer setup-python

* 🫡 to autobahntestsuite -- you served us well, but now you are dead

autobahntestsuite is abandoned, only runs on python 2, and has
dependencies like wsaccel that are *also* abandoned and *don't* run on
python 2. we could probably keep it limping along for a while with
enough effort, but I don't think it's worth it.
  • Loading branch information
njsmith committed Apr 12, 2023
1 parent a7d9ff0 commit c0a1079
Show file tree
Hide file tree
Showing 13 changed files with 33 additions and 600 deletions.
29 changes: 3 additions & 26 deletions .github/workflows/ci.yml
Expand Up @@ -17,11 +17,12 @@ jobs:
- 3.8
- 3.9
- "3.10"
- pypy-3.7
- "3.11"
- pypy-3.9

steps:
- uses: actions/checkout@v2
- uses: actions/setup-python@v2
- uses: actions/setup-python@v4
with:
python-version: ${{ matrix.python-version }}
- name: Install tox
Expand All @@ -37,27 +38,3 @@ jobs:
- uses: codecov/codecov-action@v1
with:
file: ./coverage.xml

autobahn:
runs-on: ubuntu-18.04
strategy:
matrix:
side: [client, server]

steps:
- uses: actions/checkout@v2
- uses: actions/setup-python@v2
with:
python-version: 3.8
- name: Install tox
run: |
python -m pip install --upgrade pip setuptools
pip install --upgrade tox
- name: Initialize tox envs
run: |
tox --parallel auto --notest -e autobahn
- name: Run autobahn
run: |
tox --parallel 0 -e autobahn
env:
SIDE: ${{ matrix.side }}
5 changes: 0 additions & 5 deletions .gitignore
Expand Up @@ -13,8 +13,3 @@ coverage.xml
docs/build
build
dist

compliance/reports
compliance/auto-tests-server-config.json
compliance/auto-tests-client-config.json
compliance/autobahntestsuite-venv/
5 changes: 0 additions & 5 deletions MANIFEST.in
@@ -1,13 +1,8 @@
graft src/wsproto
graft compliance
graft example
graft docs
graft test
graft bench
prune docs/build
prune compliance/reports
prune compliance/auto-tests-server-config.json
prune compliance/auto-tests-client-config.json
prune compliance/autobahntestsuite-venv
include README.rst LICENSE CHANGELOG.rst tox.ini
global-exclude *.pyc *.pyo *.swo *.swp *.map *.yml *.DS_Store .coverage
31 changes: 0 additions & 31 deletions README.rst
Expand Up @@ -91,37 +91,6 @@ And wsproto will issue events if the data contains any WebSocket messages or sta
Take a look at our docs for a `full list of events
<https://wsproto.readthedocs.io/en/latest/api.html#events>`!

Testing
=======

It passes the autobahn test suite completely and strictly in both client and
server modes and using permessage-deflate.

If you want to run the compliance tests, go into the compliance directory and
then to test client mode, in one shell run the Autobahn test server:

.. code-block:: console
$ wstest -m fuzzingserver -s ws-fuzzingserver.json
And in another shell run the test client:

.. code-block:: console
$ python test_client.py
And to test server mode, run the test server:

.. code-block:: console
$ python test_server.py
And in another shell run the Autobahn test client:

.. code-block:: console
$ wstest -m fuzzingclient -s ws-fuzzingclient.json

Documentation
=============
Expand Down
253 changes: 0 additions & 253 deletions compliance/run-autobahn-tests.py

This file was deleted.

0 comments on commit c0a1079

Please sign in to comment.