Skip to content

Commit cc5b423

Browse files
garlickmergify[bot]
authored andcommitted
testsuite: add mincrit to topology unit test
Problem: there is no unit test coverage for the 'mincrit' topology. Add unit tests.
1 parent a0862db commit cc5b423

File tree

1 file changed

+18
-0
lines changed

1 file changed

+18
-0
lines changed

src/broker/test/topology.c

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -250,6 +250,24 @@ struct internal_ranks_test internal_ranks_tests[] = {
250250
{ 4, "binomial","0,2" },
251251
{ 8, "binomial","0,2,4,6" },
252252
{ 16, "binomial","0,2,4,6,8,10,12,14" },
253+
{ 4096, "mincrit:0", "0" },
254+
{ 1, "mincrit:2", "" },
255+
{ 2, "mincrit:2", "0" },
256+
{ 3, "mincrit:2", "0" },
257+
{ 4, "mincrit:2", "0-1" },
258+
{ 5, "mincrit:2", "0-2" },
259+
{ 6, "mincrit:2", "0-2" },
260+
{ 7, "mincrit:2", "0-2" },
261+
{ 1024, "mincrit:2", "0-2" },
262+
{ 512, "mincrit", "0" },
263+
// leader has 1024 children at size=1024+1=1025
264+
{ 1025, "mincrit", "0" },
265+
{ 1026, "mincrit", "0-2" },
266+
// 2 routers have 1024 children each at size=1024+1024+3=2051
267+
{ 2051, "mincrit", "0-2" },
268+
{ 2052, "mincrit", "0-3" },
269+
// a large el capitan run
270+
{ 10800, "mincrit", "0-11" },
253271
{ -1, NULL, NULL }
254272
};
255273

0 commit comments

Comments
 (0)