Skip to content

Commit c60413e

Browse files
committed
Fix broken tests
- the new module path purpose was missing in some tests
1 parent 419c460 commit c60413e

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

Tests/HttpModule.Tests/CommandResultHttpTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ private byte[] DecryptCookieData(string data)
9393
{
9494
return MachineKey.Unprotect(
9595
HttpRequestData.GetBinaryData(data),
96-
"Sustainsys.Saml2");
96+
"Sustainsys.Saml2", "/saml2");
9797
}
9898

9999
[TestMethod]

Tests/HttpModule.Tests/HttpRequestBaseExtensionsTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public void HttpRequestBaseExtensions_ToHttpRequestData()
4141
var cookieValue = HttpRequestData.ConvertBinaryData(
4242
MachineKey.Protect(
4343
new StoredRequestState(null, new Uri("urn:someUri"), null, null).Serialize(),
44-
HttpRequestBaseExtensions.ProtectionPurpose));
44+
HttpRequestBaseExtensions.ProtectionPurpose, "/saml2"));
4545
request.Cookies.Returns(new HttpCookieCollection());
4646
request.Cookies.Add(new HttpCookie(StoredRequestState.CookieNameBase + "SomeState", cookieValue));
4747

Tests/Mvc.Tests/Saml2ControllerTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ public void Saml2Controller_Acs_Works()
151151
HttpRequestData.ConvertBinaryData(
152152
MachineKey.Protect(
153153
new StoredRequestState(null, null, new Saml2Id("InResponseToId"), null).Serialize(),
154-
HttpRequestBaseExtensions.ProtectionPurpose))));
154+
HttpRequestBaseExtensions.ProtectionPurpose, "/Saml2"))));
155155

156156
var httpContext = Substitute.For<HttpContextBase>();
157157
httpContext.Request.Returns(request);

Tests/Owin.Tests/Saml2AuthenticationMiddlewareTests.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -972,7 +972,7 @@ public async Task Saml2AuthenticationMiddleware_AcsRedirectsToAuthPropsReturnUri
972972

973973
var cookieData = HttpRequestData.ConvertBinaryData(
974974
CreateAppBuilder().CreateDataProtector(
975-
typeof(Saml2AuthenticationMiddleware).FullName)
975+
typeof(Saml2AuthenticationMiddleware).FullName, "/Saml2")
976976
.Protect(state.Serialize()));
977977

978978
context.Request.Headers["Cookie"] = $"{StoredRequestState.CookieNameBase}{relayState}={cookieData}";
@@ -1044,7 +1044,7 @@ public async Task Saml2AuthenticationMiddleware_AcsRedirectsToAuthProps_StoredRe
10441044

10451045
var cookieData = HttpRequestData.ConvertBinaryData(
10461046
CreateAppBuilder().CreateDataProtector(
1047-
typeof(Saml2AuthenticationMiddleware).FullName)
1047+
typeof(Saml2AuthenticationMiddleware).FullName, "/Saml2")
10481048
.Protect(state.Serialize()));
10491049

10501050
context.Request.Headers["Cookie"] = $"{StoredRequestState.CookieNameBase}{relayState}={cookieData}";
@@ -1120,7 +1120,7 @@ public async Task Saml2AuthenticationMiddleware_AcsWorks()
11201120

11211121
var cookieData = HttpRequestData.ConvertBinaryData(
11221122
CreateAppBuilder().CreateDataProtector(
1123-
typeof(Saml2AuthenticationMiddleware).FullName)
1123+
typeof(Saml2AuthenticationMiddleware).FullName, "/Saml2")
11241124
.Protect(state.Serialize()));
11251125

11261126
context.Request.Headers["Cookie"] = $"{StoredRequestState.CookieNameBase}{relayState}={cookieData}";

0 commit comments

Comments
 (0)