Replace emojis in parameterized test display name

This change replaces smiley emojis with ascii,
in new parameterized tests for console line wrapping.
This hopefully avoids confusing GitHub jobs.

In addition, emojis in IOConsoleTests
are replaced with unicode codepoints.

See: #2650

Author: trancexpress

debug/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ConsoleOutputLineTruncateTest.java

@@ -19,6 +19,7 @@
 import java.util.stream.Stream;
 
 import org.eclipse.ui.internal.console.ConsoleOutputLineTruncate;
+import org.junit.jupiter.api.Named;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.MethodSource;
@@ -114,7 +115,7 @@ private static Parameters test(int limit, int chunks, int repeat, String input,
 	};
 
 	private static Stream<Arguments> tests() {
-		return Arrays.stream(TESTS).map(Arguments::of);
+		return Arrays.stream(TESTS).map(ConsoleOutputLineTruncateTest::named);
 	}
 
 	@ParameterizedTest
@@ -134,4 +135,11 @@ public void test(Parameters p) {
 	private static String repeat(String s, int r, String nl) {
 		return (nl + s).repeat(r).substring(nl.length());
 	}
+
+	private static Arguments named(Parameters p) {
+		String name = "limit=" + p.limit + ", chunks=" + p.chunks + ", repeat=" + p.repeat +
+				", input=" + ConsoleOutputLineWrapTest.replaceSmiley(p.input) +
+				", output" + ConsoleOutputLineWrapTest.replaceSmiley(p.output);
+		return Arguments.of(Named.of(name, p));
+	}
 }

debug/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ConsoleOutputLineWrapTest.java

@@ -19,6 +19,7 @@
 import java.util.stream.Stream;
 
 import org.eclipse.ui.internal.console.ConsoleOutputLineWrap;
+import org.junit.jupiter.api.Named;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.MethodSource;
@@ -127,7 +128,7 @@ private static Parameters test(int limit, int chunks, int repeat, String input,
 	};
 
 	private static Stream<Arguments> tests() {
-		return Arrays.stream(TESTS).map(Arguments::of);
+		return Arrays.stream(TESTS).map(ConsoleOutputLineWrapTest::named);
 	}
 
 	@ParameterizedTest
@@ -147,4 +148,21 @@ public void test(Parameters p) {
 	private static String repeat(String s, int r, String nl) {
 		return (nl + s).repeat(r).substring(nl.length());
 	}
+
+	private static Arguments named(Parameters p) {
+		String name = "limit=" + p.limit + ", chunks=" + p.chunks + ", repeat=" + p.repeat +
+				", input=" + replaceSmiley(p.input) +
+				", output" + replaceSmiley(p.output);
+		return Arguments.of(Named.of(name, p));
+	}
+
+	static String replaceSmiley(String s) {
+		/**
+		 * We don't want emojis in test names, to avoid potentially breaking GitHub actions.
+		 * See: https://github.com/eclipse-platform/eclipse.platform/issues/2650
+		 */
+		String smiley = "\uD83D\uDE00";
+		String ascii = ":)";
+		return s.replaceAll(smiley, ascii);
+	}
 }

debug/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/IOConsoleTests.java

@@ -751,13 +751,13 @@ public void testTrimSurrogateCharacters() throws Exception {
 		final IOConsoleTestUtil c = getTestUtil("Test trim newline");
 		try (IOConsoleOutputStream out = c.getDefaultOutputStream()) {
 			c.writeFast("first\n");
-			c.writeFast("1😀2😀3\n", out);
+			c.writeFast("1\uD83D\uDE002\uD83D\uDE003\n", out);
 			c.write("last\n");
 			c.verifyContentByLine("first", 0).verifyContentByLine("last", -2);
 			assertTrue(c.getDocument().getNumberOfLines() > 2, "Document not filled.");
 			c.getConsole().setWaterMarks(8, 16);
 			c.waitForScheduledJobs();
-			c.verifyContentByOffset("😀3\nlas", 0);
+			c.verifyContentByOffset("\uD83D\uDE003\nlas", 0);
 			closeConsole(c);
 		}
 	}
@@ -810,12 +810,12 @@ public void testTrimLongLineSurrogateCharacters() throws Exception {
 		try (IOConsoleOutputStream out = c.getDefaultOutputStream()) {
 			c.getConsole().setLimitLineLength(false, 8);
 			c.writeFast("first\n");
-			c.writeFast("01234😀😀😀\n", out);
+			c.writeFast("01234\uD83D\uDE00\uD83D\uDE00\uD83D\uDE00\n", out);
 			c.write("last\n");
 			c.verifyContentByLine("first", 0).verifyContentByLine("last", -2);
 			assertTrue(c.getDocument().getNumberOfLines() > 2, "Document not filled.");
 			c.waitForScheduledJobs();
-			c.verifyContent("first\n01234😀 ...\nlast\n");
+			c.verifyContent("first\n01234\uD83D\uDE00 ...\nlast\n");
 			closeConsole(c);
 		}
 	}
@@ -870,12 +870,12 @@ public void testWrapLongLineSurrogateCharacters() throws Exception {
 			String nl = c.getConsole().getDocument().getLegalLineDelimiters()[0];
 			c.getConsole().setLimitLineLength(true, 8);
 			c.writeFast("first\n");
-			c.writeFast("0123456😀😀\n", out);
+			c.writeFast("0123456\uD83D\uDE00\uD83D\uDE00\n", out);
 			c.write("last\n");
 			c.verifyContentByLine("first", 0).verifyContentByLine("last", -2);
 			assertTrue(c.getDocument().getNumberOfLines() > 2, "Document not filled.");
 			c.waitForScheduledJobs();
-			c.verifyContent("first\n0123456" + nl + "😀😀\nlast\n");
+			c.verifyContent("first\n0123456" + nl + "\uD83D\uDE00\uD83D\uDE00\nlast\n");
 			closeConsole(c);
 		}
 	}