mirror of
https://github.com/jason5ng32/MyIP.git
synced 2026-11-04 08:55:47 +00:00
Claude
96525d2fc7
A pass over every file the branch touched, after the iOS animation hunt left tries and reverts behind it. Removed what nothing reaches: the view-transition rule for a transition no code starts, an unreachable guard in the transition hook, a no-op KeepAlive key branch, a sign-in guard the menu already enforces, usage-dialog wrappers that only forwarded to the store, a duplicated modifier-click check and card filter (isPlainClick / listedTools), a test that hand-rolled listedTools, a drawer-era fixture parameter. auth-flow.js folded into auth-redirect.js. Comments cut to what the code cannot say itself — file headers of a few lines, the measured iOS reasons, non-obvious orderings — and stale ones fixed (Nav under KeepAlive, Home.vue reusing DEFAULT_META, "from Nav", enabledCards). AGENTS.md sections tightened to the code as it is now. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013GbLTGMZ45MV78qd2tHoco
78 lines
3.2 KiB
JavaScript
78 lines
3.2 KiB
JavaScript
// Tests for frontend/utils/auth-redirect.js: the popup → redirect fallback
|
|
// codes, and the pending-redirect marker driven through a fake sessionStorage.
|
|
|
|
import assert from 'node:assert/strict';
|
|
import { describe, it, beforeEach } from 'node:test';
|
|
import {
|
|
shouldFallBackToRedirect,
|
|
markPendingRedirectSignIn,
|
|
hasPendingRedirectSignIn,
|
|
takePendingRedirectSignIn,
|
|
} from '../frontend/utils/auth-redirect.js';
|
|
|
|
const createStorage = () => ({
|
|
data: {},
|
|
broken: false,
|
|
getItem(k) { if (this.broken) throw new Error('storage disabled'); return this.data[k] ?? null; },
|
|
setItem(k, v) { if (this.broken) throw new Error('storage disabled'); this.data[k] = String(v); },
|
|
removeItem(k) { if (this.broken) throw new Error('storage disabled'); delete this.data[k]; },
|
|
});
|
|
|
|
let storage;
|
|
beforeEach(() => {
|
|
storage = createStorage();
|
|
});
|
|
|
|
describe('shouldFallBackToRedirect', () => {
|
|
it('falls back when the popup cannot open', () => {
|
|
assert.equal(shouldFallBackToRedirect('auth/popup-blocked'), true);
|
|
assert.equal(shouldFallBackToRedirect('auth/operation-not-supported-in-this-environment'), true);
|
|
});
|
|
|
|
it('does not fall back when the visitor closed the popup, or on other codes', () => {
|
|
assert.equal(shouldFallBackToRedirect('auth/popup-closed-by-user'), false);
|
|
assert.equal(shouldFallBackToRedirect('auth/cancelled-popup-request'), false);
|
|
assert.equal(shouldFallBackToRedirect('auth/network-request-failed'), false);
|
|
assert.equal(shouldFallBackToRedirect(undefined), false);
|
|
});
|
|
});
|
|
|
|
describe('auth-redirect marker', () => {
|
|
it('reports nothing pending on a fresh storage', () => {
|
|
assert.equal(hasPendingRedirectSignIn(storage), false);
|
|
assert.equal(takePendingRedirectSignIn(storage), null);
|
|
});
|
|
|
|
it('has() reads without consuming', () => {
|
|
markPendingRedirectSignIn('google', storage);
|
|
hasPendingRedirectSignIn(storage);
|
|
assert.equal(hasPendingRedirectSignIn(storage), true);
|
|
});
|
|
|
|
it('take() returns the provider once and clears the marker', () => {
|
|
markPendingRedirectSignIn('github', storage);
|
|
assert.deepEqual(takePendingRedirectSignIn(storage), { providerKey: 'github' });
|
|
assert.equal(hasPendingRedirectSignIn(storage), false);
|
|
assert.equal(takePendingRedirectSignIn(storage), null);
|
|
});
|
|
|
|
it('a later mark replaces the earlier provider', () => {
|
|
markPendingRedirectSignIn('google', storage);
|
|
markPendingRedirectSignIn('github', storage);
|
|
assert.deepEqual(takePendingRedirectSignIn(storage), { providerKey: 'github' });
|
|
});
|
|
|
|
it('never throws when storage is disabled', () => {
|
|
storage.broken = true;
|
|
assert.doesNotThrow(() => markPendingRedirectSignIn('google', storage));
|
|
assert.equal(hasPendingRedirectSignIn(storage), false);
|
|
assert.equal(takePendingRedirectSignIn(storage), null);
|
|
});
|
|
|
|
it('never throws when no storage exists at all', () => {
|
|
assert.doesNotThrow(() => markPendingRedirectSignIn('google', null));
|
|
assert.equal(hasPendingRedirectSignIn(null), false);
|
|
assert.equal(takePendingRedirectSignIn(null), null);
|
|
});
|
|
});
|