-
-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Labels
Description
Description
Track the expansion of time format test coverage.
ovos_date_parser/res/en-us/date_time_test.json
🛠️ Refactor suggestion
Improve test data structure and coverage in test_nice_year section.
Several improvements are needed for better test data structure and coverage:
- Use proper JSON array format for datetime_param instead of comma-separated strings
- Use JSON boolean values instead of strings for the "bc" field
- Add missing test cases for edge scenarios
Apply these changes to improve the test data structure:
"test_nice_year": {
- "1": {"datetime_param": "1, 1, 31, 13, 22, 3", "bc": "True", "assertEqual": "one b.c." },
+ "1": {"datetime_param": [1, 1, 31, 13, 22, 3], "bc": true, "assertEqual": "one B.C." },
Consider adding these test cases:
{
"25": {"datetime_param": [0, 1, 31, 13, 22, 3], "bc": false, "assertEqual": "zero"},
"26": {"datetime_param": [9999, 1, 31, 13, 22, 3], "bc": false, "assertEqual": "nine thousand nine hundred ninety nine"},
"27": {"datetime_param": [10000, 1, 31, 13, 22, 3], "bc": false, "assertEqual": "ten thousand"}
}
Requester: @JarbasAl
References:
- PR: feat: nice_xx #1
- Comment: feat: nice_xx #1 (comment)