Improve accuracy of rounding with multipleOf

[seamusv]

Ensure the accuracy of the multipleOf contract.

[codecov-io]

Codecov Report

Merging #243 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##             master    #243   +/-   ##
========================================
  Coverage      70.8%   70.8%           
  Complexity      575     575           
========================================
  Files            68      68           
  Lines          2264    2264           
  Branches        491     491           
========================================
  Hits           1603    1603           
  Misses          459     459           
  Partials        202     202

Impacted Files	Coverage Δ	Complexity Δ
...java/com/networknt/schema/MultipleOfValidator.java	87.5% <100%> (ø)	5 <0> (ø)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d95b85b...0fac9c1. Read the comment docs.

[stevehu]

@seamusv Thanks a lot for your contribution. This makes perfect sense and I am a little surprised that the official test case doesn't cover this scenario. Since we constantly sync the official test cases to our codebase, I think the best option for us is to separate the new test cases to another JSON file. At the same time, let's open an issue to add these to the official test suite. What do you think?

https://github.com/json-schema-org/JSON-Schema-Test-Suite

[seamusv]

@stevehu I have moved the test data out of the official specifications and into a top-level JSON file. The test is only in the draft7 suite. I'll submit a PR to the official test suite repo.

[seamusv]

I have submitted the request at json-schema-org/JSON-Schema-Test-Suite#321

[stevehu]

@seamusv Thanks a lot for the effort. Whenever we have a separate file, it won't be overwritten.