Code review feedback change: code style changes, delete unintended test class inclusion, fix copyright headers.

Author: feilimb

tests/org.eclipse.e4.ui.tests/src/org/eclipse/e4/ui/tests/application/CloseActiveEditorsTest.java

@@ -1,3 +1,15 @@
+/*******************************************************************************
+* Copyright (c) 2024 Feilim Breatnach and others.
+*
+* 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 https://www.eclipse.org/legal/epl-2.0/
+*
+* SPDX-License-Identifier: EPL-2.0
+*
+* Contributors: Feilim Breatnach, Pilz Ireland - PR #2360
+*******************************************************************************/
+
 package org.eclipse.ui.tests.e4;
 
 import java.util.Collections;
@@ -40,8 +52,8 @@
 import org.osgi.framework.ServiceReference;
 
 /**
- * @since 3.5
- *
+ * Tests the enabled when and execution logic within the
+ * {@link CloseAllHandler}.
  */
 public class CloseAllHandlerTest {
 
@@ -69,27 +81,19 @@ private MApplication getApplicationModel() {
 	}
 
 	/**
-	 * <p>
 	 * Tests the enabled when and execution logic within the
 	 * {@link CloseAllHandler}.
-	 * </p>
 	 *
-	 * <p>
 	 * Scenario 1: E4 style part contribution which is tagged as representing an
 	 * 'editor' is closed via the handler (and the enablement of handler is
 	 * checked).
-	 * </p>
 	 *
-	 * <p>
 	 * Scenario 2: compatibility layer type editor is closed via the handler (and
 	 * the enablement of handler is checked).
-	 * </p>
 	 *
-	 * <p>
 	 * Scenario 3: a mix of an open compatibility layer type editor *and* an E4
 	 * style part contribution which is tagged as representing an 'editor' are both
 	 * closed via the handler (and the enablement of handler is checked).
-	 * </p>
 	 */
 	@Test
 	public void testCloseMixedEditorTypes() {
@@ -270,13 +274,9 @@ public IFile getFile() {
 			return null;
 		}
 
-		/**
-		 * {@inheritDoc}
-		 */
 		@Override
 		public IStorage getStorage() {
 			return null;
 		}
 	}
-
-}
+}

tests/org.eclipse.ui.tests/Eclipse UI Tests/org/eclipse/ui/tests/e4/CloseAllHandlerTest.java

@@ -1,16 +1,14 @@
 /*******************************************************************************
- * Copyright (c) 2010 IBM Corporation and others.
- *
- * 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
- * https://www.eclipse.org/legal/epl-2.0/
- *
- * SPDX-License-Identifier: EPL-2.0
- *
- * Contributors:
- *     IBM Corporation - initial API and implementation
- ******************************************************************************/
+* Copyright (c) 2024 Feilim Breatnach and others.
+*
+* 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 https://www.eclipse.org/legal/epl-2.0/
+*
+* SPDX-License-Identifier: EPL-2.0
+*
+* Contributors: Feilim Breatnach, Pilz Ireland - PR #2360
+*******************************************************************************/
 
 package org.eclipse.ui.tests.e4;