Can you add a test for global::Generic<int>.MyEvent? I'm expecting it to fail.

For that, I suggest using this extension method

Yes, it failed both for generic and non-generic. I added more tests and fix that

GetRightmostName could return null (at least it's annotated as such), does this need a null check? My gut is it seems like you'd need to have something malformed to the left of the period but still somehow have succeeded in binding the event, so that seems like it'd be a difficult thing to achieve.

@jasonmalinowski Nice catch!

But will it really be something malformed? I think it can be null in a completely valid scenario like this?

using System;

class C
{
    public event EventHandler MyEvent;
    
    public static C CreateC()
    {
        return new C();
    }
    
    public void M2()
    {
        CreateC().MyEvent +=
    }
}

Here memberAccessExpression.Expression is InvocationExpressionSyntax where GetRightmostName returns null. We need to fall into syntaxFactsService below when it's null, I think.

@Youssef1313 Ah right, of course. I was mentally reading through the GetRightmostName() then forgetting that we're passing in the LHS of the original dot, so yeah. If we don't have a test for that case, we should add one regardless,

lhs is not null maybe