improvement(list): Increase branch coverage

Author: kutyel

__tests__/list.test.ts

@@ -13,6 +13,11 @@ interface IPerson {
   Age?: number
 }
 
+interface IDriver {
+  Id: number
+  Car: String
+}
+
 interface IPet {
   Name: string
   Age?: number
@@ -396,6 +401,24 @@ test('GroupBy', t => {
     ),
     result
   )
+  // example taken from https://stackoverflow.com/a/7325306/2834553
+  const drivers = new List<IDriver>([
+    { Id: 1, Car: 'Ferrari' },
+    { Id: 1, Car: 'BMW' },
+    { Id: 2, Car: 'Audi' }
+  ])
+  const result2 = {
+    '1': ['Ferrari', 'BMW'],
+    '2': ['Audi']
+  }
+  t.deepEqual(
+    drivers.GroupBy(
+      p => p.Id,
+      p => p.Car
+      // (key, g) => ({ [key]: g.ToList() })
+    ),
+    result2
+  )
 })
 
 test('GroupJoin', t => {
@@ -1052,6 +1075,13 @@ test('ToDictionary', t => {
     dictionary.Select(x => x.Value),
     people
   )
+  // example taken from https://stackoverflow.com/a/3611140/2834553
+  const dict = people.ToDictionary(
+    x => x,
+    x => x.Age
+  )
+  t.is(dict.Select(x => x.Value).Max(), 50)
+  t.is(dict.Select(x => x.Value).Min(), 15)
 })
 
 test('ToList', t => {

package-lock.json

@@ -8,7 +8,7 @@
   "types": "dist/src/index.d.ts",
   "scripts": {
     "build": "tsc",
-    "check-coverage": "nyc check-coverage --statements 100 --branches 95 --functions 98 --lines 100",
+    "check-coverage": "nyc check-coverage --statements 100 --branches 98 --functions 98 --lines 100",
     "commit": "git-cz",
     "cover": "nyc --require ts-node/register --reporter=lcov npm t",
     "docs": "typedoc --out ../docs/ src/index.ts -m commonjs -t ES6",
@@ -58,7 +58,7 @@
     "tslint-config-prettier": "^1.13.0",
     "tslint-config-standard": "^7.1.0",
     "typedoc": "^0.26.5",
-    "typescript": "^5.0.4"
+    "typescript": "^5.5.4"
   },
   "config": {
     "ghooks": {

package.json

@@ -220,17 +220,14 @@ class List<T> {
   ): { [key: string]: TResult[] } {
     const initialValue: { [key: string]: TResult[] } = {}
     return this.Aggregate((ac, v) => {
-      if (v !== undefined) {
-        const key = grouper(v)
-        const existingGroup = isObj(ac)
-          ? (ac as { [key: string]: TResult[] })[key]
-          : undefined
-        const mappedValue = mapper(v)
-        if (existingGroup) {
-          existingGroup.push(mappedValue)
-        } else {
-          ;(ac as { [key: string]: TResult[] })[key] = [mappedValue]
-        }
+      const key = grouper(v)
+      const existingGroup =
+        isObj(ac) && (ac as { [key: string]: TResult[] })[key]
+      const mappedValue = mapper(v)
+      if (!!existingGroup) {
+        existingGroup.push(mappedValue)
+      } else {
+        ;(ac as { [key: string]: TResult[] })[key] = [mappedValue]
       }
       return ac
     }, initialValue)
@@ -552,7 +549,7 @@ class List<T> {
       //   : v
       dicc.Add({
         Key: this.Select(key).ElementAt(i),
-        Value: value ? this.Select(value).ElementAt(i) : v
+        Value: !!value ? this.Select(value).ElementAt(i) : v
       })
       return dicc
     }, new List<{ Key: TKey; Value: T | TValue }>())