Fix analyzer issues

[pshao25]

Fix Azure/autorest.csharp#3710
Root cause: Azure/autorest.csharp#3710 (comment)
Quote here:

The get sub client method returns the sub client called Operation, which is a criterion of a service method in analyzer

public virtual Operation GetOperationClient(string apiVersion = "1.0.0")
{
    Argument.AssertNotNull(apiVersion, nameof(apiVersion));

    return new Operation(ClientDiagnostics, _pipeline, _endpoint, apiVersion);
}

Solution: When checking whether a method is a service method, not just checking the type name, but also checking the namespace name.

Fix Azure/autorest.csharp#3708
Root cause: Azure/autorest.csharp#3708 (comment)
Quote here:

We generate static properties in clients, so there will be a static constructor without any parameters implicitly added, which should not be in the searching scope of a constructor equivalent.

Solution: exclude static constructor when looking for a equivalent constructor method, because we only care about the constructors explicitly written by generator or customization.

Preview in azure-sdk-for-net: Azure/azure-sdk-for-net#39672

[jsquire]

Can we please add details to the description. What are the intended changes to the current ruleset? The code itself has few comments as to the intent of the rule and what pattern its intending to validate.

[pshao25]

Can we please add details to the description. What are the intended changes to the current ruleset? The code itself has few comments as to the intent of the rule and what pattern its intending to validate.

Updated.

Remediated

[pshao25]

@m-nash could you help review? This blocks we change constructors behaviors as we discussed in Azure/autorest.csharp#3706 (comment)