Fix: gutter icon only on new ApplicationConfig, not use statement

- Restrict line marker and producer to ClassReference inside
  NewExpression (not use/import statements)
- Add isTestoConfigFile() to mixin.kt for detecting config files
- Include config files in isTestoFile() so right-click Run works
- Handle PhpFile config in setupConfiguration with --config flag

https://claude.ai/code/session_01TmFvjfMtoxTvddQRBzsbPN

Author: claude

src/main/kotlin/com/github/xepozz/testo/mixin.kt

@@ -8,6 +8,8 @@ import com.jetbrains.php.lang.psi.PhpFile
 import com.jetbrains.php.lang.psi.elements.Method
 import com.jetbrains.php.lang.psi.elements.Function
 import com.jetbrains.php.lang.psi.elements.PhpAttributesOwner
+import com.jetbrains.php.lang.psi.elements.ClassReference
+import com.jetbrains.php.lang.psi.elements.NewExpression
 import com.jetbrains.php.lang.psi.elements.PhpClass
 
 fun PsiElement.isTestoExecutable() = isTestoFunction() || isTestoMethod() || isTestoBench()
@@ -42,10 +44,13 @@ fun PsiElement.isTestoClass() = when (this) {
 }
 
 fun PsiFile.isTestoFile() = when (this) {
-    is PhpFile -> TestoTestDescriptor.isTestClassName(name.substringBeforeLast(".")) || (isTestoClassFile() || isTestoFunctionFile() || isTestBenchFile())
+    is PhpFile -> TestoTestDescriptor.isTestClassName(name.substringBeforeLast(".")) || isTestoClassFile() || isTestoFunctionFile() || isTestBenchFile() || isTestoConfigFile()
     else -> false
 }
 
+fun PhpFile.isTestoConfigFile() = PsiTreeUtil.findChildrenOfType(this, ClassReference::class.java)
+    .any { it.parent is NewExpression && it.fqn == TestoClasses.APPLICATION_CONFIG }
+
 fun PhpFile.isTestoClassFile() = PsiTreeUtil.findChildrenOfType(this, PhpClass::class.java)
     .any { it.isTestoClass() }
 

src/main/kotlin/com/github/xepozz/testo/tests/TestoTestRunLineMarkerProvider.kt

@@ -20,6 +20,7 @@ import com.jetbrains.php.lang.lexer.PhpTokenTypes
 import com.jetbrains.php.lang.psi.PhpPsiUtil
 import com.jetbrains.php.lang.psi.elements.ClassReference
 import com.jetbrains.php.lang.psi.elements.Method
+import com.jetbrains.php.lang.psi.elements.NewExpression
 import com.jetbrains.php.lang.psi.elements.Function
 import com.jetbrains.php.lang.psi.elements.PhpAttribute
 import com.jetbrains.php.lang.psi.elements.PhpAttributesOwner
@@ -49,7 +50,7 @@ class TestoTestRunLineMarkerProvider : RunLineMarkerContributor() {
         val element = leaf.parent as? PhpPsiElement ?: return null
 
         return when {
-            element is ClassReference && element.fqn == TestoClasses.APPLICATION_CONFIG -> {
+            element is ClassReference && element.parent is NewExpression && element.fqn == TestoClasses.APPLICATION_CONFIG -> {
                 getLocationHint(element.containingFile)
             }
 

src/main/kotlin/com/github/xepozz/testo/tests/run/TestoRunConfigurationProducer.kt

@@ -8,6 +8,7 @@ import com.github.xepozz.testo.isTestoBench
 import com.github.xepozz.testo.isTestoClass
 import com.github.xepozz.testo.isTestoDataProviderLike
 import com.github.xepozz.testo.isTestoExecutable
+import com.github.xepozz.testo.isTestoConfigFile
 import com.github.xepozz.testo.isTestoFile
 import com.github.xepozz.testo.isTestoFunction
 import com.github.xepozz.testo.isTestoMethod
@@ -36,6 +37,7 @@ import com.jetbrains.php.lang.psi.PhpFile
 import com.jetbrains.php.lang.psi.elements.ClassReference
 import com.jetbrains.php.lang.psi.elements.Function
 import com.jetbrains.php.lang.psi.elements.Method
+import com.jetbrains.php.lang.psi.elements.NewExpression
 import com.jetbrains.php.lang.psi.elements.PhpAttribute
 import com.jetbrains.php.lang.psi.elements.PhpClass
 import com.jetbrains.php.lang.psi.elements.PhpNamedElement
@@ -63,7 +65,7 @@ class TestoRunConfigurationProducer : PhpTestConfigurationProducer<TestoRunConfi
     ): PsiElement? {
         val testRunnerSettings = testRunnerSettings as TestoRunnerSettings
 
-        if (element is ClassReference && element.fqn == TestoClasses.APPLICATION_CONFIG) {
+        if (element is ClassReference && element.parent is NewExpression && element.fqn == TestoClasses.APPLICATION_CONFIG) {
             testRunnerSettings.scope = PhpTestRunnerSettings.Scope.ConfigurationFile
             testRunnerSettings.isUseAlternativeConfigurationFile = true
             testRunnerSettings.configurationFilePath = virtualFile.path
@@ -126,6 +128,12 @@ class TestoRunConfigurationProducer : PhpTestConfigurationProducer<TestoRunConfi
                 }
             }
         }
+        if (element is PhpFile && element.isTestoConfigFile()) {
+            testRunnerSettings.scope = PhpTestRunnerSettings.Scope.ConfigurationFile
+            testRunnerSettings.isUseAlternativeConfigurationFile = true
+            testRunnerSettings.configurationFilePath = virtualFile.path
+            return element
+        }
         val result = super.setupConfiguration(testRunnerSettings, element, virtualFile)
         return result
     }
@@ -134,7 +142,7 @@ class TestoRunConfigurationProducer : PhpTestConfigurationProducer<TestoRunConfi
         testRunnerSettings: PhpTestRunnerSettings,
         element: PsiElement
     ): Boolean {
-        if (element is ClassReference && element.fqn == TestoClasses.APPLICATION_CONFIG) {
+        if (element is ClassReference && element.parent is NewExpression && element.fqn == TestoClasses.APPLICATION_CONFIG) {
             return testRunnerSettings.scope == PhpTestRunnerSettings.Scope.ConfigurationFile
                 && testRunnerSettings.configurationFilePath == element.containingFile.virtualFile.path
         }
@@ -284,7 +292,7 @@ class TestoRunConfigurationProducer : PhpTestConfigurationProducer<TestoRunConfi
     }
 
     private fun findTestElement(target: PsiElement?): PsiElement? = when (target) {
-        is ClassReference -> target.takeIf { it.fqn == TestoClasses.APPLICATION_CONFIG }
+        is ClassReference -> target.takeIf { it.parent is NewExpression && it.fqn == TestoClasses.APPLICATION_CONFIG }
         is PhpAttribute -> target.takeIf { it.owner.isTestoExecutable() || it.owner.isTestoDataProviderLike() }
         is Function -> target.takeIf { it.isTestoExecutable() || it.isTestoDataProviderLike() }
         is PhpClass -> target.takeIf { it.isTestoClass() }