Skip to content

fix: Apply end_context when end/exit is a sequence step#137

Merged
lykinsbd merged 1 commit intomasterfrom
fix/scenario-end-context
Mar 2, 2026
Merged

fix: Apply end_context when end/exit is a sequence step#137
lykinsbd merged 1 commit intomasterfrom
fix/scenario-end-context

Conversation

@lykinsbd
Copy link
Collaborator

@lykinsbd lykinsbd commented Mar 2, 2026

In scenario mode, end/exit were blocked from applying their context effect. Now explicitly apply handleExitEnd when the matched sequence step was end or exit.

@lykinsbd lykinsbd force-pushed the fix/scenario-end-context branch from 4319ce2 to 89670a5 Compare March 2, 2026 19:58
In scenario mode, end and exit were blocked from applying their context
effect because inScenario was still true after the sequence step advanced.
Now explicitly apply handleExitEnd when the sequence step was end or exit.
@lykinsbd lykinsbd force-pushed the fix/scenario-end-context branch from 89670a5 to 5c41195 Compare March 2, 2026 20:08
@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2026

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ 🤖
github.com/tbotnz/cisshgo/ssh_server/handlers 97.74% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/tbotnz/cisshgo/ssh_server/handlers/ciscohandlers.go 97.74% (ø) 133 130 3

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

@lykinsbd lykinsbd merged commit f32b3b3 into master Mar 2, 2026
4 checks passed
@lykinsbd lykinsbd deleted the fix/scenario-end-context branch March 2, 2026 20:18
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