i8beef / SAML2

Other
88 stars 43 forks source link

PingCompatibilityTest DecryptPingAssertion fails #50

Closed dougwebb closed 3 years ago

dougwebb commented 4 years ago

c:\tmp\pingassertion.txt is missing. Add it to the project and change the path?

i8beef commented 4 years ago

That whole test class is marked Explicit and not for general consumption either. As I recall its inherited from the original project as a method to validate Ping assertions for specific consumers. I agree it would have been better if it had a sanitized ping assertion to check instead of being dependent on someone placing their specific assertion somewhere it can read, but I don't actually have access to such a thing.

If you DO though, Im happy to take a pull request for it.

dougwebb commented 4 years ago

Unfortunately no, I don't. I was just trying to use the Run All Tests command, and noted the two failures I've reported.