-
Notifications
You must be signed in to change notification settings - Fork 50
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
[1068] Add support for displaying details on arbitrary element kinds #1076
Conversation
sbegaudeau
commented
Feb 17, 2022
- [1068] Add support for displaying details on arbitrary element kinds
- [test] Add additional tests to the collaborative form project
81edf7a
to
3ce9663
Compare
...c/main/java/org/eclipse/sirius/components/collaborative/forms/FormDescriptionAggregator.java
Outdated
Show resolved
Hide resolved
...eclipse/sirius/components/collaborative/forms/handlers/EditMultiSelectEventHandlerTests.java
Outdated
Show resolved
Hide resolved
...eclipse/sirius/components/collaborative/forms/handlers/EditMultiSelectEventHandlerTests.java
Outdated
Show resolved
Hide resolved
.../org/eclipse/sirius/components/collaborative/forms/handlers/EditSelectEventHandlerTests.java
Outdated
Show resolved
Hide resolved
formDescriptions.add(formDescription2); | ||
|
||
FormDescriptionAggregator aggregator = new FormDescriptionAggregator(); | ||
Optional<FormDescription> optional = aggregator.aggregate(formDescriptions, List.of(object), new IObjectService.NoOp()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it relevant to test the behavior of the FormDescriptionAggregator when there are many objects?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We would only test that we iterate properly on a for loop somewhere. I wanted to have at least one test here for now.
...onents-forms/src/main/java/org/eclipse/sirius/components/forms/components/FormComponent.java
Outdated
Show resolved
Hide resolved
Bug: #1068 Signed-off-by: Stéphane Bégaudeau <stephane.begaudeau@obeo.fr>
Signed-off-by: Stéphane Bégaudeau <stephane.begaudeau@obeo.fr>
3ce9663
to
3830673
Compare
Signed-off-by: Guillaume Coutable <guillaume.coutable@obeo.fr>