Bug 575122 - [Toolsmiths] Validation builder for Palette Configuration models
Summary: [Toolsmiths] Validation builder for Palette Configuration models
Status: ASSIGNED
Alias: None
Product: Papyrus
Classification: Modeling
Component: Toolsmiths (show other bugs)
Version: 5.2.0   Edit
Hardware: All All
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: Christian Damus CLA
QA Contact: Remi Schnekenburger CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-07-29 11:13 EDT by Christian Damus CLA
Modified: 2021-08-25 15:29 EDT (History)
0 users

See Also:


Attachments
Proposed validation annotation schema (45.77 KB, application/pdf)
2021-07-29 11:13 EDT, Christian Damus CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Damus CLA 2021-07-29 11:13:03 EDT
Created attachment 286855 [details]
Proposed validation annotation schema

Palette model validation is currently covered by the GenericEMFModelBuilder as the constraints defined in the model are all that needs to be validated. The most interesting thing to validate is the element types referenced by tool configurations, but unlike some other tooling models, these are referenced by HREF and not by ID. So, the intrinsic proxy constraint is sufficient here.

The only thing that needs additional validation for palette models will be

- registration of palette configuration on the extension point
- required bundle dependencies not implied by the model in a way that the
  GenericEMFBuilder can detect (which I think currently is only the bundles
  hosting Ecore packages used by the model)

So, I propose to enhance the GenericEMFModelBuilder to handle these additional requirements of the Palette Configuration model validation that can be addressed in a generic way.. Specifically,

- validation of registration on an extension point described by Ecore annotations
- (if not done already) validation of bundle dependencies implied by resources
  cross-referenced by the resource being validated that are hosted in other
  bundles than the resource being validated

See the attachment for details of the proposal.
Comment 1 Eclipse Genie CLA 2021-08-25 15:29:20 EDT
New Gerrit change created: https://git.eclipse.org/r/c/papyrus/org.eclipse.papyrus/+/184422