fix(rating): fixed model change by clicking on disabled rating element

[Smoggy]

Before submitting a pull request, please make sure you have at least performed the following:

• read and followed the CONTRIBUTING.md guide.
• built and tested the changes locally.
• added/updated any applicable tests.
• added/updated any applicable API documentation.
• added/updated any applicable demos.

Fixes #3465

[codecov-io]

Codecov Report

Merging #3574 into master will increase coverage by 0.59%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3574      +/-   ##
==========================================
+ Coverage   91.12%   91.71%   +0.59%     
==========================================
  Files          95      100       +5     
  Lines        2793     2897     +104     
  Branches      516      535      +19     
==========================================
+ Hits         2545     2657     +112     
+ Misses        190      183       -7     
+ Partials       58       57       -1

Flag	Coverage Δ
#e2e	56.57% <0%> (+0.75%)	⬆️
#unit	88.49% <100%> (+0.5%)	⬆️

Impacted Files	Coverage Δ
src/rating/rating.ts	100% <100%> (ø)	⬆️
src/tooltip/tooltip.ts	98.43% <0%> (-0.03%)	⬇️
src/index.ts	100% <0%> (ø)	⬆️
src/tabset/tabset.module.ts	100% <0%> (ø)	⬆️
src/pagination/pagination.ts	100% <0%> (ø)	⬆️
src/progressbar/progressbar-config.ts	100% <0%> (ø)	⬆️
src/popover/popover.ts	100% <0%> (ø)	⬆️
src/modal/modal-backdrop.ts	100% <0%> (ø)	⬆️
src/accordion/accordion.ts	98.43% <0%> (ø)	⬆️
src/tabset/tabset.ts	100% <0%> (ø)	⬆️
... and 20 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f4d3848...0a808fb. Read the comment docs.

[peterblazejewicz]

@Smoggy
Is that to cover #3465? Thanks!

[Smoggy]

@peterblazejewicz Hello, yes

[maxokorokov]

Hey, @Smoggy,

thanks for the PR. While playing with the code from the stackblitz from the issue seems to fix the problem, tests are not working:

• the one you've added passes even without the fix
• second change in handleKeyDown is not covered

Cheers,
Max

[maxokorokov]

This check is not covered by tests:

[maxokorokov]

Hey, @Smoggy, thanks for fixing this.

LGTM, but could you please fix the test, as reporting is broken. You should always wrap async tests with async()

Will merge after you fix this.

[maxokorokov]

LGTM, thanks!