11 Nov
2024
11 Nov
'24
3:27 p.m.
On Mon Nov 11 13:52:36 2024 +0000, Connor McAdams wrote:
I _think_ I've changed this to be more inline with what you've suggested, but I'll let you mark this as resolved to be sure. If you meant to suggest having `check_dds_header()` take an `expected` structure as an argument, that gets kind of ugly if we're using that same function for tests outside of `test_save_surface_to_dds()`.
Yeah, I was thinking about potentially passing the structure to `check_dds_header()`. This is certainly fine but I'm wondering what makes it ugly for other tests. Is it hard to make them table-based like `test_save_surface_to_dds()` (i.e. looping on test cases in a `tests[]` array of structures)?