max / makenotwork

Drop the needless async on the CLI actor test helper

Author: Max Johnson <me@maxj.phd> · 2026-07-31 20:17 UTC

Signed with PGP, not checked

Commit: 8c730bc820cadfd3e027a25dc0a34b2199ce9904

Parent: dccfb15

1 file changed, +5 insertions, -5 deletions

M server/tests/workflows/ssh_management.rs +5 -5

			@@ -269,7 +269,7 @@
269	269		}
270	270
271	271		/// Authenticate the test client as `user_id` for the internal API.
272		-	async fn as_cli_actor(h: &mut TestHarness, user_id: db::UserId) {
	272	+	fn as_cli_actor(h: &mut TestHarness, user_id: db::UserId) {
273	273		h.client.set_bearer_token("test-cli-token");
274	274		let actor = makenotwork::crypto::mint_internal_actor_token(
275	275		user_id,
			@@ -291,7 +291,7 @@
291	291		.await
292	292		.unwrap();
293	293
294		-	as_cli_actor(&mut h, user.id).await;
	294	+	as_cli_actor(&mut h, user.id);
295	295
296	296		let resp = h.client.get("/api/internal/creator/repos").await;
297	297		assert!(resp.status.is_success(), "repo list failed: {}", resp.text);
			@@ -316,7 +316,7 @@
316	316		.await
317	317		.unwrap();
318	318
319		-	as_cli_actor(&mut h, user.id).await;
	319	+	as_cli_actor(&mut h, user.id);
320	320
321	321		// The whole point of the port: publishing, and unpublishing again, without
322	322		// touching the web UI.
			@@ -375,7 +375,7 @@
375	375		// Every verb keys on (actor, name), so another user's repo is simply not
376	376		// found rather than found-and-refused. The delete case is the one that
377	377		// matters: it removes a directory tree.
378		-	as_cli_actor(&mut h, other.id).await;
	378	+	as_cli_actor(&mut h, other.id);
379	379
380	380		let resp = h
381	381		.client
			@@ -419,7 +419,7 @@
419	419		.unwrap()
420	420		.unwrap();
421	421
422		-	as_cli_actor(&mut h, user.id).await;
	422	+	as_cli_actor(&mut h, user.id);
423	423
424	424		// Delete builds a filesystem path from this string, so the validation runs
425	425		// before the lookup rather than after.