Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test time funcs #39

Open
bbrk24 opened this issue Nov 22, 2023 · 1 comment
Open

Test time funcs #39

bbrk24 opened this issue Nov 22, 2023 · 1 comment
Labels
domain:tests Related to the automated tests type:investigate This issue needs more research

Comments

@bbrk24
Copy link
Owner

bbrk24 commented Nov 22, 2023

Right now there's one test for each of the C++ time funcs. They are arguably undertested, and the C time funcs are not tested at all.

How should the C versions even be tested, if they call time() and timespec_get()?
@bbrk24 bbrk24 added type:enhancement New feature or request type:investigate This issue needs more research domain:tests Related to the automated tests and removed type:enhancement New feature or request labels Nov 22, 2023
@bbrk24
Copy link
Owner Author

bbrk24 commented Dec 3, 2023

I'm going to say the C++ functions are good with the new "late_night_time" tests. The C functions still need testing, however.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
domain:tests Related to the automated tests type:investigate This issue needs more research
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bbrk24