Merge pull request #80 from gerneio/patch-1

Aggregate Bug #79
ENikS · Jan 12, 2019 · a02192f · a02192f
2 parents bb1984a + 0369611
commit a02192f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/lib/linq.ts b/lib/linq.ts
@@ -190,7 +190,9 @@ class EnumerableImpl<T> implements Enumerable<T>, Iterable<T>, IEnumerable<T> {
         }
         let result: A = zero;
         for (let value of this) {
-            if (!result) result = Constant.getDefaultVal(typeof (value));
+            if ([null, undefined].indexOf(result) > -1 || (isNaN(result as any) && !result)) 
+                result = Constant.getDefaultVal(typeof (value));
+
             result = method(result, value);
         }
         return selector(result);

diff --git a/test/immediate.ts b/test/immediate.ts
@@ -33,7 +33,10 @@ describe('Immediate Execution -', function () {
     it('Aggregate() - Transform', function () {
         assert.equal(1814400, Linq.From(simpleArray).Aggregate(1, (a, b) => a * b, o => o / 2));
     });
-
+
+    it('Aggregate() - Default Value [String]', function () {
+        assert.equal("123", Linq.From([1, 2, 3]).Aggregate("", (c, n) => c.toString() + n.toString()));
+    });
 
 
     // All