fix(frontend): update AppRoot test to match actual login screen text - #327
Merged
Merged
Conversation
The login screen displays '登录' as the title, not '登录或注册'. Updated the test expectation to match the actual UI text. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
![fennoai[bot]](https://avatars.githubusercontent.com/in/1924301?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
The updated AppRoot sign-out assertion now matches the LoginScreen title rendered by the current frontend (登录). The complete one-file diff is consistent with the existing login-screen tests and introduces no correctness issues.
Verification: reviewed the fixed SHA diff and corresponding head-source implementation. The focused Jest test was not run because frontend/node_modules is not present in the workspace.
znnnnnnn-wil
approved these changes
Aug 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
问题描述
CI 测试失败，因为 中的测试期望找到文本 "登录或注册"，但实际的登录界面只显示 "登录"。
修复内容
测试验证
✅ 本地运行 所有 15 个测试全部通过
相关 CI 运行