Skip to content

[MOO-2145]: fix Range Slider skipped test #494

Merged
SrirangK merged 1 commit intomainfrom
moo/MOO-2145-fix-range-slider-test
Apr 14, 2026
Merged

[MOO-2145]: fix Range Slider skipped test #494
SrirangK merged 1 commit intomainfrom
moo/MOO-2145-fix-range-slider-test

Conversation

@SrirangK
Copy link
Copy Markdown
Contributor

Checklist

  • Contains unit tests ✅
  • Contains breaking changes ✅
  • Compatible with: MX 8, 9, 10
  • Did you update version and changelog? ❌
  • PR title properly formatted ([XX-000]: description)? ✅
  • Works in Android ✅
  • Works in iOS ✅
  • Works in Tablet ✅

This PR contains

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Other (describe)

What is the purpose of this PR?

The skipped test was expecting incorrect behavior that never existed in the implementation. It assumed the component would default to step: 1 when given an invalid (negative) step size, but the component has always validated this and shown an error message instead.

Relevant changes

Removed the skipped test completely.

@SrirangK SrirangK requested a review from a team as a code owner April 13, 2026 09:51
@SrirangK SrirangK merged commit 4915387 into main Apr 14, 2026
35 of 37 checks passed
@SrirangK SrirangK deleted the moo/MOO-2145-fix-range-slider-test branch April 14, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants