feat(number): bigint multipleOf

[soc221b]

Adds the multipleOf option to faker.number.bigInt():

• Preview NumberModule

[netlify]

✅ Deploy Preview for fakerjs ready!

Name	Link
🔨 Latest commit	c563103
🔍 Latest deploy log	https://app.netlify.com/sites/fakerjs/deploys/67c31a35a13d740008460b8c
😎 Deploy Preview	https://deploy-preview-3402.fakerjs.dev
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.97%. Comparing base (f591459) to head (c563103).

Additional details and impacted files

@@           Coverage Diff           @@
##             next    #3402   +/-   ##
=======================================
  Coverage   99.97%   99.97%           
=======================================
  Files        2811     2811           
  Lines      216972   216982   +10     
  Branches      944      951    +7     
=======================================
+ Hits       216917   216929   +12     
+ Misses         55       53    -2     

Files with missing lines	Coverage Δ
src/modules/number/index.ts	100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

[ST-DDT]

Thanks for your contribution. ❤️
I'll look at it more closely soon.
Have a nice weekend

[ST-DDT]

I finally had some time to look into you PR and run some tests locally.
I found a few cases that behave differently from the existing methods:

faker.number.int({ 'min': 6, 'max': 9, 'multipleOf': 5 }) // Error: No suitable integer value between 6 and 9 found.
faker.number.bigInt({ 'min': 6, 'max': 9, 'multipleOf': 5 }) // 10

faker.number.int({ 'min': -9, 'max': 0, 'multipleOf': 5 }) // -5
faker.number.bigInt({ 'min': -9, 'max': 0, 'multipleOf': 5 }) // Error: Multiple of 5n should be less than or equal to max 0n.

Please fix these cases and add tests for them.

[ST-DDT]

Sorry, for the slow review. It looks good to me, just a few suggestions to make it easier to read.

[ST-DDT]

@faker-js/maintainers @faker-js/members
Not sure about this one. BigInt or bigint?

Suggested change

	`No suitable bigint value between ${min} and ${max} found.`
	`No suitable BigInt value between ${min} and ${max} found.`

If we change it here, we should also change it in the jsdocs above.

[soc221b]

Sorry, for the slow review. It looks good to me, just a few suggestions to make it easier to read.

Thank you for your review. I have made the changes you suggested.