Relax test SkuTelemetry strictness #1875
This commit is contained in:
parent
fcfa161cb7
commit
5d5438a7f0
|
|
@ -195,7 +195,7 @@ namespace Microsoft.AspNetCore.Authentication.Test.OpenIdConnect
|
|||
ValidateState(actualValues, errors, htmlEncoded);
|
||||
break;
|
||||
case OpenIdConnectParameterNames.SkuTelemetry:
|
||||
ValidateSkuTelemetry(actualValues, errors, htmlEncoded);
|
||||
ValidateSkuTelemetry(actualValues, errors);
|
||||
break;
|
||||
case OpenIdConnectParameterNames.VersionTelemetry:
|
||||
ValidateVersionTelemetry(actualValues, errors, htmlEncoded);
|
||||
|
|
@ -258,14 +258,13 @@ namespace Microsoft.AspNetCore.Authentication.Test.OpenIdConnect
|
|||
private void ValidateState(IDictionary<string, string> actualParams, ICollection<string> errors, bool htmlEncoded) =>
|
||||
ValidateParameter(OpenIdConnectParameterNames.State, ExpectedState, actualParams, errors, htmlEncoded);
|
||||
|
||||
private void ValidateSkuTelemetry(IDictionary<string, string> actualParams, ICollection<string> errors, bool htmlEncoded) =>
|
||||
#if NETCOREAPP2_2
|
||||
ValidateParameter(OpenIdConnectParameterNames.SkuTelemetry, "ID_NETSTANDARD1_4", actualParams, errors, htmlEncoded);
|
||||
#elif NET461
|
||||
ValidateParameter(OpenIdConnectParameterNames.SkuTelemetry, "ID_NET451", actualParams, errors, htmlEncoded);
|
||||
#else
|
||||
#error Invalid target framework.
|
||||
#endif
|
||||
private static void ValidateSkuTelemetry(IDictionary<string, string> actualParams, ICollection<string> errors)
|
||||
{
|
||||
if (!actualParams.ContainsKey(OpenIdConnectParameterNames.SkuTelemetry))
|
||||
{
|
||||
errors.Add($"Parameter {OpenIdConnectParameterNames.SkuTelemetry} is missing");
|
||||
}
|
||||
}
|
||||
|
||||
private void ValidateVersionTelemetry(IDictionary<string, string> actualParams, ICollection<string> errors, bool htmlEncoded) =>
|
||||
ValidateParameter(OpenIdConnectParameterNames.VersionTelemetry, typeof(OpenIdConnectMessage).GetTypeInfo().Assembly.GetName().Version.ToString(), actualParams, errors, htmlEncoded);
|
||||
|
|
|
|||
Loading…
Reference in New Issue