coheigea commented on a change in pull request #456: CXF-7867 - Allow the
AbstractSpnegoAuthSupplier loginConfig to be used
URL: https://github.com/apache/cxf/pull/456#discussion_r224366969
##########
File path:
services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/kerberos/KerberosDelegationTokenTest.java
##########
@@ -135,6 +138,48 @@ public void testKerberosTokenJAXRS() throws Exception {
org.junit.Assert.assertEquals(2 * numToDouble, resp);
}
+ @org.junit.Test
+ public void testKerberosTokenJAXRSCustomLogin() throws Exception {
Review comment:
This test class is not actually run (there is a junit Ignore specified on
the class). Could you instead move the test to the systests/kerberos directory?
https://github.com/apache/cxf/tree/master/systests/kerberos/src/test/java/org/apache/cxf/systest/kerberos/jaxrs/kerberos
The test should show the need for this PR by failing before the fix was
applied, and passing after.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
