fix: correct the data-provider index FQN and the test-method generator tail

- TestoDataProvidersIndex matched the non-existent \Testo\Sample\DataProvider,
  so it indexed nothing real. Point it at TestoClasses.DATA_PROVIDER
  (\Testo\Data\DataProvider) and bump getVersion() (1->2) so stale on-disk
  indexes rebuild on upgrade.
- TestoGenerateTestMethodAction.fillVariablesSegments used substring(min(from,
  length-1)), which re-emitted the final char when a placeholder ended the
  template and threw StringIndexOutOfBounds on empty text. Emit the tail only
  when from < length.

Author: xepozz

src/main/kotlin/com/github/xepozz/testo/actions/TestoGenerateTestMethodAction.kt

@@ -10,7 +10,6 @@ import com.intellij.psi.PsiFile
 import com.jetbrains.php.lang.psi.elements.PhpClass
 import com.jetbrains.php.phpunit.PhpUnitTestDescriptor
 import java.util.*
-import kotlin.math.min
 
 class TestoGenerateTestMethodAction : TestoGenerateMethodActionBase("Testo Test Method") {
     override fun isValidForFile(project: Project, editor: Editor, file: PsiFile) = file.isTestoFile()
@@ -32,7 +31,9 @@ class TestoGenerateTestMethodAction : TestoGenerateMethodActionBase("Testo Test
         while (true) {
             val index = methodText.indexOf("\${CAPITALIZED_NAME}", from)
             if (index < 0) {
-                methodTemplate.addTextSegment(methodText.substring(min(from, methodText.length - 1)))
+                // Emit only the remaining tail. The old min(from, length-1) re-emitted the final char when the
+                // placeholder ended the template, and threw StringIndexOutOfBounds on empty input.
+                if (from < methodText.length) methodTemplate.addTextSegment(methodText.substring(from))
                 return
             }
 

src/main/kotlin/com/github/xepozz/testo/index/TestoDataProvidersIndex.kt

@@ -1,5 +1,6 @@
 package com.github.xepozz.testo.index
 
+import com.github.xepozz.testo.TestoClasses
 import com.github.xepozz.testo.isTestoClass
 import com.intellij.openapi.util.Pair
 import com.intellij.openapi.util.text.StringUtil
@@ -51,7 +52,8 @@ class TestoDataProvidersIndex : FileBasedIndexExtension<String, TestoDataProvide
     override fun getValueExternalizer(): DataExternalizer<TestoDataProvidersIndexType> =
         DataProviderUsageExternalizer.INSTANCE
 
-    override fun getVersion() = 1
+    // Bumped to 2 when DATA_PROVIDER_ATTRIBUTE was corrected, so stale (empty) on-disk indexes rebuild on upgrade.
+    override fun getVersion() = 2
 
     override fun getInputFilter() = FileBasedIndex.InputFilter { it.fileType is PhpFileType }
 
@@ -104,14 +106,15 @@ class TestoDataProvidersIndex : FileBasedIndexExtension<String, TestoDataProvide
 
     companion object Companion {
         val KEY = ID.create<String, TestoDataProvidersIndexType>("Testo.DataProviders")
-        private const val PHPUNIT_DATA_PROVIDER_ATTRIBUTE = "\\Testo\\Sample\\DataProvider"
+        // The real Testo data-provider attribute; the previous "\Testo\Sample\DataProvider" matched nothing.
+        private const val DATA_PROVIDER_ATTRIBUTE = TestoClasses.DATA_PROVIDER
 
         fun getDataProvidersFromAttributes(function: Function): MutableSet<Pair<String, String>> {
             val result = mutableSetOf<Pair<String, String>>()
 
             val targetFQN = function.asSafely<Method>()?.containingClass?.fqn ?: function.fqn
 
-            for (dataProvider in function.getAttributes(PHPUNIT_DATA_PROVIDER_ATTRIBUTE)) {
+            for (dataProvider in function.getAttributes(DATA_PROVIDER_ATTRIBUTE)) {
                 val argument = getAttributeArgument(dataProvider, "provider", 0) ?: continue
                 val methodNameArg = argument as? PhpExpectedFunctionScalarArgument ?: continue
                 val attributeValue = methodNameArg.value