diff options
author | Andrew Arnott <andrewarnott@gmail.com> | 2009-07-02 23:36:55 -0700 |
---|---|---|
committer | Andrew Arnott <andrewarnott@gmail.com> | 2009-07-02 23:36:55 -0700 |
commit | 733fd208b1006b7a974519f46c766efab39acc5c (patch) | |
tree | 354f60de68a07f2d9c4f59eaf4163e73d119a581 | |
parent | a2ade46df119aceeabebe99a56cc83452aaa0a55 (diff) | |
download | DotNetOpenAuth-733fd208b1006b7a974519f46c766efab39acc5c.zip DotNetOpenAuth-733fd208b1006b7a974519f46c766efab39acc5c.tar.gz DotNetOpenAuth-733fd208b1006b7a974519f46c766efab39acc5c.tar.bz2 |
Activated a CC that isn't causing a problem any more.
-rw-r--r-- | src/DotNetOpenAuth/OpenId/ProviderEndpointDescription.cs | 2 | ||||
-rw-r--r-- | src/DotNetOpenAuth/OpenId/RelyingParty/IProviderEndpoint.cs | 114 |
2 files changed, 56 insertions, 60 deletions
diff --git a/src/DotNetOpenAuth/OpenId/ProviderEndpointDescription.cs b/src/DotNetOpenAuth/OpenId/ProviderEndpointDescription.cs index 5564084..0c23fdc 100644 --- a/src/DotNetOpenAuth/OpenId/ProviderEndpointDescription.cs +++ b/src/DotNetOpenAuth/OpenId/ProviderEndpointDescription.cs @@ -131,8 +131,6 @@ namespace DotNetOpenAuth.OpenId { /// the extension in the request and see if a response comes back for that extension. /// </remarks> public bool IsExtensionSupported(Type extensionType) { - ErrorUtilities.VerifyArgumentNotNull(extensionType, "extensionType"); - ErrorUtilities.VerifyArgument(typeof(IOpenIdMessageExtension).IsAssignableFrom(extensionType), OpenIdStrings.TypeMustImplementX, typeof(IOpenIdMessageExtension).FullName); var extension = (IOpenIdMessageExtension)Activator.CreateInstance(extensionType); return this.IsExtensionSupported(extension); } diff --git a/src/DotNetOpenAuth/OpenId/RelyingParty/IProviderEndpoint.cs b/src/DotNetOpenAuth/OpenId/RelyingParty/IProviderEndpoint.cs index b1ac3f6..7571442 100644 --- a/src/DotNetOpenAuth/OpenId/RelyingParty/IProviderEndpoint.cs +++ b/src/DotNetOpenAuth/OpenId/RelyingParty/IProviderEndpoint.cs @@ -62,67 +62,65 @@ namespace DotNetOpenAuth.OpenId.RelyingParty { bool IsExtensionSupported(Type extensionType); } - // For some odd reason, having this next class causes our test project to fail to build with this error: - // C:\Program Files\MSBuild\Microsoft\VisualStudio\v9.0\TeamTest\Microsoft.TeamTest.targets(14,5): error : Method 'DotNetOpenAuth.OpenId.RelyingParty.IProviderEndpoint.IsExtensionSupported' does not have a method body. - /////// <summary> - /////// Code contract for the <see cref="IProviderEndpoint"/> type. - /////// </summary> - ////[ContractClassFor(typeof(IProviderEndpoint))] - ////internal abstract class IProviderEndpointContract : IProviderEndpoint { - //// #region IProviderEndpoint Members + /// <summary> + /// Code contract for the <see cref="IProviderEndpoint"/> type. + /// </summary> + [ContractClassFor(typeof(IProviderEndpoint))] + internal abstract class IProviderEndpointContract : IProviderEndpoint { + #region IProviderEndpoint Members - //// /// <summary> - //// /// Gets the detected version of OpenID implemented by the Provider. - //// /// </summary> - //// Version IProviderEndpoint.Version { - //// get { throw new NotImplementedException(); } - //// } + /// <summary> + /// Gets the detected version of OpenID implemented by the Provider. + /// </summary> + Version IProviderEndpoint.Version { + get { throw new NotImplementedException(); } + } - //// /// <summary> - //// /// Gets the URL that the OpenID Provider receives authentication requests at. - //// /// </summary> - //// Uri IProviderEndpoint.Uri { - //// get { throw new NotImplementedException(); } - //// } + /// <summary> + /// Gets the URL that the OpenID Provider receives authentication requests at. + /// </summary> + Uri IProviderEndpoint.Uri { + get { throw new NotImplementedException(); } + } - //// /// <summary> - //// /// Checks whether the OpenId Identifier claims support for a given extension. - //// /// </summary> - //// /// <typeparam name="T">The extension whose support is being queried.</typeparam> - //// /// <returns> - //// /// True if support for the extension is advertised. False otherwise. - //// /// </returns> - //// /// <remarks> - //// /// Note that a true or false return value is no guarantee of a Provider's - //// /// support for or lack of support for an extension. The return value is - //// /// determined by how the authenticating user filled out his/her XRDS document only. - //// /// The only way to be sure of support for a given extension is to include - //// /// the extension in the request and see if a response comes back for that extension. - //// /// </remarks> - //// bool IProviderEndpoint.IsExtensionSupported<T>() { - //// throw new NotImplementedException(); - //// } + /// <summary> + /// Checks whether the OpenId Identifier claims support for a given extension. + /// </summary> + /// <typeparam name="T">The extension whose support is being queried.</typeparam> + /// <returns> + /// True if support for the extension is advertised. False otherwise. + /// </returns> + /// <remarks> + /// Note that a true or false return value is no guarantee of a Provider's + /// support for or lack of support for an extension. The return value is + /// determined by how the authenticating user filled out his/her XRDS document only. + /// The only way to be sure of support for a given extension is to include + /// the extension in the request and see if a response comes back for that extension. + /// </remarks> + bool IProviderEndpoint.IsExtensionSupported<T>() { + throw new NotImplementedException(); + } - //// /// <summary> - //// /// Checks whether the OpenId Identifier claims support for a given extension. - //// /// </summary> - //// /// <param name="extensionType">The extension whose support is being queried.</param> - //// /// <returns> - //// /// True if support for the extension is advertised. False otherwise. - //// /// </returns> - //// /// <remarks> - //// /// Note that a true or false return value is no guarantee of a Provider's - //// /// support for or lack of support for an extension. The return value is - //// /// determined by how the authenticating user filled out his/her XRDS document only. - //// /// The only way to be sure of support for a given extension is to include - //// /// the extension in the request and see if a response comes back for that extension. - //// /// </remarks> - //// bool IProviderEndpoint.IsExtensionSupported(Type extensionType) { - //// Contract.Requires<ArgumentNullException>(extensionType != null); - //// Contract.Requires<ArgumentException>(typeof(IOpenIdMessageExtension).IsAssignableFrom(extensionType), string.Format(CultureInfo.CurrentCulture, OpenIdStrings.TypeMustImplementX, typeof(IOpenIdMessageExtension).FullName)); - //// throw new NotImplementedException(); - //// } + /// <summary> + /// Checks whether the OpenId Identifier claims support for a given extension. + /// </summary> + /// <param name="extensionType">The extension whose support is being queried.</param> + /// <returns> + /// True if support for the extension is advertised. False otherwise. + /// </returns> + /// <remarks> + /// Note that a true or false return value is no guarantee of a Provider's + /// support for or lack of support for an extension. The return value is + /// determined by how the authenticating user filled out his/her XRDS document only. + /// The only way to be sure of support for a given extension is to include + /// the extension in the request and see if a response comes back for that extension. + /// </remarks> + bool IProviderEndpoint.IsExtensionSupported(Type extensionType) { + Contract.Requires<ArgumentNullException>(extensionType != null); + Contract.Requires<ArgumentException>(typeof(IOpenIdMessageExtension).IsAssignableFrom(extensionType), string.Format(CultureInfo.CurrentCulture, OpenIdStrings.TypeMustImplementX, typeof(IOpenIdMessageExtension).FullName)); + throw new NotImplementedException(); + } - //// #endregion - ////} + #endregion + } } |