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

[ghiscoding]

• 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 #955 (4aa682d) into master (7712848) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #955   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           15        15           
  Lines          969       983   +14     
  Branches       322       331    +9     
=========================================
+ Hits           969       983   +14     

Impacted Files	Coverage Δ
...lia-slickgrid/custom-elements/aurelia-slickgrid.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.