Claudenw commented on code in PR #358:
URL: 
https://github.com/apache/commons-collections/pull/358#discussion_r1032797383


##########
src/test/java/org/apache/commons/collections4/bloomfilter/SimpleBloomFilterTest.java:
##########
@@ -24,4 +29,15 @@ public class SimpleBloomFilterTest extends 
AbstractBloomFilterTest<SimpleBloomFi
     protected SimpleBloomFilter createEmptyFilter(final Shape shape) {
         return new SimpleBloomFilter(shape);
     }
+
+    @Test
+    public void testMergeShortBitMapProducer() {
+        SimpleBloomFilter filter = createEmptyFilter(getTestShape());
+        // create a producer that returns too few values

Review Comment:
   Updated.  we are actually sending 1 long when 2 are expected.  Changed to be 
a  non-zero argument and test.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to