Skip to content

Assert reopened session ID in DDI integration test#377

Draft
mhatrevi with Copilot wants to merge 2 commits into
mainfrom
copilot/add-test-case-different-session-id
Draft

Assert reopened session ID in DDI integration test#377
mhatrevi with Copilot wants to merge 2 commits into
mainfrom
copilot/add-test-case-different-session-id

Conversation

Copilot AI commented May 8, 2026

Copy link
Copy Markdown
Contributor

The reopen-session path should preserve the original session ID, including the ID returned in the DDI response body. The issue requested coverage without expanding emulator behavior.

  • Reopen session coverage
    • Added an assertion to test_reopen_session verifying resp.data.sess_id matches the original setup_res.session_id.
    • Keeps the existing emulator behavior unchanged.
assert_eq!(resp.hdr.sess_id, Some(setup_res.session_id));
assert_eq!(resp.data.sess_id, setup_res.session_id);

Copilot AI requested review from Copilot and removed request for Copilot May 8, 2026 22:46
Copilot AI review requested due to automatic review settings May 8, 2026 22:50
Copilot AI changed the title [WIP] Add test case for different session id on reopen Assert reopened session ID in DDI integration test May 8, 2026
Copilot AI removed the request for review from Copilot May 8, 2026 22:50
Copilot AI requested a review from mhatrevi May 8, 2026 22:50
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.

Add a test case where a different session id is returned on reopen session.

2 participants