3 Commits (a999b4250e1f0e57e4c28ed184daead3d8fa4fc4)

Author	SHA1	Message	Date
Dmitry S	93a2d26182	(core) Fix a bug with editing numbers in some locales. ... Summary: Adds a new test for formatting and fix several related bugs it uncovered: 1. When editing a number with "," decimal separator, ensure it opens in the editor with "," (rather than ".", the original bug motivating this). 2. When guessing number format, set maxDecimals when it's needed (otherwise, e.g. "$1.234", or "4.5%" weren't guessed as numeric) 3. When guessing number format, ignore whitespace when deciding if guessed format is correct (otherwise percents can't be guessed in locales which add "%" with a non-breaking space before it). Test Plan: Added a test case that exercises all fixed behaviors. Reviewers: paulfitz Reviewed By: paulfitz Subscribers: paulfitz Differential Revision: https://phab.getgrist.com/D4177	8 months ago
Alex Hall	7f091cf057	(core) Fix number parsing/formatting tests after updates to node version and locales ... Summary: The en-ZA locale was recently updated: https://github.com/nodejs/node/issues/48120 This caused test failures which I was happy about, until I saw af-ZA is still using the old separators. Test Plan: Tests failed locally when using the latest node 18, then passed after this change. Reviewers: jarek Reviewed By: jarek Differential Revision: https://phab.getgrist.com/D4079	11 months ago
Jarosław Sadziński	a52d56f613	(core) Moving client and common tests to core ... Summary: - Moved /test/client and /test/common to core. - Moved two files (CircularArray and RecentItems) from app/common to core/app/common. - Moved resetOrg test to gen-server. - `testrun.sh` is now invoking common and client test from core. - Added missing packages to core's package.json (and revealed underscore as it is used in the main app). - Removed Coord.js as it is not used anywhere. Test Plan: Existing tests Reviewers: paulfitz Reviewed By: paulfitz Subscribers: paulfitz Differential Revision: https://phab.getgrist.com/D3590	2 years ago