fix: pass id provider to keymapper also

[tepi]

Fixes vaadin/flow#19361

[mshabarov]

Looks good to me, however, I'd recommend to add an end-to-end test that asserts the MSCB marks all duplicates checked, when identity provider is defined.

That's the point of this fix - selection for duplicates. Before it, MSCB didn't tick duplicates.

[tepi]

Looks good to me, however, I'd recommend to add an end-to-end test that asserts the MSCB marks all duplicates checked, when identity provider is defined.

That's the point of this fix - selection for duplicates. Before it, MSCB didn't tick duplicates.

I can't test it with an IT via Testbench API since ComboBox does not differentiate on the amount of selected items if they are considered equal. Also a screenshot comparison test does not seem to work. So hopefully the unit test for checking that the id providers match is enough.

[mshabarov]

Ok, let's stick with only the unit test then.

Let's Design System team decide if it can be merge and make another review, if needed.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud