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

Added state not reached warning during the validation + Test state machine + assertion #25

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

LeanderVonSeelstrang
Copy link
Contributor

I don't know how to run the tests. Do I run them with maven. I am getting a Aborted (core dumped) error before the tests ran.

I did only two things for testing:

  1. Added the state machine missingIncomingTransition.xml
  2. In StatemachineTest added testMissingIncomingTransition()

@LeroyR
Copy link
Member

LeroyR commented Jan 17, 2024

Hope to be able to rewiew this weekend.

We may want to add a flag and make this/all checks opt-in or opt out but we should do that separately and rebase this ontop.

@LeroyR LeroyR force-pushed the master branch 2 times, most recently from 50ff965 to 60cbd73 Compare April 6, 2024 20:56
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.

2 participants