On 6/8/24 10:20, David Gow wrote:
On Tue, 4 Jun 2024 at 20:32, Ivan Orlov ivan.orlov0322@gmail.com wrote:
Since assert_test covers the part of the KUnit core (the assertion formatting functions), I believe it would be better to have it merged into kunit-test (as it is done for other tests for the KUnit core).
Signed-off-by: Ivan Orlov ivan.orlov0322@gmail.com
Personally, I don't mind if the assert tests are in a separate file, but do think it's worth having them in the same module.
Do you think it'd be better to keep them as separate source files (but put them in the same module via the Makefile), or is having a single source file cleaner?
Either way, I think this should be merged with the previous patch (see my comments there.)
Hi David,
Thank you for the review and sorry for the late reply.
I believe having the assert test in the same file as other kunit core tests would be more consistent with the existing test structure (otherwise, the tests for other parts of the kunit core should be extracted to separate files as well).
What do you think is more appropriate?