Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Global references are not kept when replacing an event (interrupting to non-interrupting) #2249

Open
nikku opened this issue Oct 11, 2024 · 1 comment
Labels
backlog Queued in backlog bug Something isn't working modeling spring cleaning Could be cleaned up one day

Comments

@nikku
Copy link
Member

nikku commented Oct 11, 2024

Describe the Bug

I replace an event (i.e. a boundary event) from interrupting to non-interrupting. I see that the global message reference gets lost.

We have existing test cases for the desired behavior (reference is kept), but due to nested it(..., function() { it(...); } calls these are not actually executed 🙈.

#2248 uncovered this miss-use, and the bug.

Steps to Reproduce

  1. Uncomment these tests
  2. See that test suite fails

Expected Behavior

When replacing an event (toggling interrupting / non-interrupting) we keep the global element reference.

Environment

  • Browser: Any
  • OS: Any
  • Library version: v17.1.0

Related to #1906

@sombrek
Copy link
Contributor

sombrek commented Oct 14, 2024

Cf. #1906

@nikku nikku added the backlog Queued in backlog label Oct 14, 2024 — with bpmn-io-tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog bug Something isn't working modeling spring cleaning Could be cleaned up one day
Projects
None yet
Development

No branches or pull requests

2 participants