fix(test): update rewind E2E Test 1 assertion after isRealUserTurn fix (#3622)

Test 1 asserted `say exactly GAMMA3` after pressing Up once in the
rewind selector, but that only passed because `/rewind` was incorrectly
counted as a user turn. After `isRealUserTurn()` excluded slash commands,
the turn list is [ALPHA1, BETA2, GAMMA3] and Up from the initial
selection (GAMMA3) lands on BETA2. Update the assertion to match.

Ref: https://github.com/QwenLM/qwen-code/pull/3441#issuecomment-4319798259

Co-authored-by: jinye.djy <jinye.djy@alibaba-inc.com>
Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
This commit is contained in:
jinye 2026-04-26 06:49:42 +08:00 committed by GitHub
parent 83d1e6dcae
commit 72c31d378d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -299,13 +299,12 @@ test_rewind_command() {
send_keys y
wait_for "Conversation rewound" || return 1
# After rewind: the input should be pre-populated with the selected turn's
# text ("say exactly GAMMA3..."). The GAMMA3 *response* turn should be gone
# from the conversation, but the text appears in the input bar — which is
# the correct pre-population behavior.
# Verify pre-population: the input bar should contain GAMMA3 text
assert_screen "say exactly GAMMA3" || return 1
# Verify the earlier turns (ALPHA1, BETA2) are still in conversation
# After rewind: pressing Up once from the initial selection (GAMMA3, the last
# real user turn) lands on BETA2. Rewind targets BETA2, so its text gets
# pre-populated into the input bar. Slash commands like /rewind are excluded
# from the turn list by isRealUserTurn().
assert_screen "say exactly BETA2" || return 1
# Verify the earlier turn (ALPHA1) is still in conversation
assert_scrollback "ALPHA1" || return 1
}