Graph: re-entry transition is missing its entry action

[nightroman]

Here is the graph of the official BugTrackerExample

Note that it correctly shows the label Assign / OnAssigned on transitioning from Open to Assigned and from Deferred to Assigned.

But on transitioning from Assigned to Assigned (re-entry) the label is just Assign instead of expected Assign / OnAssigned.
(OnAssigned is called on re-entry to Assigned and this should be reflected).

[mclift]

Thanks for writing this up, @nightroman.

I think that in this case the graph output is correct since OnAssigned is specifically configured for the parameterised Assign trigger, whereas the re-entrant transition uses the Assign trigger directly.

stateless/example/BugTrackerExample/Bug.cs

Lines 42 to 43 in d58e33b

	.OnEntryFrom(_assignTrigger, OnAssigned) // This is where the TriggerWithParameters is used. Note that the TriggerWithParameters object is used, not something from the enum
	.PermitReentry(Trigger.Assign)

Does this explain what you're seeing?

Thanks

[nightroman]

On transitioning from Assigned to Assigned (re-entry) the parameterized trigger Assign is called, i.e. one has to specify the new bug assignee. E.g. when assigning from Harry to Fred here, we provide the parameter "Fred":

stateless/example/BugTrackerExample/Program.cs

Lines 13 to 14 in d58e33b

	bug.Assign("Harry");
	bug.Assign("Fred");

In other words, .PermitReentry(Trigger.Assign) does not mean that the re-entry is not parameterized. The same parameterized .OnEntryFrom(_assignTrigger, OnAssigned) applies and the call to OnAssigned is happening on re-entry (debugging, testing, logging confirm this).

[mclift]

Thanks for the quick reply. I really shouldn't try to debug stuff when I'm tired :)

I'll try to spend some time on this over the next few days, unless you're already working on it.

Thanks

[nightroman]

@mclift Thank you. No, I am not working on this (yet, maybe another day and another issue). Besides, this issue may be related or be in the same area as #587 and you have already started some work #589 (great!). So it might be reasonable either to fix both issues together or one after another.

[nightroman]

As for contributing, in a way... at this point I've been working on PowerShell helpers for Stateless: #592 These tools are useful for state machine interactive testing, low ceremony graph generation/show, etc.