ahochsteger / gmail-processor / 23770882320

93%

push

github

🧪 Add tests for essentialObject in ConfigUtils.ts (#645)

* test: add missing tests for essentialObject utility

This commit adds a new test file `src/lib/utils/ConfigUtils.spec.ts` to
improve the test coverage for the `essentialObject` function. The tests
cover various scenarios including:
- Property removal based on default values.
- Removal of null, empty arrays, and empty objects.
- Retention of required properties.
- Custom transformation logic using `propEssentialMap`.
- Nested object and array processing.

The `essentialObject` function is a core utility for object manipulation
and configuration processing, making these tests vital for the reliability
of the codebase.

Co-authored-by: ahochsteger <207989+ahochsteger@users.noreply.github.com>

* fix(utils): persist propEssentialMap transformations in essentialObject

This commit fixes a bug in `essentialObject` where transformations from
`propEssentialMap` were not being persisted back to the object when they
did not meet the removal criteria. This was particularly problematic for
array mappings that return new array references.

Additionally, this commit adds a comprehensive test suite for the
`essentialObject` utility in `src/lib/utils/ConfigUtils.spec.ts`
covering:
- Default value removal.
- Null/empty property removal.
- Required property retention.
- Complex transformations via `propEssentialMap`.
- Recursive and nested object/array processing.

Co-authored-by: ahochsteger <207989+ahochsteger@users.noreply.github.com>

---------

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: ahochsteger <207989+ahochsteger@users.noreply.github.com>

893 of 1029 branches covered (86.78%)

Branch coverage included in aggregate %.

4 of 4 new or added lines in 1 file covered. (100.0%)

9097 of 9786 relevant lines covered (92.96%)

141.79 hits per line