Skip to content

test(oauth): 授权测试夹具改用公共的 previousStepCode() - #19

Merged
Conner512 merged 1 commit into
mainfrom
chore/totp-helper-reuse
Oct 15, 2026
Merged

Conner512 merged 1 commit into
mainfrom
chore/totp-helper-reuse

Conversation

@Conner512

Copy link
Copy Markdown
Owner

PR #17 在 services/oauth/test/fixture.mjs 里内联了「避开 30 秒时间窗边界」的等待逻辑,PR #18 又新增了公共函数 previousStepCode()。两个 PR 都已合并,这里让 fixture.mjs 也改用公共函数,去掉重复代码。行为不变。

验证(本地,Node 26.11):npm run test:oauth 208/210(失败的 CFG-01..06、SHARED-01 是本机 Node 26 非 LTS 触发的已知环境问题),npm run test:publication 20/20。

🤖 Generated with Claude Code

PR #17 inlined the step-boundary wait in fixture.mjs while PR #18 added the
shared previousStepCode() helper; use the helper in both places.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Conner512
Conner512 merged commit bf76c15 into main Oct 15, 2026
27 checks passed
@Conner512
Conner512 deleted the chore/totp-helper-reuse branch October 15, 2026 05:11
Conner512 added a commit that referenced this pull request Oct 15, 2026
Brings PR #14–#19 (hybrid search, connection fixes, test fixture
hardening) into the console redesign.

Conflicts:
- services/oauth/test/fixture.mjs, helpers/identity-fixture.mjs,
  helpers/totp.mjs: both sides fixed the same port-reuse and TOTP
  step-boundary flakes with the same previousStepCode() helper; keep main's
  version and drop the duplicate imports the auto-merge left in
  identity-boundaries and registration-console tests
- web/console/app.mjs: keep the redesigned console and apply main's hybrid
  search default there and in visuals.mjs libraryView()

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant