Use abbreviation when returning arguments for a KSType #928
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR fixes a problem in KSType where it was not returning the right type arguments for TypeAliases.
For input:
Querying the typeArguments of
x
would return[Int]
while also returning its declaration asMyList
.This makes it looks like
MyList<Int><Int>
.To fix it, I updated
KSType.arguments
to use the abbreviation type instead of the KotlinType.Also, updated the test to add more cases and make it print the arguments for each type. I refactored the
test a bit to print types of properties instead of all found types to make the output more readable (happy to revert that part).
Fixes: #881
Test: typeAlias.kt