PASS: the test servers now start on a port the system picks, and the stop test again sends its long poll through the listener the app opens, which fixes the earlier finding.
Model: opus-5-5
Review: FAIL (needs-rework)
src/popup/views/settings.js,saveEndpoint(): a save refused because the field no longer shows the URL gives the user no answer. The last flash is "Testing…
Rework for sneak/neoirc#212 (comment): startHTTP now keeps the listener it binds in a listener field on Server, and internal/server/export_test.go has a…
Rework for sneak/AutistMask#693 (comment):
- Fixed. The failed-request test runs for all four cases, and every check this PR adds has a test that fails…
FAIL (needs-rework)
internal/server/server_test.go,TestStopWaitsForRequestInProgress: this test used to send its long poll to the configured port, through the listener the start…
PASS. PrefixHandler.flushLoop now takes a waiting stop request first and starts no flush once the stop deadline has passed, the new test fails on the code before the change, and the two…
bcdffa56a4
fix: Settings stores the URL on screen when another page stored a different one (closes #688)
Also in scope: the AS and peer handler tests can fail the same way through their two-second flush timer, as the review of sneak/routewatch#190 found for the prefix…