Skip to content

Implement an exact match on the unit test suite#640

Closed
donporter wants to merge 3 commits intovmware:mainfrom
donporter:don/exact-match-unit-test
Closed

Implement an exact match on the unit test suite#640
donporter wants to merge 3 commits intovmware:mainfrom
donporter:don/exact-match-unit-test

Conversation

@donporter
Copy link
Contributor

Currently, the unit test tool for splinterdb implements a prefix match. The comments indicate it should be an exact match. For instance, if one wishes to run the btree suite in isolation, it also runs btree_stress.

From current main:

Running 2 CTests, suite name 'btree', test case 'all'.
TEST 1/7 btree:test_leaf_hdr_search [OK]
TEST 2/7 btree:test_index_hdr [OK]
TEST 3/7 btree:test_index_hdr_search [OK]
TEST 4/7 btree:test_leaf_split [OK]
TEST 5/7 btree:test_leaf_hdr [OK]
TEST 6/7 btree_stress:test_random_inserts_concurrent

With this change:

Running 1 CTests, suite name 'btree', test case 'all'.
TEST 1/5 btree:test_leaf_hdr_search [OK]
TEST 2/5 btree:test_index_hdr [OK]
TEST 3/5 btree:test_index_hdr_search [OK]
TEST 4/5 btree:test_leaf_split [OK]
TEST 5/5 btree:test_leaf_hdr [OK]
RESULTS: 5 tests (5 ok, 0 failed, 0 skipped) ran in 2 ms

@netlify
Copy link

netlify bot commented May 7, 2025

Deploy Preview for splinterdb canceled.

Name Link
🔨 Latest commit 1da9c14
🔍 Latest deploy log https://app.netlify.com/sites/splinterdb/deploys/681ba4362b7cf800085800e3

Signed-off-by: Don Porter <porter@cs.unc.edu>
@donporter donporter force-pushed the don/exact-match-unit-test branch from 0faeed2 to 1beb093 Compare May 7, 2025 16:57
@netlify
Copy link

netlify bot commented Jun 24, 2025

Deploy Preview for splinterdb canceled.

Name Link
🔨 Latest commit 14d0b2e
🔍 Latest deploy log https://app.netlify.com/projects/splinterdb/deploys/685b1265d68a65000870c8e0

@rtjohnso
Copy link
Collaborator

I tweaked your code and pushed it as a new PR #643 , which is now merged. So, closing this PR.

@rtjohnso rtjohnso closed this Jun 25, 2025
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

Successfully merging this pull request may close these issues.

2 participants