Fix JUnit 5 migration issues in org.eclipse.jface.tests

This commit fixes compilation and test failures introduced during the
JUnit 4 to JUnit 5 migration in org.eclipse.jface.tests.

Changes:
- Added missing import for org.junit.jupiter.api.function package in
  MANIFEST.MF to resolve NoClassDefFoundError for Executable class
  used by assertThrows() in LabelProviderTest
- Fixed assertEquals parameter order in Bug205700TreeViewerTest to match
  JUnit 5 signature: assertEquals(expected, actual, message)
- Fixed assertEquals parameter order in WizardProgressMonitorTest to match
  JUnit 5 signature: assertEquals(expected, actual, message)

The JUnit 5 assertEquals method has a different parameter order than
JUnit 4. In JUnit 4 it was assertEquals(message, expected, actual) but
in JUnit 5 it is assertEquals(expected, actual, message).

All 1298 tests now pass successfully (0 failures, 0 errors, 3 skipped).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: 2 people

tests/org.eclipse.jface.tests/META-INF/MANIFEST.MF

@@ -13,6 +13,7 @@ Require-Bundle: org.junit;bundle-version="4.12.0",
  org.eclipse.ui.tests.harness
 Import-Package: org.osgi.framework,
  org.junit.jupiter.api;version="[5.14.0,6.0.0)",
+ org.junit.jupiter.api.function;version="[5.14.0,6.0.0)",
  org.junit.platform.suite.api;version="[1.14.0,2.0.0)"
 Export-Package: org.eclipse.jface.tests.fieldassist;x-internal:=true,
  org.eclipse.jface.tests.preferences;x-internal:=true,

tests/org.eclipse.jface.tests/src/org/eclipse/jface/tests/viewers/Bug205700TreeViewerTest.java

@@ -88,7 +88,7 @@ private void assertItemNames(String[] names) {
 	}
 
 	private void assertItemName(int index, String name) {
-		assertEquals("at " + index, name, viewer.getTree().getItem(index).getText());
+		assertEquals(name, viewer.getTree().getItem(index).getText(), "at " + index);
 	}
 
 	@Test

tests/org.eclipse.jface.tests/src/org/eclipse/jface/tests/wizards/WizardProgressMonitorTest.java

@@ -84,8 +84,8 @@ public void testProgressLabelsClearedBug271530() throws Exception {
 
 	protected void performAsserts() {
 
-		assertEquals("The progress monitor's label should have been cleared", //$NON-NLS-1$
-				"", dialog.getProgressMonitorLabelText()); //$NON-NLS-1$
+		assertEquals("", dialog.getProgressMonitorLabelText(), //$NON-NLS-1$
+				"The progress monitor's label should have been cleared"); //$NON-NLS-1$
 
 		String subTask = dialog.getProgressMonitorSubTaskText();
 		if (subTask != null && !subTask.isEmpty()) {
@@ -97,8 +97,8 @@ protected IRunnableWithProgress getRunnable(final String taskName) {
 		return monitor -> {
 
 			// check that the label is empty
-			assertEquals("The progress monitor's label is not initially empty", //$NON-NLS-1$
-					"", dialog.getProgressMonitorLabelText()); //$NON-NLS-1$
+			assertEquals("", dialog.getProgressMonitorLabelText(), //$NON-NLS-1$
+					"The progress monitor's label is not initially empty"); //$NON-NLS-1$
 
 			// check the subtask as well
 			String subTask = dialog.getProgressMonitorSubTaskText();