Handle assert for missing node kind in isDeclarationVisible for functionType and constructorType

Fix for #1288, by adding missing node kinds to the switch statement and test cases.
I'm not familiar enough with what this function does, but it still seems like this is non-exhaustive; what about a type reference? It seems like that would *not* be visible, but we don't actually account for it.