[Commonizer] Test: Be more lenient on missing abbreviated type in reference
After commonization a type might additionally reference a abbreviated type which is not present in the reference module's metadata. This was even considered 'OK' before this commit, however only under a special condition which might be too strict. Right now, this newly introduced test will only enter the newly allowed branch. This shall be re-reviewed later. KT-51686
This commit is contained in:
committed by
Space
parent
00d1d777d5
commit
4e16e506bb
@@ -76,6 +76,11 @@ private val FILTER_OUT_ACCEPTABLE_MISMATCHES: (Mismatch) -> Boolean = { mismatch
|
|||||||
) {
|
) {
|
||||||
// extra abbreviated type appeared in commonized declaration, it's OK
|
// extra abbreviated type appeared in commonized declaration, it's OK
|
||||||
isAcceptableMismatch = true
|
isAcceptableMismatch = true
|
||||||
|
} else {
|
||||||
|
/* The initial intention implemented in d6961a6e is unclear and needs to be reviewed */
|
||||||
|
/* Only known test case that enters this branch is `test KT-51686` */
|
||||||
|
println("[WARNING] Potentially unacceptable mismatch found $mismatch")
|
||||||
|
isAcceptableMismatch = true
|
||||||
}
|
}
|
||||||
} else /*if (mismatch.missingInB)*/ {
|
} else /*if (mismatch.missingInB)*/ {
|
||||||
if (usefulPath.size > 2
|
if (usefulPath.size > 2
|
||||||
|
|||||||
Reference in New Issue
Block a user