Skip to content

Commit 7ea8ac2

Browse files
committed
test(hono): remove skipped FAPI proxy e2e test
The test was written for this PR but immediately skipped because clerkFrontendApiProxy in proxy.ts uses the raw request URL for the Clerk-Proxy-Url header, which is a pre-existing issue outside this PR's scope. Remove rather than ship dead test code.
1 parent 291c009 commit 7ea8ac2

1 file changed

Lines changed: 0 additions & 13 deletions

File tree

integration/tests/hono/proxy.test.ts

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -22,19 +22,6 @@ testAgainstRunningApps({ withEnv: [appConfigs.envs.withEmailCodesProxy] })(
2222
await app.teardown();
2323
});
2424

25-
test.skip('proxies Frontend API requests to /__clerk path', async ({ page, context }) => {
26-
// TODO: clerkFrontendApiProxy sends Clerk-Proxy-Url with localhost origin
27-
// because it uses requestUrl directly. FAPI rejects this with 400.
28-
// Needs the same forwarded-header awareness in proxy.ts.
29-
const u = createTestUtils({ app, page, context });
30-
const url = new URL('/api/__clerk/v1/client?_is_native=false', app.serverUrl);
31-
const res = await u.page.request.get(url.toString());
32-
33-
expect(res.status()).toBe(200);
34-
const body = await res.json();
35-
expect(body).toHaveProperty('response');
36-
});
37-
3825
test('protected routes still require auth when proxy is enabled', async ({ page, context }) => {
3926
const u = createTestUtils({ app, page, context });
4027
await u.page.goToRelative('/');

0 commit comments

Comments
 (0)