Merge pull request #490 from twitter/jnievelt/489

Test Semigroup#sumOption using Iterator instead of List
twitter · Sep 9, 2015 · b6c076f · b6c076f
2 parents 26f554e + 4c9d8ee
commit b6c076f
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/algebird-test/src/main/scala/com/twitter/algebird/BaseProperties.scala b/algebird-test/src/main/scala/com/twitter/algebird/BaseProperties.scala
@@ -55,7 +55,7 @@ object BaseProperties {
 
   def semigroupSumWorks[T: Semigroup: Arbitrary: Equiv] = 'semigroupSumWorks |: forAll { (in: List[T]) =>
     in.isEmpty || {
-      Equiv[T].equiv(Semigroup.sumOption(in).get, in.reduceLeft(Semigroup.plus(_, _)))
+      Equiv[T].equiv(Semigroup.sumOption(in.iterator).get, in.reduceLeft(Semigroup.plus(_, _)))
     }
   }