Page MenuHomeFreeBSD

Add simple regression tests for tree(3).
ClosedPublic

Authored by trasz on May 7 2019, 5:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
May 10 2024, 5:38 PM
Unknown Object (File)
May 4 2024, 8:00 PM
Unknown Object (File)
May 2 2024, 8:18 PM
Unknown Object (File)
May 2 2024, 8:17 PM
Unknown Object (File)
May 2 2024, 4:14 PM
Unknown Object (File)
May 2 2024, 11:43 AM
Unknown Object (File)
Apr 20 2024, 5:58 AM
Unknown Object (File)
Apr 15 2024, 4:48 AM
Subscribers

Details

Summary

Add simple regression tests for tree(3). Those are ATF-ified versions
of OpenBSD's regress/sys/sys/tree/.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 24137
Build 22990: arc lint + arc unit

Event Timeline

seemslegit

tests/sys/sys/splay_test.c
68–94

General comment (I know this is upstream code): testing against ITER/ITER-1 (because the splay is removed on 92) is ok, but it is a bit fragile in the event that more code is added in the middle of the two loops. It would be better to increment the maximum value, decrement when removing, then test against the new smaller maximum in the second loop.

71–74

That's an interesting way to handle collisions...

This revision is now accepted and ready to land.May 8 2019, 11:31 PM
This revision was automatically updated to reflect the committed changes.