forked from eclipse-platform/eclipse.platform
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Call the "activated" method of the default tab in a Launch Config Dialog
Also add a new test bundle that contains the proper regression test. Bundle: org.eclipse.debug.ui.tests Class: LaunchConfigurationTabGroupViewerTest This class also contains a regression test for eclipse-platform#766 Fixes: eclipse-platform#859
- Loading branch information
1 parent
b345854
commit a2cdf99
Showing
8 changed files
with
344 additions
and
13 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
188 changes: 188 additions & 0 deletions
188
...pse.debug.tests/src/org/eclipse/debug/tests/ui/LaunchConfigurationTabGroupViewerTest.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,188 @@ | ||
| /******************************************************************************* | ||
| * Copyright (c) 2018, 2019 SAP SE and others. | ||
| * All rights reserved. This program and the accompanying materials | ||
| * are made available under the terms of the Eclipse Public License v2.0 | ||
| * which accompanies this distribution, and is available at | ||
| * http://www.eclipse.org/legal/epl-v10.html | ||
| * | ||
| * Contributors: | ||
| * SAP SE - initial version | ||
| *******************************************************************************/ | ||
|
|
||
| package org.eclipse.debug.tests.ui; | ||
|
|
||
| import static org.hamcrest.CoreMatchers.not; | ||
| import static org.hamcrest.MatcherAssert.assertThat; | ||
|
|
||
| import java.util.concurrent.atomic.AtomicReference; | ||
|
|
||
| import org.eclipse.core.runtime.CoreException; | ||
| import org.eclipse.debug.core.DebugPlugin; | ||
| import org.eclipse.debug.core.ILaunchConfigurationType; | ||
| import org.eclipse.debug.core.ILaunchConfigurationWorkingCopy; | ||
| import org.eclipse.debug.core.ILaunchManager; | ||
| import org.eclipse.debug.internal.ui.DebugUIPlugin; | ||
| import org.eclipse.debug.internal.ui.launchConfigurations.LaunchConfigurationPresentationManager; | ||
| import org.eclipse.debug.internal.ui.launchConfigurations.LaunchConfigurationsDialog; | ||
| import org.eclipse.debug.ui.IDebugUIConstants; | ||
| import org.eclipse.debug.ui.ILaunchConfigurationDialog; | ||
| import org.eclipse.debug.ui.ILaunchConfigurationTab; | ||
| import org.eclipse.debug.ui.ILaunchConfigurationTabGroup; | ||
| import org.eclipse.swt.widgets.Display; | ||
| import org.junit.Before; | ||
| import org.junit.Test; | ||
|
|
||
| public class LaunchConfigurationTabGroupViewerTest { | ||
|
|
||
| private static interface ThrowingRunnable<T extends Exception> { | ||
| void run() throws T; | ||
| } | ||
|
|
||
| private static final String LAUNCH_CONFIG_TYPE_ID = "org.eclipse.debug.ui.tests.launchConfigurationType1"; | ||
| private static final String LAUNCH_CONFIG_MODE = ILaunchManager.RUN_MODE; | ||
| private ILaunchConfigurationType fLaunchConfigurationType; | ||
| private LaunchConfigurationsDialog fLaunchConfigurationsDialog; | ||
|
|
||
| @Before | ||
| public void createDialog() throws CoreException { | ||
| fLaunchConfigurationType = getLaunchManager().getLaunchConfigurationType(LAUNCH_CONFIG_TYPE_ID); | ||
| ILaunchConfigurationTabGroup tabGroup = getLaunchConfigurationTabGroup(fLaunchConfigurationType); | ||
|
|
||
| fLaunchConfigurationsDialog = (LaunchConfigurationsDialog) createLaunchConfigurationDialog(); | ||
| tabGroup.createTabs(fLaunchConfigurationsDialog, ILaunchManager.RUN_MODE); | ||
| } | ||
|
|
||
| @Test | ||
| public void tesAllTabsAreInitializedByDefault() throws CoreException { | ||
| // Create a launch configuration with a unique name | ||
| ThrowingRunnable<CoreException> createAndSelect1LaunchConfig = () -> { | ||
| fLaunchConfigurationsDialog.getTabViewer().setInput(createLaunchConfigurationInstance()); | ||
| }; | ||
|
|
||
| final ILaunchConfigurationTab[] tabs = runOnDialog(createAndSelect1LaunchConfig); | ||
|
|
||
| for (int i = 0; i < tabs.length; i++) { | ||
| assertThat("Tab " + i + " was not initialized", ((SpyTab) tabs[i]).isInitialized()); | ||
| } | ||
| } | ||
|
|
||
| @Test | ||
| public void testFirstTabIsActivatedByDefault() throws CoreException { | ||
| // Create a launch configuration with a unique name | ||
| ThrowingRunnable<CoreException> createAndSelect1LaunchConfig = () -> { | ||
| fLaunchConfigurationsDialog.getTabViewer().setInput(createLaunchConfigurationInstance()); | ||
| }; | ||
|
|
||
| final ILaunchConfigurationTab[] tabs = runOnDialog(createAndSelect1LaunchConfig); | ||
| assertThat("The 1st tab was not activated", ((SpyTab) tabs[0]).isActivated()); | ||
| } | ||
|
|
||
| @Test | ||
| public void testOtherTabInOtherConfigIsActivated() throws CoreException { | ||
| int secondTabIndex = 1; | ||
|
|
||
| ThrowingRunnable<CoreException> setActiveTab = () -> { | ||
| // Create and select launch config | ||
| fLaunchConfigurationsDialog.getTabViewer().setInput(createLaunchConfigurationInstance()); | ||
|
|
||
| // Select another tab | ||
| fLaunchConfigurationsDialog.getTabViewer().setActiveTab(secondTabIndex); | ||
|
|
||
| // Create a new launch config. This one should activate the same tab by default. | ||
| fLaunchConfigurationsDialog.getTabViewer().setInput(createLaunchConfigurationInstance()); | ||
| }; | ||
|
|
||
| final ILaunchConfigurationTab[] tabs = runOnDialog(setActiveTab); | ||
|
|
||
| assertThat("The 1st tab of the other launch configuration shouldn't have been activated", | ||
| not(((SpyTab) tabs[0]).isActivated())); | ||
| assertThat("The tab was not activated", ((SpyTab) tabs[secondTabIndex]).isActivated()); | ||
| } | ||
|
|
||
| @Test | ||
| public void testOtherTabIsActivated() throws CoreException { | ||
| int secondTabIndex = 1; | ||
|
|
||
| ThrowingRunnable<CoreException> setActiveTab = () -> { | ||
| // Create and select launch config | ||
| fLaunchConfigurationsDialog.getTabViewer().setInput(createLaunchConfigurationInstance()); | ||
|
|
||
| // Select another tab | ||
| fLaunchConfigurationsDialog.getTabViewer().setActiveTab(secondTabIndex); | ||
| }; | ||
|
|
||
| final ILaunchConfigurationTab[] tabs = runOnDialog(setActiveTab); | ||
|
|
||
| assertThat("The tab was not activated", ((SpyTab) tabs[secondTabIndex]).isActivated()); | ||
| } | ||
|
|
||
| private ILaunchConfigurationWorkingCopy createLaunchConfigurationInstance() throws CoreException { | ||
| return fLaunchConfigurationType.newInstance(null, "MyLaunchConfiguration_" + System.currentTimeMillis()); | ||
| } | ||
|
|
||
| private <T extends Exception> ILaunchConfigurationTab[] runOnDialog(ThrowingRunnable<T> runnable) { | ||
| AtomicReference<ILaunchConfigurationTab[]> tabsRef = new AtomicReference<>(); | ||
| AtomicReference<Throwable> throwableRef = new AtomicReference<>(); | ||
|
|
||
| Display.getCurrent().asyncExec(() -> { | ||
| try { | ||
|
|
||
| runnable.run(); | ||
|
|
||
| // I need to store the tabs here because the tab viewer (and all its tabs) are | ||
| // gone as soon as the dialog is closed | ||
| tabsRef.set(fLaunchConfigurationsDialog.getTabs()); | ||
|
|
||
| } catch (Throwable e) { | ||
| // neither calling "fail" not throwing an exception will let the test fail so I | ||
| // need to store this and check it outside of the runnable | ||
| throwableRef.set(e); | ||
| DebugPlugin.log(e); | ||
| } finally { | ||
| fLaunchConfigurationsDialog.close(); | ||
| } | ||
| }); | ||
|
|
||
| fLaunchConfigurationsDialog.open(); | ||
|
|
||
| if (throwableRef.get() != null) { | ||
| throw new AssertionError("An exception occurred while executing the runnable.", throwableRef.get()); | ||
| } | ||
|
|
||
| return tabsRef.get(); | ||
| } | ||
|
|
||
| /** | ||
| * Returns the standard java launch tab group | ||
| * | ||
| * @return the standard java launch tab group | ||
| * @throws CoreException | ||
| * | ||
| */ | ||
| protected ILaunchConfigurationTabGroup getLaunchConfigurationTabGroup( | ||
| ILaunchConfigurationType launchConfigurationType) throws CoreException { | ||
| return LaunchConfigurationPresentationManager.getDefault().getTabGroup(launchConfigurationType, | ||
| LAUNCH_CONFIG_MODE); | ||
| } | ||
|
|
||
| /** | ||
| * Returns an instance of the launch configuration dialog on the the specified | ||
| * launch mode | ||
| * | ||
| * @return an new instance of <code>IlaunchConfigurationDialog</code> | ||
| * | ||
| */ | ||
| protected ILaunchConfigurationDialog createLaunchConfigurationDialog() { | ||
| return new LaunchConfigurationsDialog(null, DebugUIPlugin.getDefault().getLaunchConfigurationManager() | ||
| .getLaunchGroup(IDebugUIConstants.ID_DEBUG_LAUNCH_GROUP)); | ||
| } | ||
|
|
||
| /** | ||
| * Returns the launch manager | ||
| * | ||
| * @return launch manager | ||
| */ | ||
| protected ILaunchManager getLaunchManager() { | ||
| return DebugPlugin.getDefault().getLaunchManager(); | ||
| } | ||
| } |
73 changes: 73 additions & 0 deletions
73
debug/org.eclipse.debug.tests/src/org/eclipse/debug/tests/ui/SpyTab.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,73 @@ | ||
| /******************************************************************************* | ||
| * Copyright (c) 2018, 2019 SAP SE and others. | ||
| * All rights reserved. This program and the accompanying materials | ||
| * are made available under the terms of the Eclipse Public License v2.0 | ||
| * which accompanies this distribution, and is available at | ||
| * http://www.eclipse.org/legal/epl-v10.html | ||
| * | ||
| * Contributors: | ||
| * SAP SE - initial version | ||
| *******************************************************************************/ | ||
|
|
||
| package org.eclipse.debug.tests.ui; | ||
|
|
||
| import org.eclipse.debug.core.ILaunchConfiguration; | ||
| import org.eclipse.debug.core.ILaunchConfigurationWorkingCopy; | ||
| import org.eclipse.debug.ui.AbstractLaunchConfigurationTab; | ||
| import org.eclipse.swt.widgets.Composite; | ||
|
|
||
| /** | ||
| * A Tab whose sole purpose is to say if it was initialized and activated | ||
| * properly | ||
| */ | ||
| public abstract class SpyTab extends AbstractLaunchConfigurationTab { | ||
|
|
||
| private boolean initialized; | ||
| private boolean activated; | ||
|
|
||
| @Override | ||
| public void createControl(Composite parent) { | ||
| } | ||
|
|
||
| @Override | ||
| public String getName() { | ||
| return getClass().getSimpleName(); | ||
| } | ||
|
|
||
| @Override | ||
| public void initializeFrom(ILaunchConfiguration configuration) { | ||
| initialized = true; | ||
| } | ||
|
|
||
| @Override | ||
| public void activated(ILaunchConfigurationWorkingCopy workingCopy) { | ||
| activated = true; | ||
| } | ||
|
|
||
| @Override | ||
| public void performApply(ILaunchConfigurationWorkingCopy configuration) { | ||
| } | ||
|
|
||
| @Override | ||
| public void setDefaults(ILaunchConfigurationWorkingCopy configuration) { | ||
| } | ||
|
|
||
| public boolean isInitialized() { | ||
| return initialized; | ||
| } | ||
|
|
||
| public boolean isActivated() { | ||
| return activated; | ||
| } | ||
|
|
||
| // These 3 are necessary because I need 3 tabs in the launch config and using | ||
| // always the same kind of tab produces incorrect results | ||
| public static class SpyTabA extends SpyTab { | ||
| } | ||
|
|
||
| public static class SpyTabB extends SpyTab { | ||
| } | ||
|
|
||
| public static class SpyTabC extends SpyTab { | ||
| } | ||
| } |
28 changes: 28 additions & 0 deletions
28
debug/org.eclipse.debug.tests/src/org/eclipse/debug/tests/ui/SpyTabGroup.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| /******************************************************************************* | ||
| * Copyright (c) 2018, 2019 SAP SE and others. | ||
| * All rights reserved. This program and the accompanying materials | ||
| * are made available under the terms of the Eclipse Public License v2.0 | ||
| * which accompanies this distribution, and is available at | ||
| * http://www.eclipse.org/legal/epl-v10.html | ||
| * | ||
| * Contributors: | ||
| * SAP SE - initial version | ||
| *******************************************************************************/ | ||
|
|
||
| package org.eclipse.debug.tests.ui; | ||
|
|
||
| import org.eclipse.debug.tests.ui.SpyTab.SpyTabA; | ||
| import org.eclipse.debug.tests.ui.SpyTab.SpyTabB; | ||
| import org.eclipse.debug.tests.ui.SpyTab.SpyTabC; | ||
| import org.eclipse.debug.ui.AbstractLaunchConfigurationTabGroup; | ||
| import org.eclipse.debug.ui.ILaunchConfigurationDialog; | ||
| import org.eclipse.debug.ui.ILaunchConfigurationTab; | ||
|
|
||
| public class SpyTabGroup extends AbstractLaunchConfigurationTabGroup { | ||
|
|
||
| @Override | ||
| public void createTabs(ILaunchConfigurationDialog dialog, String mode) { | ||
| setTabs(new ILaunchConfigurationTab[] { new SpyTabA(), new SpyTabB(), new SpyTabC() }); | ||
| } | ||
|
|
||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.