SHIBUI-1807

Added missing index field to the representation
TIER · Jun 10, 2021 · f942bea · f942bea
1 parent 4ab559d
commit f942bea
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 27 deletions.
diff --git a/...net2/tier/shibboleth/admin/ui/domain/frontend/AssertionConsumerServiceRepresentation.java b/...net2/tier/shibboleth/admin/ui/domain/frontend/AssertionConsumerServiceRepresentation.java
@@ -2,37 +2,19 @@
 
 import java.io.Serializable;
 
-public class AssertionConsumerServiceRepresentation implements Serializable {
+import lombok.Getter;
+import lombok.Setter;
 
+@Getter
+@Setter
+public class AssertionConsumerServiceRepresentation implements Serializable {
     private static final long serialVersionUID = 7610150456756113460L;
 
     private String locationUrl;
 
     private String binding;
 
     private boolean makeDefault;
-
-    public String getLocationUrl() {
-        return locationUrl;
-    }
-
-    public void setLocationUrl(String locationUrl) {
-        this.locationUrl = locationUrl;
-    }
-
-    public String getBinding() {
-        return binding;
-    }
-
-    public void setBinding(String binding) {
-        this.binding = binding;
-    }
-
-    public boolean isMakeDefault() {
-        return makeDefault;
-    }
-
-    public void setMakeDefault(boolean makeDefault) {
-        this.makeDefault = makeDefault;
-    }
+
+    private Integer index;
 }
diff --git a/...n/java/edu/internet2/tier/shibboleth/admin/ui/service/JPAEntityDescriptorServiceImpl.java b/...n/java/edu/internet2/tier/shibboleth/admin/ui/service/JPAEntityDescriptorServiceImpl.java
@@ -168,6 +168,7 @@ void setupACSs(EntityDescriptor ed, EntityDescriptorRepresentation representatio
                 }
                 assertionConsumerService.setBinding(acsRepresentation.getBinding());
                 assertionConsumerService.setLocation(acsRepresentation.getLocationUrl());
+                assertionConsumerService.setIndex(acsRepresentation.getIndex());
             }
         } else {
             ed.getOptionalSPSSODescriptor().ifPresent(spssoDescriptor -> spssoDescriptor.getAssertionConsumerServices().clear());

diff --git a/...edu/internet2/tier/shibboleth/admin/ui/service/JPAEntityDescriptorServiceImplTests.groovy b/...edu/internet2/tier/shibboleth/admin/ui/service/JPAEntityDescriptorServiceImplTests.groovy
@@ -528,8 +528,8 @@ class JPAEntityDescriptorServiceImplTests extends Specification {
                      xmlns:saml="urn:oasis:names:tc:SAML:2.0:assertion"
                      entityID="http://test.example.org/test1">
                      <md:SPSSODescriptor>
-                        <md:AssertionConsumerService Binding="urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST" Location="https://test.example.org/SAML/POST"/>
-                        <md:AssertionConsumerService Binding="urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect" Location="https://test.example.org/SAML/GET" isDefault="true"/>
+                        <md:AssertionConsumerService Binding="urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST" Location="https://test.example.org/SAML/POST" index="2"/>
+                        <md:AssertionConsumerService Binding="urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect" Location="https://test.example.org/SAML/GET" index="1" isDefault="true"/>
                      </md:SPSSODescriptor>
 </md:EntityDescriptor>
 '''
@@ -539,12 +539,14 @@ class JPAEntityDescriptorServiceImplTests extends Specification {
                     new AssertionConsumerServiceRepresentation().with {
                         it.binding = 'urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST'
                         it.locationUrl = 'https://test.example.org/SAML/POST'
+                        it.index = 2
                         it
                     },
                     new AssertionConsumerServiceRepresentation().with {
                         it.binding = 'urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect'
                         it.locationUrl = 'https://test.example.org/SAML/GET'
                         it.makeDefault = true
+                        it.index = 1
                         it
                     }
             ]