Is this the right way to check for inheritance or ...

# konsist

[ceedee]

Is this the right way to check for inheritance or is there an out-of-the-box way using Konsist?

@Test
fun `every class with a name ending on FragmentDelegate needs to be (indirectly) derived from BaseFragmentDelegate`() {
    Konsist.scopeFromProduction()
        .classes()
        .withNameEndingWith("FragmentDelegate")
        .takeIf { it.isNotEmpty() }
        ?.assert { klass ->
            val classToCheckForInheritance = Class.forName(klass.fullyQualifiedName)
            BaseFragmentDelegate::class.java.isAssignableFrom(classToCheckForInheritance)
        }
}

[igor.wojda]

@Test
fun `every class with a name ending on FragmentDelegate needs to be (indirectly) derived from BaseFragmentDelegate`() {
    Konsist.scopeFromProduction()
        .classes()
        .withNameEndingWith("FragmentDelegate")
        ?.assert {
            it.hasParentOf<BaseFragmentDelegate>()
        }
}

[igor.wojda]

BTW .takeIf { it.isNotEmpty() } should not be required - please tell me whats happens if this is not present (this may be a bug) 🤔