This is rather an early experiment that started to work only on

<?php

function foo(): int {}

foo();
$f = foo(...);

but doesn't work on methods yet.

For now, I am following the current behaviour on a real instance and not sure if I should include all needed tests in GotoDefinitionHandlerTest or somewhere else.

The path through WorseReflection is, as usual, very complex, and I easily get lost there. On the other hand, for comparison, I have the path of the "normal" call expression and I'm trying to figure out the differences.

This way, I tested what happens (in NodeContextFromMemberAccess) when we return a MethodCallContext for FCC: gotdef works, but we get the wrong type (the one returned by the method). It turns out that a MethodCallContext with the FCC type passes both the old and new tests. Even if this change is based on guesswork, I hope this is a good starting point for review.