Minor: print "<root>" for root package instead empty string in Lookup tests.
Additionally fixed warnings.
Original commit: 2d859d6191
This commit is contained in:
@@ -23,7 +23,6 @@ import org.jetbrains.kotlin.test.JetTestUtils
|
||||
import org.jetbrains.kotlin.utils.join
|
||||
import java.io.File
|
||||
import java.util.*
|
||||
import kotlin.test.fail
|
||||
|
||||
private val DECLARATION_KEYWORDS = listOf("interface", "class", "enum class", "object", "fun", "val", "var")
|
||||
|
||||
@@ -34,7 +33,7 @@ abstract class AbstractLookupTrackerTest : AbstractIncrementalJpsTest(
|
||||
// ignore KDoc like comments which starts with `/**`, example: /** text */
|
||||
val COMMENT_WITH_LOOKUP_INFO = "/\\*[^*]+\\*/".toRegex()
|
||||
|
||||
override fun createLookupTracker() = TestLookupTracker()
|
||||
override fun createLookupTracker(): LookupTracker = TestLookupTracker()
|
||||
|
||||
override fun checkLookups(lookupTracker: LookupTracker) {
|
||||
if (lookupTracker !is TestLookupTracker) throw AssertionError("Expected TestLookupTracker, but: ${lookupTracker.javaClass}")
|
||||
@@ -50,7 +49,7 @@ abstract class AbstractLookupTrackerTest : AbstractIncrementalJpsTest(
|
||||
|
||||
val text = file.readText()
|
||||
|
||||
val matchResult = COMMENT_WITH_LOOKUP_INFO.match(text)
|
||||
val matchResult = COMMENT_WITH_LOOKUP_INFO.find(text)
|
||||
if (matchResult != null) {
|
||||
fail("File $file unexpectedly contains multiline comments. In range ${matchResult.range} found: ${matchResult.value} in $text")
|
||||
}
|
||||
@@ -61,7 +60,7 @@ abstract class AbstractLookupTrackerTest : AbstractIncrementalJpsTest(
|
||||
val columnToLookups = lookupsFromLine.groupBy { it.lookupColumn!! }.toList().sortedBy { it.first }
|
||||
|
||||
val lineContent = lines[line - 1]
|
||||
val parts = ArrayList<CharSequence>(columnToLookups.size() * 2)
|
||||
val parts = ArrayList<CharSequence>(columnToLookups.size * 2)
|
||||
|
||||
var start = 0
|
||||
|
||||
@@ -81,7 +80,7 @@ abstract class AbstractLookupTrackerTest : AbstractIncrementalJpsTest(
|
||||
else -> "(" + it.name + ")"
|
||||
}
|
||||
|
||||
it.scopeKind.toString()[0].toLowerCase().toString() + ":" + it.scopeFqName + name
|
||||
it.scopeKind.toString()[0].toLowerCase().toString() + ":" + it.scopeFqName.let { if (it.isNotEmpty()) it else "<root>" } + name
|
||||
}
|
||||
|
||||
parts.add(lookups)
|
||||
@@ -89,7 +88,7 @@ abstract class AbstractLookupTrackerTest : AbstractIncrementalJpsTest(
|
||||
start = end
|
||||
}
|
||||
|
||||
lines[line - 1] = parts.join("") + lineContent.subSequence(start, lineContent.length())
|
||||
lines[line - 1] = parts.join("") + lineContent.subSequence(start, lineContent.length)
|
||||
}
|
||||
|
||||
val actual = lines.joinToString("\n")
|
||||
|
||||
Reference in New Issue
Block a user