Skip to content

fix(Slider): precision problem#13720

Open
pany-ang wants to merge 1 commit into
youzan:mainfrom
pany-ang:fix-slider-precision
Open

fix(Slider): precision problem#13720
pany-ang wants to merge 1 commit into
youzan:mainfrom
pany-ang:fix-slider-precision

Conversation

@pany-ang

@pany-ang pany-ang commented Dec 18, 2025

Copy link
Copy Markdown
Contributor

修复 issue: #13718 提到的问题


该 PR 没有提供单测,因为不知为何原因,以下想新增的单测始终过不了

test('should have precision', async () => {
  const wrapper = mount(Slider, {
    props: {
      max: 1000000,
      step: 0.01,
      modelValue: 300000.03,
    },
  });

  await later();
  expect(wrapper.emitted('update:modelValue')![0]).toEqual([300000.03]);
});

显示 wrapper.emitted('update:modelValue') 始终为 undefined

@codecov-commenter

codecov-commenter commented Dec 18, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.96%. Comparing base (ec5b45b) to head (32399a3).
⚠️ Report is 255 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13720      +/-   ##
==========================================
+ Coverage   89.60%   89.96%   +0.35%     
==========================================
  Files         257      257              
  Lines        7013     7052      +39     
  Branches     1736     1748      +12     
==========================================
+ Hits         6284     6344      +60     
+ Misses        384      372      -12     
+ Partials      345      336       -9     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants