[java-metadata-aggregator] branch master updated: MDA-226 - Verify that setter is no longer cumulative

[Ian Young]

This is an automated email from the git hooks/post-receive script.

iay pushed a commit to branch master
in repository java-metadata-aggregator.

View the commit online:
http://git.shibboleth.net/view/?p=java-metadata-aggregator.git;a=commit;h=f9276e3335cd303c4b593d5469633dbeed8ab818

The following commit(s) were added to refs/heads/master by this push:
       new  f9276e3   MDA-226 - Verify that setter is no longer cumulative
f9276e3 is described below

commit f9276e3335cd303c4b593d5469633dbeed8ab818
Author: Ian Young <ian at iay.org.uk>
AuthorDate: Fri Mar 13 14:00:44 2020 +0000

    MDA-226 - Verify that setter is no longer cumulative
    
    https://issues.shibboleth.net/jira/browse/MDA-226
---
 .../metadata/pipeline/ItemIdTransformStageTest.java   | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemIdTransformStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemIdTransformStageTest.java
index 7cacdeb..317df20 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemIdTransformStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemIdTransformStageTest.java
@@ -82,4 +82,23 @@ public class ItemIdTransformStageTest {
         Assert.assertTrue(sha1Match);
         Assert.assertTrue(md5Match);
     }
+    
+    @Test
+    public void testMDA226() throws Exception {
+        final var stage = new ItemIdTransformStage();
+        
+        // Initial list should be empty
+        Assert.assertEquals(stage.getIdTransformers().size(), 0);
+        
+        // Put in a list of one thing.
+        stage.setIdTransformers(List.of(new MDQueryMD5ItemIdTransformer()));
+        Assert.assertEquals(stage.getIdTransformers().size(), 1);
+        
+        // Put in a list of one other thing
+        stage.setIdTransformers(List.of(new MDQuerySHA1ItemIdTransformer()));
+        // Should still be one thing, not accumulated to two
+        Assert.assertEquals(stage.getIdTransformers().size(), 1);
+        // Should also be the new thing, not the old thing.
+        Assert.assertTrue(List.copyOf(stage.getIdTransformers()).get(0) instanceof MDQuerySHA1ItemIdTransformer);
+    }
 }

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.