isaacbrodsky / kepler.gl

67%

push

github

[chore] Add several vis state mergers combineConfigs and improve TS (#2634)

- Adds `combineConfigs` to some vis state saved configs that are non-array complex objects:
  1. `interactionConfig`
  2. `layerBlending`
  3. `overlayBlending`
  4. `editor`
  
  Each of these has inline comments (and unit tests) explaining the decisions made on how to combine values in different situations. For example, `interactionConfig.brush.size` gets aggregated into keeping the single largest value among all brush sizes.

- Improves and fixes some TS defs.

- Adds jest unit tests for newly added `combineConfigs`.

Signed-off-by: Ihor Dykhta <dikhta.igor@gmail.com>
Co-authored-by: Ilya Boyandin <iboyandin@foursquare.com>

5901 of 10163 branches covered (58.06%)

Branch coverage included in aggregate %.

42 of 42 new or added lines in 2 files covered. (100.0%)

12093 of 16544 relevant lines covered (73.1%)

89.91 hits per line