test(reborn): post the renamed 'slack' extension id in personal OAuth start tests

The model-B remodel renamed the installable Slack tools extension
slack_user -> slack (and the bot channel slack -> slack_bot), but
slack_personal_oauth_start_serves_through_composed_router /
_fails_closed_without_slot still posted "slack_bot" — the hidden bot
channel, which is not OAuth-installable — so the handler correctly
rejected it (400) and the composed-router test asserted 200 and failed.
Post "slack" so the tests exercise the real installable extension.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
BenKurrek
2026-07-06 15:19:40 -04:00
parent 0fb2887ae5
commit fee5549bc8

View File

@@ -2127,7 +2127,7 @@ mod slack_personal_oauth_serve {
let app = build_app_with_slack_personal_oauth().await;
let response =
post_extension_oauth_start(&app, "slack_bot", slack_oauth_start_body()).await;
post_extension_oauth_start(&app, "slack", slack_oauth_start_body()).await;
assert_eq!(response.status(), StatusCode::OK);
let body = read_body_string(response).await;
@@ -2155,7 +2155,7 @@ mod slack_personal_oauth_serve {
let (app, _) = build_app_with_product_auth();
let response =
post_extension_oauth_start(&app, "slack_bot", slack_oauth_start_body()).await;
post_extension_oauth_start(&app, "slack", slack_oauth_start_body()).await;
assert_eq!(response.status(), StatusCode::SERVICE_UNAVAILABLE);
let body = read_body_string(response).await;