toFixed(): Add edge case tests (Jasmine & QUnit) - Githubissues

openexchangerates / accounting.js

A lightweight JavaScript library for number, money and currency formatting - fully localisable, zero dependencies.

http://openexchangerates.github.io/accounting.js

MIT License

4.95k stars 528 forks source link

toFixed(): Add edge case tests (Jasmine & QUnit) #168

Open kennyklee opened 7 years ago

kennyklee commented 7 years ago

Hi there @wjcrowcroft!

I created a few tests (both Jasmine & QUnit) for the recent toFixed() edge cases in the merged PR https://github.com/openexchangerates/accounting.js/pull/164 and the fixes that are included in the open PR https://github.com/openexchangerates/accounting.js/pull/165 from @gustavlrsn.

In addition, I created a test for open issue https://github.com/openexchangerates/accounting.js/issues/167. I have a feeling you will see a solution to this one in the near future. :)

Thanks for your contribution to the open source community!

Cheers! Kenny