feat: improve the robustness of `PathResolver`

tbouffard commented 1 year ago

Add several unit tests and simplify the implementation.

closes #18

### Tasks
- [x] Check if we need the code that currently check that edges really exist in the model
- [x] Refactor the implementation if possible
- [x] Add link to the bpmn-visualization issue in tests to explain why extra erroneous edge ids are returned (the test will break once `bv-experimental-add-ons` uses the new release)

Notes

Current limitation

The current implementation doesn't filter out erroneous edge identifiers that are resolved from incorrectly defined incoming and outgoing identifiers. bpmn-visualization doesn't filter them out itself when it should, so I decided not to do it here but to create an issue in the bpmn-visualization repository instead: https://github.com/process-analytics/bpmn-visualization-js/issues/2852

Note that the old implementation didn't filter out these erroneous edge identifiers either, so there's no reason to keep it instead of switching to the new implementation.

All code is now fully covered by tests 🎉

=============================== Coverage summary ===============================
Statements   : 100% ( 63/63 )
Branches     : 100% ( 26/26 )
Functions    : 100% ( 19/19 )
Lines        : 100% ( 62/62 )
================================================================================