Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement a test server #1

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Implement a test server #1

wants to merge 8 commits into from

Conversation

fwcd
Copy link
Member

@fwcd fwcd commented Feb 20, 2022

Make the lighthouse server URL customizable and add a simple test server with a web interface that can replace the official lighthouse server for testing purposes (LighthouseTestServer).

To do:

  • Add test server target
  • Implement test server
    • Add web interface (which includes a basic client that renders displays to a canvas)
    • Handle /websocket endpoint on the server side
  • Use async/await in the test server (specifically in the ConnectionHandler)
    • Fix macOS build (there seem to be a few differences in how Linux and macOS Vapor handle asynchrony, we should address these)
  • Document how to run test server (swift run LighthouseTestServer) and how to use it (--url ws://localhost:8080/websocket arg for demo etc.)
  • Perhaps modularize into multiple packages so consumers that just need the lighthouse client don't have to pull the Vapor dependencies for the test server?

@fwcd fwcd added the enhancement New feature or request label Feb 20, 2022
@fwcd fwcd marked this pull request as ready for review February 20, 2022 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant