Skip to content

Add Autobahn echo server with Dockerfile#101

Open
djones6 wants to merge 1 commit intomasterfrom
autobahn-in-docker
Open

Add Autobahn echo server with Dockerfile#101
djones6 wants to merge 1 commit intomasterfrom
autobahn-in-docker

Conversation

@djones6
Copy link
Contributor

@djones6 djones6 commented Oct 4, 2019

While testing Kitura-WebSocket, I wanted to run the Autobahn tests against my changes. The existing guide in AutobahnTests.md is helpful, but it's a pain to have to install all the dependencies for the client, deal with versions of Python, missing macOS headers and so on.

I found that there's an autobahn docker image (https://github.com/crossbario/autobahn-testsuite/tree/master/docker) that we can use for the client. And we can do everything within Docker if we dockerize the echo server and run both on a network.

To that end, I've created an Autobahn subproject containing a Dockerfile and a run.sh which wil build the server into a wsserver image, run that as a container on a Docker network called autobahn, and then run the autobahn client (on the same network) targetting wsserver:9001. At the end of the run, it'll open the HTML report that the client generates.

Add run script for building server and running client tests

WORKDIR /WebSocketEchoServer

RUN swift build
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Release mode? Or not...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm good question. I'd gone for debug mode since we're dealing with tests, but as this is essentially acceptance testing, perhaps it should be release mode.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants