KTIJ-25232 [FIR IDE] Do not shorten properties with non-trivial receiver

If property call receiver is something real (like another property or a
function call), then it should not be shortened because the semantics
might change

^KTIJ-25232 Fixed
This commit is contained in:
Roman Golyshev
2023-04-13 14:39:38 +02:00
committed by teamcity
parent 4ad16d5cf6
commit f662908440
7 changed files with 74 additions and 0 deletions
@@ -874,6 +874,10 @@ private class ElementsToShortenCollector(
val availableCallables = shorteningContext.findPropertiesInScopes(scopes, callableSymbol.name)
val firPropertyAccess = qualifiedProperty.getOrBuildFir(firResolveSession) as? FirQualifiedAccessExpression ?: return
// if explicit receiver is a property access or a function call, we cannot shorten it
if (firPropertyAccess.explicitReceiver !is FirResolvedQualifier) return
if (availableCallables.isNotEmpty() && shortenIfAlreadyImported(firPropertyAccess, callableSymbol, referenceExpression)) {
addElementToShorten(ShortenQualifier(qualifiedProperty))
return
@@ -298,6 +298,18 @@ public class FirIdeNormalAnalysisSourceModuleReferenceShortenerTestGenerated ext
runTest("analysis/analysis-api/testData/components/referenceShortener/referenceShortener/sameTypeNamesWithinSameScopes.kt");
}
@Test
@TestMetadata("selfPropertyChain.kt")
public void testSelfPropertyChain() throws Exception {
runTest("analysis/analysis-api/testData/components/referenceShortener/referenceShortener/selfPropertyChain.kt");
}
@Test
@TestMetadata("selfPropertyChain1.kt")
public void testSelfPropertyChain1() throws Exception {
runTest("analysis/analysis-api/testData/components/referenceShortener/referenceShortener/selfPropertyChain1.kt");
}
@Test
@TestMetadata("shortenAlreadyImportedClass.kt")
public void testShortenAlreadyImportedClass() throws Exception {