max / makenotwork

Carry the notify test's doc comment across with the test The split left the comment in pages.rs and took the test to signup_notify.rs, which orphaned it above an unrelated function and tripped clippy's empty_line_after_outer_attr.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: Max Johnson <me@maxj.phd> · 2026-08-24 21:28 UTC

Signed with PGP, not checked

Commit: 26ad9bfbaea8be07f29df5d6fcd38bc98504f274

Parent: 933cd28

2 files changed, +5 insertions, -8 deletions

M server/tests/workflows/pages.rs -8

			@@ -483,14 +483,6 @@
483	483		}
484	484		}
485	485
486		-	// ── Landing notify-me capture ──
487		-
488		-	/// The form works with JS off.
489		-	///
490		-	/// It had no action and no method, so a browser without JS submitted a GET to
491		-	/// `/` and the address was dropped silently. This is the only email capture on
492		-	/// the landing page, so a silent loss is a lost signup nobody can count.
493		-
494	486		/// A method mismatch on a path that exists renders the branded error page.
495	487		///
496	488		/// GET /logout is the case that showed up in the wild: axum answered with a

M server/tests/workflows/signup_notify.rs +5

			@@ -7,6 +7,11 @@
7	7
8	8		use crate::harness::TestHarness;
9	9
	10	+	/// The form works with JS off.
	11	+	///
	12	+	/// It had no action and no method, so a browser without JS submitted a GET to
	13	+	/// `/` and the address was dropped silently. This is the only email capture on
	14	+	/// the landing page, so a silent loss is a lost signup nobody can count.
10	15		#[tokio::test]
11	16		async fn notify_form_without_js_stores_the_address() {
12	17		let mut h = TestHarness::new().await;