| Summary: | Saving process in ResourceSaveDiagnose should be atomic | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Modeling] Sirius | Reporter: | Steve Monnier <steve.monnier> | ||||
| Component: | Core | Assignee: | Steve Monnier <steve.monnier> | ||||
| Status: | RESOLVED WONTFIX | QA Contact: | |||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | pierre-charles.david | ||||
| Version: | 5.0.0 | Keywords: | triaged | ||||
| Target Milestone: | 5.0.3 | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| See Also: |
https://git.eclipse.org/r/104982 https://git.eclipse.org/r/104983 https://git.eclipse.org/r/104988 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=8cc9ce0fe1a08659f7313575d73f1ff20c79ba78 https://git.eclipse.org/r/105226 |
||||||
| Whiteboard: | need_test | ||||||
| Attachments: |
|
||||||
|
Description
Steve Monnier
New Gerrit change created: https://git.eclipse.org/r/104982 New Gerrit change created: https://git.eclipse.org/r/104983 New Gerrit change created: https://git.eclipse.org/r/104988 Gerrit change https://git.eclipse.org/r/104988 was merged to [v5.0.x]. Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=8cc9ce0fe1a08659f7313575d73f1ff20c79ba78 Fixed by 8cc9ce0fe1a08659f7313575d73f1ff20c79ba78, but I'm keeping the ticket open to see if we can create an automated non-regression test. Created attachment 270186 [details]
ecore_sample_for_validation
Validation step: - 1/ In org.eclipse.sirius.business.internal.session.danalysis.ResourceSaveDiagnose.hasDifferentSerialization(Map<?, ?>) add a Thread.sleep(20000) after resourcetoSave.eSetDeliver(false); - 2/ Launch a runtime and import the sample in attachments - 3/ Create a new modeling project - 4/ Drag and drop the model from the imported project to the new project - 5/ Activate the Design viewpoint - 6/ create a new representation -> Check that you don't have every diagram element stacked on (0,0) -> Check that you can move diagram elements If you can't do steps 3 to 6 in 20s, you can increase the value in step1. New Gerrit change created: https://git.eclipse.org/r/105226 Closing all clones created to report fixes on a hypothetical version 5.0.3: there is no plan to actually release that version. |