fix(presets): dynamic columns should be auto-inserted with Grid Presets

[ghiscoding]

• fixes issue brought in comment under issue #1092
• there are 3 dynamically created columns (RowMove, RowSelection & RowDetail) and all 3 should be auto-inserted when enabled and used with columns presets
• the 2nd issue will be fixed in Slickgrid-Universal, the issue is when Row Detail & Row Selection are used together, an extra checkbox appears in the filter section of the RowDetail column when it shouldn't (see 2nd print screen)
  • this is fixed in Slickgrid-Universal PR 938

TODOs

• requires a new Slickgrid-Universal version
• add new unit tests coverage

Using all 3 extensions (RowMove, RowSelection & RowDetail) now works correctly when used with Grid Presets

the following presets now works

presets: {
  columns: [
    { columnId: 'title' },
    { columnId: 'duration' },
    { columnId: 'start' },
    { columnId: 'finish' },
    { columnId: 'percent2' },
    { columnId: 'effort-driven' }
  ]
}

is now correctly bringing back all 3 dynamically created columns as shown below

2nd Bug will be fixed in Slickgrid-Universal PR 938

[codecov]

Codecov Report

Merging #1116 (0285916) into master (d417e75) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master     #1116   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           11        11           
  Lines          911       925   +14     
  Branches       298       307    +9     
=========================================
+ Hits           911       925   +14     

Impacted Files	Coverage Δ
...lickgrid/components/angular-slickgrid.component.ts	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.