Unit tests for result of ticket validation with subject

spring-security-kerberos-core/src/main/java/org/springframework/security/kerberos/authentication/KerberosTicketValidation.java

@@ -35,7 +35,7 @@ public KerberosTicketValidation(String username, String servicePrincipal, byte[]
 	}
 
 	public KerberosTicketValidation(String username, Subject subject, byte[] responseToken, GSSContext gssContext) {
-		this(username, subject, responseToken, gssContext, null)
+		this(username, subject, responseToken, gssContext, null);
 	}
 
 	public KerberosTicketValidation(String username, Subject subject, byte[] responseToken, GSSContext gssContext, GSSCredential delegationCredential) {

spring-security-kerberos-core/src/main/java/org/springframework/security/kerberos/authentication/sun/SunJaasKerberosTicketValidator.java

@@ -295,7 +295,7 @@ public LoginConfig(String keyTabLocation, String servicePrincipalName, String re
             this.realmName = realmName;
             this.multiTier = multiTier;
             this.debug = debug;
-            this.refreshKrb5Config = refreshKrb5Configx
+            this.refreshKrb5Config = refreshKrb5Config;
         }
 
         @Override

spring-security-kerberos-core/src/test/java/org/springframework/security/kerberos/authentication/KerberosTicketValidationTest.java

@@ -0,0 +1,52 @@
+package org.springframework.security.kerberos.authentication;
+
+import org.ietf.jgss.GSSContext;
+import org.ietf.jgss.GSSCredential;
+import org.junit.Test;
+
+import javax.security.auth.Subject;
+
+import static org.junit.Assert.*;
+import static org.mockito.Mockito.*;
+
+public class KerberosTicketValidationTest {
+
+    private String username = "username";
+    private Subject subject = new Subject();
+    private byte[] responseToken = "token".getBytes();
+    private GSSContext gssContext = mock(GSSContext.class);
+    private GSSCredential delegationCredential = mock(GSSCredential.class);
+
+    @Test
+    public void createResultOfTicketValidationWithSubject() {
+
+        KerberosTicketValidation ticketValidation = new KerberosTicketValidation(
+                username,
+                subject,
+                responseToken,
+                gssContext);
+
+        assertEquals(username, ticketValidation.username());
+        assertEquals(responseToken, ticketValidation.responseToken());
+        assertEquals(gssContext, ticketValidation.getGssContext());
+
+        assertNull("With no credential delegation", ticketValidation.getDelegationCredential());
+    }
+
+    @Test
+    public void createResultOfTicketValidationWithSubjectAndDelegation() {
+
+        KerberosTicketValidation ticketValidation = new KerberosTicketValidation(
+                username,
+                subject,
+                responseToken,
+                gssContext,
+                delegationCredential);
+
+        assertEquals(username, ticketValidation.username());
+        assertEquals(responseToken, ticketValidation.responseToken());
+        assertEquals(gssContext, ticketValidation.getGssContext());
+
+        assertEquals("With no credential delegation", delegationCredential, ticketValidation.getDelegationCredential());
+    }
+}