Skip to content

Commit

Permalink
[518] Add dynamic margin property check on interaction container tests
Browse files Browse the repository at this point in the history 
A system property has been introduced to choose if the left side of the
'interaction container' show dynamically follow the first (most-left)
element or be fixed. The Interaction container swtbot test now check
both modes.

Bug: #518
Signed-off-by: Steve Monnier <[email protected]>
  • Loading branch information
@SteveMonnier
SteveMonnier committed Dec 19, 2024
1 parent d6ffec1 commit 7a6c520
Show file tree
Hide file tree
Showing 2 changed files with 138 additions and 16 deletions.
77 changes: 72 additions & 5 deletions ...rc/org/eclipse/sirius/tests/swtbot/sequence/InteractionContainerAdditionalLayerTests.java
View file
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*******************************************************************************
* Copyright (c) CEA.
* Copyright 2024 (c) CEA.
* This program and the accompanying materials
* are made available under the terms of the Eclipse Public License 2.0
* which accompanies this distribution, and is available at
Expand All @@ -19,14 +19,18 @@
import org.eclipse.draw2d.geometry.Rectangle;
import org.eclipse.gef.EditPart;
import org.eclipse.gef.editparts.AbstractGraphicalEditPart;
import org.eclipse.sirius.diagram.DDiagram;
import org.eclipse.sirius.diagram.sequence.business.internal.elements.InteractionContainer;
import org.eclipse.sirius.diagram.sequence.business.internal.layout.LayoutConstants;
import org.eclipse.sirius.diagram.sequence.ui.tool.internal.edit.part.EndOfLifeEditPart;
import org.eclipse.sirius.diagram.sequence.ui.tool.internal.edit.part.InteractionContainerEditPart;
import org.eclipse.sirius.diagram.sequence.ui.tool.internal.edit.part.LifelineEditPart;
import org.eclipse.sirius.diagram.tools.api.preferences.SiriusDiagramPreferencesKeys;
import org.eclipse.sirius.diagram.ui.tools.api.preferences.SiriusDiagramUiPreferencesKeys;
import org.eclipse.sirius.ext.base.Option;
import org.eclipse.sirius.ext.base.Options;
import org.eclipse.sirius.tests.swtbot.support.api.business.UIDiagramRepresentation;
import org.eclipse.sirius.tests.swtbot.support.api.business.UIResource;
import org.eclipse.sirius.tests.swtbot.support.api.condition.OperationDoneCondition;
import org.eclipse.sirius.tests.swtbot.support.api.editor.SWTBotSiriusDiagramEditor;
import org.eclipse.sirius.tests.unit.diagram.sequence.InteractionsConstants;
Expand Down Expand Up @@ -67,13 +71,29 @@ public class InteractionContainerAdditionalLayerTests extends AbstractSequenceDi

private UIDiagramRepresentation diagram;


/**
* {@inheritDoc}
*/
@Override
protected void onSetUpAfterOpeningDesignerPerspective() throws Exception {
super.onSetUpAfterOpeningDesignerPerspective();
changeDiagramUIPreference(SiriusDiagramUiPreferencesKeys.PREF_OLD_UI.name(), true);
changeDiagramPreference(SiriusDiagramPreferencesKeys.PREF_DISPLAY_HEADER_SECTION.name(), false);
}

public void initDiagram() {
if (getSessionModel() == null) {

} else {
sessionAirdResource = new UIResource(designerProject, FILE_DIR, getSessionModel());
localSession = designerPerspective.openSessionFromFile(sessionAirdResource, true);
Option<String> dRepresentationName = getDRepresentationName();
if (dRepresentationName.some()) {
editor = (SWTBotSiriusDiagramEditor) openRepresentation(localSession.getOpenedSession(), getRepresentationId(), dRepresentationName.get(), DDiagram.class, true, true);
}
}

initEditor();
diagram = localSession.getLocalSessionBrowser().perCategory().selectViewpoint(VIEWPOINT_NAME).selectRepresentation(getRepresentationId())
.selectRepresentationInstance(getDRepresentationName().get(), UIDiagramRepresentation.class);

Expand All @@ -87,9 +107,54 @@ protected void onSetUpAfterOpeningDesignerPerspective() throws Exception {
}

/**
* Check that on lifeline move and change position, the interaction container bounds also move accordingly.
* Check that on lifeline move and change position, the interaction container bounds also move accordingly.In this
* version of the test, the property to have a left margin dynamic is true. This test needs to be run on its own because
* the property is initialized for the whole VM and can't be changed.
*/
public void _testInteractionResizeOnInstanceRolePositionChangeWithLeftMargin() {
String oldPreferenceValue = System.getProperty("org.eclipse.sirius.diagram.sequence.layout.interaction.container.dynamic.left");
try {
System.setProperty("org.eclipse.sirius.diagram.sequence.layout.interaction.container.dynamic.left", String.valueOf(true));
initDiagram();
// Activate the extension layer.
ICondition done = new OperationDoneCondition();
diagram.changeLayerActivation(INTERACTION_CONTAINER_LAYER);
bot.waitUntil(done);

// Check that the Interaction Container east bound correspond to the Instance Role C east bound + margin
SWTBotGefEditPart interactionContainereditPart = editor.getEditPart("Lifelines", InteractionContainerEditPart.class);
Rectangle interactionContainerBounds = editor.getBounds(interactionContainereditPart);
assertEquals("Interaction Container east bound is not where expected", instanceRoleEditPartCBounds.getRight().x + InteractionContainer.MARGIN, interactionContainerBounds.getRight().x, 1);

/*
* Move lifelines to graphically change their order.
*/
// drag LIFELINE_C to (250,0) delta
editor.drag(instanceRoleEditPartCBot, instanceRoleEditPartCBounds.x + 250, origin.y);
// Drag LIFELINE_A to (300,0) delta, Lifeline A forth between Lifeline B
// and Lifeline C
editor.drag(instanceRoleEditPartABot, instanceRoleEditPartABounds.x + 300, origin.y);

// Check that the Interaction Container east bound still correspond to the Instance Role C east bound + margin
instanceRoleEditPartCBot = editor.getEditPart(LIFELINE_C);
instanceRoleEditPartCBounds = editor.getBounds(instanceRoleEditPartCBot);
instanceRoleEditPartBBot = editor.getEditPart(LIFELINE_B);
instanceRoleEditPartBBounds = editor.getBounds(instanceRoleEditPartBBot);
interactionContainereditPart = editor.getEditPart("Lifelines", InteractionContainerEditPart.class);
interactionContainerBounds = editor.getBounds(interactionContainereditPart);
assertEquals("Interaction Container east bound is not where expected", instanceRoleEditPartCBounds.getRight().x + InteractionContainer.MARGIN, interactionContainerBounds.getRight().x, 1);
assertEquals("Interaction Container west bound is not where expected", instanceRoleEditPartBBounds.getLeft().x - InteractionContainer.MARGIN, interactionContainerBounds.getLeft().x, 1);
} finally {
System.setProperty("org.eclipse.sirius.diagram.sequence.layout.interaction.container.dynamic.left", oldPreferenceValue != null ? oldPreferenceValue : String.valueOf(false));
}
}

/**
* Check that on lifeline move and change position, the interaction container bounds also move accordingly.In this
* version of the test, the property to have a left margin dynamic is true.
*/
public void testInteractionResizeOnInstanceRolePositionChange() {
public void testInteractionResizeOnInstanceRolePositionChangeWithoutLeftMargin() {
initDiagram();
// Activate the extension layer.
ICondition done = new OperationDoneCondition();
diagram.changeLayerActivation(INTERACTION_CONTAINER_LAYER);
Expand Down Expand Up @@ -117,13 +182,14 @@ public void testInteractionResizeOnInstanceRolePositionChange() {
interactionContainereditPart = editor.getEditPart("Lifelines", InteractionContainerEditPart.class);
interactionContainerBounds = editor.getBounds(interactionContainereditPart);
assertEquals("Interaction Container east bound is not where expected", instanceRoleEditPartCBounds.getRight().x + InteractionContainer.MARGIN, interactionContainerBounds.getRight().x, 1);
assertEquals("Interaction Container west bound is not where expected", instanceRoleEditPartBBounds.getLeft().x - InteractionContainer.MARGIN, interactionContainerBounds.getLeft().x, 1);
assertEquals("Interaction Container west bound is not where expected", -1, interactionContainerBounds.getLeft().x, 1);
}

/**
* Check that when the last lifeline move back and forth, the interaction container bounds also move accordingly.
*/
public void testInteractionResizeOnInstanceRoleMoveBackAndForth() {
initDiagram();
// Activate the extension layer.
ICondition done = new OperationDoneCondition();
diagram.changeLayerActivation(INTERACTION_CONTAINER_LAYER);
Expand Down Expand Up @@ -164,6 +230,7 @@ public void testInteractionResizeOnInstanceRoleMoveBackAndForth() {
* accordingly.
*/
public void testInteractionResizeOnExecutionMove() {
initDiagram();
// Activate the extension layer.
ICondition done = new OperationDoneCondition();
diagram.changeLayerActivation(INTERACTION_CONTAINER_LAYER);
Expand Down
77 changes: 66 additions & 11 deletions ...t/src/org/eclipse/sirius/tests/swtbot/sequence/InteractionContainerDefaultLayerTests.java
View file
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*******************************************************************************
* Copyright (c) CEA.
* Copyright (c) 2024 CEA.
* This program and the accompanying materials
* are made available under the terms of the Eclipse Public License 2.0
* which accompanies this distribution, and is available at
Expand All @@ -13,14 +13,16 @@
package org.eclipse.sirius.tests.swtbot.sequence;

import org.eclipse.draw2d.geometry.Rectangle;
import org.eclipse.gef.EditPart;
import org.eclipse.gef.editparts.AbstractGraphicalEditPart;
import org.eclipse.sirius.diagram.DDiagram;
import org.eclipse.sirius.diagram.sequence.business.internal.elements.InteractionContainer;
import org.eclipse.sirius.diagram.sequence.ui.tool.internal.edit.part.InteractionContainerEditPart;
import org.eclipse.sirius.diagram.tools.api.preferences.SiriusDiagramPreferencesKeys;
import org.eclipse.sirius.diagram.ui.tools.api.preferences.SiriusDiagramUiPreferencesKeys;
import org.eclipse.sirius.ext.base.Option;
import org.eclipse.sirius.ext.base.Options;
import org.eclipse.sirius.tests.swtbot.Activator;
import org.eclipse.sirius.tests.swtbot.support.api.business.UIDiagramRepresentation;
import org.eclipse.sirius.tests.swtbot.support.api.business.UIResource;
import org.eclipse.sirius.tests.swtbot.support.api.editor.SWTBotSiriusDiagramEditor;
import org.eclipse.swtbot.eclipse.gef.finder.widgets.SWTBotGefEditPart;

Expand Down Expand Up @@ -62,13 +64,29 @@ public class InteractionContainerDefaultLayerTests extends AbstractSequenceDiagr

private UIDiagramRepresentation diagram;


/**
* {@inheritDoc}
*/
@Override
protected void onSetUpAfterOpeningDesignerPerspective() throws Exception {
super.onSetUpAfterOpeningDesignerPerspective();
changeDiagramUIPreference(SiriusDiagramUiPreferencesKeys.PREF_OLD_UI.name(), true);
changeDiagramPreference(SiriusDiagramPreferencesKeys.PREF_DISPLAY_HEADER_SECTION.name(), false);
}

public void initDiagram() {
if (getSessionModel() == null) {

} else {
sessionAirdResource = new UIResource(designerProject, FILE_DIR, getSessionModel());
localSession = designerPerspective.openSessionFromFile(sessionAirdResource, true);
Option<String> dRepresentationName = getDRepresentationName();
if (dRepresentationName.some()) {
editor = (SWTBotSiriusDiagramEditor) openRepresentation(localSession.getOpenedSession(), getRepresentationId(), dRepresentationName.get(), DDiagram.class, true, true);
}
}

initEditor();
diagram = localSession.getLocalSessionBrowser().perCategory().selectViewpoint(VIEWPOINT_NAME).selectRepresentation(getRepresentationId())
.selectRepresentationInstance(getDRepresentationName().get(), UIDiagramRepresentation.class);

Expand All @@ -90,9 +108,49 @@ protected void onSetUpBeforeClosingWelcomePage() throws Exception {
}

/**
* Check that on lifeline move and change position, the interaction container bounds also move accordingly.
* Check that on lifeline move and change position, the interaction container bounds also move accordingly. In this
* version of the test, the property to have a left margin dynamic is true. This test needs to be run on its own because
* the property is initialized for the whole VM and can't be changed.
*/
public void _testInteractionResizeOnInstanceRolePositionChangeWithLeftMargin() {
String oldPreferenceValue = System.getProperty("org.eclipse.sirius.diagram.sequence.layout.interaction.container.dynamic.left");
try {
System.setProperty("org.eclipse.sirius.diagram.sequence.layout.interaction.container.dynamic.left", String.valueOf(true));
initDiagram();

// Check that the Interaction Container east bound correspond to the Instance Role C east bound + margin
SWTBotGefEditPart interactionContainereditPart = editor.getEditPart("Lifelines", InteractionContainerEditPart.class);
Rectangle interactionContainerBounds = editor.getBounds(interactionContainereditPart);
assertEquals("Interaction Container east bound is not where expected", instanceRoleEditPartCBounds.getRight().x + InteractionContainer.MARGIN, interactionContainerBounds.getRight().x, 1);

/*
* Move lifelines to graphically change their order.
*/
// drag LIFELINE_C to (250,0) delta
editor.drag(instanceRoleEditPartCBot, instanceRoleEditPartCBounds.x + 250, origin.y);
// Drag LIFELINE_A to (300,0) delta, Lifeline A forth between Lifeline B
// and Lifeline C
editor.drag(instanceRoleEditPartABot, instanceRoleEditPartABounds.x + 300, origin.y);

// Check that the Interaction Container east bound still correspond to the Instance Role C east bound + margin
instanceRoleEditPartCBot = editor.getEditPart(LIFELINE_C);
instanceRoleEditPartCBounds = editor.getBounds(instanceRoleEditPartCBot);
instanceRoleEditPartBBot = editor.getEditPart(LIFELINE_B);
instanceRoleEditPartBBounds = editor.getBounds(instanceRoleEditPartBBot);
interactionContainereditPart = editor.getEditPart("Lifelines", InteractionContainerEditPart.class);
interactionContainerBounds = editor.getBounds(interactionContainereditPart);
assertEquals("Interaction Container east bound is not where expected", instanceRoleEditPartCBounds.getRight().x + InteractionContainer.MARGIN, interactionContainerBounds.getRight().x, 1);
assertEquals("Interaction Container west bound is not where expected", instanceRoleEditPartBBounds.getLeft().x - InteractionContainer.MARGIN, interactionContainerBounds.getLeft().x, 1);
} finally {
System.setProperty("org.eclipse.sirius.diagram.sequence.layout.interaction.container.dynamic.left", oldPreferenceValue != null ? oldPreferenceValue : String.valueOf(false));
}
}

/**
* Check that on lifeline move and change position, the interaction container bounds also move accordingly. In this
* version of the test, the property to have a left margin dynamic is false.
*/
public void testInteractionResizeOnInstanceRolePositionChange() {
public void testInteractionResizeOnInstanceRolePositionChangeWithoutLeftMargin() {
// Check that the Interaction Container east bound correspond to the Instance Role C east bound + margin
SWTBotGefEditPart interactionContainereditPart = editor.getEditPart("Lifelines", InteractionContainerEditPart.class);
Rectangle interactionContainerBounds = editor.getBounds(interactionContainereditPart);
Expand All @@ -115,13 +173,14 @@ public void testInteractionResizeOnInstanceRolePositionChange() {
interactionContainereditPart = editor.getEditPart("Lifelines", InteractionContainerEditPart.class);
interactionContainerBounds = editor.getBounds(interactionContainereditPart);
assertEquals("Interaction Container east bound is not where expected", instanceRoleEditPartCBounds.getRight().x + InteractionContainer.MARGIN, interactionContainerBounds.getRight().x, 1);
assertEquals("Interaction Container west bound is not where expected", instanceRoleEditPartBBounds.getLeft().x - InteractionContainer.MARGIN, interactionContainerBounds.getLeft().x, 1);
assertEquals("Interaction Container west bound is not where expected", -1, interactionContainerBounds.getLeft().x, 1);
}

/**
* Check that when the last lifeline move back and forth, the interaction container bounds also move accordingly.
*/
public void testInteractionResizeOnInstanceRoleMoveBackAndForth() {
initDiagram();
// Check that the Interaction Container east bound correspond to the Instance Role C east bound + margin
SWTBotGefEditPart interactionContainereditPart = editor.getEditPart("Lifelines", InteractionContainerEditPart.class);
Rectangle interactionContainerBounds = editor.getBounds(interactionContainereditPart);
Expand Down Expand Up @@ -152,10 +211,6 @@ public void testInteractionResizeOnInstanceRoleMoveBackAndForth() {
assertEquals("Interaction Container east bound is not where expected", instanceRoleEditPartCBounds.getRight().x + InteractionContainer.MARGIN, interactionContainerBounds.getRight().x, 1);
}

private SWTBotGefEditPart getBotEditPart(AbstractGraphicalEditPart parentExec, final Class<? extends EditPart> expectedEditPartType) {
return editor.getEditPart(parentExec.getFigure().getBounds().getCopy().getCenter(), expectedEditPartType);
}

private void initBotsAndBounds(SWTBotSiriusDiagramEditor swtBotEditor) {
instanceRoleEditPartABot = swtBotEditor.getEditPart(LIFELINE_A);
instanceRoleEditPartBBot = swtBotEditor.getEditPart(LIFELINE_B);
Expand Down

0 comments on commit 7a6c520

Please sign in to comment.