effect icon indicating copy to clipboard operation
effect copied to clipboard
verified publisher icon Effect-TS

test: use Equal for succeed comparison

Open Hoishin opened this issue 3 weeks ago • 1 comments

Type

  • [ ] Refactor
  • [ ] Feature
  • [ ] Bug Fix
  • [ ] Optimization
  • [ ] Documentation Update
  • [x] Test fix

Description

Node.js 24 has a change to deepStrictEqual for how it handles circular references in objects. Because of that, a test for BigDecimal.normalized is not passing. It seems to make sense to make use of Equal.equals when it is available, then fall back to deepStrictEqual.

Related

  • Related Issue #
  • Closes #

Hoishin avatar Dec 09 '25 08:12 Hoishin

⚠️ No Changeset found

Latest commit: 0805ee282c22a84d3bddd2c7bd56268bfacb40a1

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Dec 09 '25 08:12 changeset-bot[bot]