OpenVoiceOS / OpenVoiceOS/ovos-date-parser
Expand date-time test coverage in 'fa-ir/date_time_test.json'
Open
@JarbasAl is already working on this.
Since Nov 7, 2024.
fa
- Dominant language
- Python
- Stars
- 0
- Forks
- 8
- Avg merge
- 7h 9m
- Merged PRs (30d)
- 17
Description
As per the discussion in PR #1, we need to expand the date-time test coverage to include additional test cases.
Comment reference: https://github.com/OpenVoiceOS/ovos-date-parser/pull/1#discussion_r1831942272
Details:
The current test cases only cover a single afternoon time. We should add tests for:
- Midnight (00:00) and noon (12:00)
- Morning times (AM)
- Edge cases like 23:59:59
- Single-digit hours and minutes (e.g., 01:05, 5:08)
Requester: @JarbasAl
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.