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.

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

Dispatch in tests? #341

Open
kilohsakul opened this issue Sep 24, 2023 · 1 comment
Open

Dispatch in tests? #341

kilohsakul opened this issue Sep 24, 2023 · 1 comment

Comments

@kilohsakul
Copy link
Collaborator

I think testing algorithms using dispatch functions is annoying to debug, one needs to spend time figuring out what the was actually called.
Any thoughs?
It is true though that writing tests is faster.

@Adda0
Copy link
Collaborator

Adda0 commented Sep 25, 2023

I disagree. I see no problem with debugging dispatch functions. In tests, you should test both the dispatch function (general test on the corresponding operation) and each specific implementation (testing the correct implementation of the specific algorithm, perf-testing the specific algorithm, correct handling of inputs specific to that algorithm, ...).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants