Fix(Log): undefined for keeping tree structure

techor-dev · Feb 22, 2023 · c681465 · c681465
1 parent dd6eb13
commit c681465
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/packages/log/src/tree.ts b/packages/log/src/tree.ts
@@ -13,10 +13,7 @@ const tree = (object: JSON | object) => {
 }
 
 function removeEmpty(data) {
-    //transform properties into key-values pairs and filter all the empty-values
-    const entries = Object.entries(data).filter(([, value]) => value != null)
-    //map through all the remaining properties and check if the value is an object.
-    //if value is object, use recursion to remove empty properties
+    const entries = Object.entries(data).filter(([, value]) => value !== null)
     const clean = entries
         .map(([key, v]) => {
             const value = typeof v == 'object' ? removeEmpty(v) : v

diff --git a/packages/log/tests/tree.test.ts b/packages/log/tests/tree.test.ts
@@ -1,6 +1,6 @@
 import log from '@techor/log'
 
 it('prevent outputting items with empty value', () => {
-    expect(log.tree({ a: 1, b: null, c: { c1: null, c2: 1 }, d: { d1: null } }))
+    expect(log.tree({ a: 1, b: null, c: { c1: null, c2: 1 }, d: { d1: undefined } }))
         .not.toContain('c1')
 })