fix: Non-ASCII test names being escaped when creating run configurations

[chika3742]

StringUtil.escapeProperty() was incorrectly applied to test names in CommonTestConfigUtils.findTestName(), converting non-ASCII characters (e.g. Japanese "テスト") to Unicode escape sequences. This caused flutter test --plain-name to receive an escaped string that did not match the actual test name, preventing the test from running.

Fix by removing the escapeProperty() call, which is intended for .properties file encoding and is unnecessary here.

fixes #7985

---

• I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

• See our [contributor guide](https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md for general expectations for PRs.
• Larger or significant changes should be discussed in an issue before creating a PR.
• Dart contributions to our repos should follow the Dart style guide and use dart format.
• Java and Kotlin contributions should strive to follow Java and Kotlin best practices (discussion).

[gemini-code-assist]

Code Review

This pull request correctly resolves an issue where non-ASCII test names were improperly escaped, preventing tests from running. The refactoring of test name extraction logic into a separate, testable method is a positive change for code quality. My review includes suggestions to further improve the new code by adhering to the repository's style guide, specifically by using java.util.Optional to avoid returning null values, which enhances null safety.

[helin24]

Thanks for the contribution! This seems like a good change. Are you able to run locally (or build a local copy of the plugin) and provide a gif of this working? (also, sorry about the delay in my response. I missed checking PRs last week)

[chika3742]

@helin24 Thanks for the review! No worries about the delay.

Here's a demo of it working locally:

Before	After

While recording this, I noticed that the first test marker in the file is not showing sometimes. After some investigation, I found the following issues:

1. Comparison between the cache and the value obtained from outlineService is incorrect, causing outlineOutdated to always be true if the cache is present.
   

   flutter-intellij/src/io/flutter/run/common/CommonTestConfigUtils.java

   Lines 95 to 96 in 6572cf2

   	final OutlineCache entry = cache.get(path);
   	outlineOutdated = cache.containsKey(path) && outline != entry.outline;

2. Because the workaround applied for main functions (line 81) is not applied for other test calls, the first test markers disappears when editing the file.
   

   flutter-intellij/src/io/flutter/run/common/TestLineMarkerContributor.java

   Lines 62 to 81 in 6572cf2

   	if (dartId.getParent().getParent() instanceof DartCallExpression dartCallExpression) {
   	final TestType testCall = testConfigUtils.asTestCall(dartCallExpression);
   	if (testCall != null) {
   	final Icon icon = getTestStateIcon(element, testCall.getIcon());
   	final Function<PsiElement, String> tooltipProvider =
   	psiElement -> testCall.getTooltip(psiElement, testConfigUtils);
   	return new RunLineMarkerContributor.Info(icon, ExecutorAction.getActions(), tooltipProvider);
   	}
   	}
   	else if (dartId.getParent().getParent() instanceof DartFunctionDeclarationWithBodyOrNative) {
   	if (testConfigUtils instanceof TestConfigUtils) {
   	// TODO(messick) Find a better way to eliminate duplicate pop-up menu entries.
   	// The issue is that there are two contributors, one for normal Flutter, one for Bazel,
   	// and they should not both produce contributions at the same time.
   	return null;
   	}
   	if ("main".equals(dartId.getText())) {
   	// There seems to be an intermittent timing issue that causes the first test call to not get marked.
   	// Priming the cache here solves it.
   	testConfigUtils.refreshOutline(element);

I can open a separate PR (issue) to fix these if you'd like.

[helin24]

Thanks for the demo! This LGTM, and I think a separate PR for the missing arrow would be reasonable.

Edit: A couple other things - add a note to CHANGELOG and make sure you are in the AUTHORS file.

[helin24]

lgtm - adding @pq to review and merge.