Remove duplicate tests #578
Conversation
| it('should accepts Arabic numbers', () => { | ||
| expect(aNInput.getNumericString()).toEqual(''); | ||
| aNInput.set('١٠٢٣٤٥٦٧.٨٩'); | ||
| aNInput.set('۱٠۲۳۴۵۶۷.۸۹'); |
There was a problem hiding this comment.
Aren't you changing the Arabic numbers to the Persian ones here?
|
Thanks for the pull request @Apostrophii and pointing out inconsistencies in the unit tests with those three repeated test blocks. So instead of just removing those, I've modified and added the tests I should I wrote in the first place. Thank you again for your contribution and taking the time to look at the AutoNumeric source. I'll close this PR since most of the changes will be overwritten by the next commit, however about the change of the Arabic numbers to Persian ones, if you feel I missed something here, do not hesitate to reopen it. |
3 participants
No description provided.