8000 Invite follow-up: end-to-end integration tests · Issue #72 · ssbc/go-ssb-room · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Invite follow-up: end-to-end integration tests #72

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.

Sign up for GitHub

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

Jump to bottom
Open
cryptix opened this issue Mar 10, 2021 · 0 comments
Open

Invite follow-up: end-to-end integration tests #72

cryptix opened this issue Mar 10, 2021 · 0 comments
Labels
Go golang related stuff Testing everything that's testing related

Comments

@cryptix
Copy link
Member
cryptix commented Mar 10, 2021

All the important parts have unit tests.

  • HTML Forms are checked to have the fields
  • HTTP POST endpoints are checked
  • sqlite crud interface is tested, including .Consume() adding to the allow list

... butt I would prefer to also start testing the whole thing in one piece, including the muxrpc stuff. Didn't want to whip all this up on the back of #59, tho.

Qs to answer:

  • with or without https?
  • Do we run it as a _test or whip up scripts to run cmd/server as an executable?
  • what's does the client test driver look like?

Also the current way to spawn the web + muxrpc stuff is only in cmd/server so a proper e2e suite either needs to run that or refactor the server code to become a slim CLI flag passing shim.
@cryptix cryptix added Go golang related stuff Testing everything that's testing related labels Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Go golang related stuff Testing everything that's testing related
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cryptix
0