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

Add model validation #2487

Open
jecisc opened this issue Apr 6, 2023 · 0 comments
Open

Add model validation #2487

jecisc opened this issue Apr 6, 2023 · 0 comments

Comments

@jecisc
Copy link
Member

jecisc commented Apr 6, 2023

We often find some incoherences in models depending on the parser used or the version of the parser.
It might be nice to add a validation system to the model doing some checks.

For example:

  • Can we have stub entities in non stub entities? I think no but Anne has that in a model. We could add a validation about that
  • We discussed once with Anne and Nicolas that stub classes with the same name should share the same instance. Maybe we could validate that? stubClassNames := model allClasses select: #isStub thenCollect: #mooseName. self assert: stubClassNames size = stubClassNames asSet size
  • We could check that the number of non stub entities equals the number of all children of root entities + the number of root entities
  • ...

We don't have to enable this check at the import, but it might still be nice to have the possibility and to use that in the tests of our importers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant