Allow callback for "dotStyle" and "activeDotStyle" props - with tests

[yann-combarnous]

Same as PR #828, but with added tests.

• updated README
• updated typescript typedefs
• added callback option for the "dotStyle" and "activeDotStyle" props
• added tests

close #828

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
slider	✅ Ready (Inspect)	Visit Preview	Jun 15, 2022 at 7:11AM (UTC)

[yann-combarnous]

@yoyo837 , I re-opened this PR, with tests added, even though I cannot run them on my system.

[yann-combarnous]

@yoyo837 , used the same test syntax as in other tests now: expect(...).toHaveStyle. Hopefully tests should pass now :).

[yann-combarnous]

@yoyo837 , fixed the actual/expected values, please allow the tests to run. Thx.

[yoyo837]

CI failed.

[yann-combarnous]

CI failed.

Yep, but first test passed (container1), so now I fixed offset for second test (container2). Sorry for the iterations, a bit difficult to get right without being able to run the tests locally.

[codecov]

Codecov Report

Merging #830 (58102a1) into master (410717a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #830   +/-   ##
=======================================
  Coverage   99.81%   99.81%           
=======================================
  Files          13       13           
  Lines         533      535    +2     
  Branches      135      148   +13     
=======================================
+ Hits          532      534    +2     
  Misses          1        1           

Impacted Files	Coverage Δ
src/Slider.tsx	100.00% <ø> (ø)
src/Steps/index.tsx	100.00% <ø> (ø)
src/Steps/Dot.tsx	100.00% <100.00%> (ø)

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more