StoneCypher / jssm / 7f4349a419caf989e121a1dea34a8d2083811fa1

100%

master: 100%

push

github

fix: correct _has_transition_hooks init and prevent histograph input mutation

_has_transition_hooks and _has_post_transition_hooks were initialized to true
in the Machine constructor despite no hooks being registered, causing an
unnecessary edges_between lookup on every plain transition. Changed both to
false and updated transition_impl to also check _has_post_transition_hooks
when determining trans_type, so posthooks still fire correctly.

histograph() called ar.sort() which mutates the caller's array. Changed to
[...ar].sort() to avoid the side effect.

Fixes stonecypher/fsl#1320, stonecypher/fsl#1321

553 of 554 branches covered (99.82%)

Branch coverage included in aggregate %.

6 of 6 new or added lines in 3 files covered. (100.0%)

1159 of 1159 relevant lines covered (100.0%)

23046.7 hits per line