]> sigrok.org Git - libsigrok.git/commitdiff
tests: adjust test coverage for power of two and input value 0
authorGerhard Sittig <redacted>
Thu, 21 Dec 2023 20:22:43 +0000 (21:22 +0100)
committerGerhard Sittig <redacted>
Thu, 21 Dec 2023 20:34:24 +0000 (21:34 +0100)
An input value 0 for the sr_next_power_of_two() routine is not invalid
any longer. Remove the test case that is expected to fail. Add a regular
case to cover that code path instead.

tests/strutil.c

index 3cf132f082ae984251b18759c332d925fa82574d..3e43dbdca2c2ba2a278c6e72b6306656da33be88 100644 (file)
@@ -578,6 +578,7 @@ static const struct power_case_t {
        size_t want_bits;
        size_t want_power;
 } power_cases[] = {
+       { 0, 1, 1, },
        { 1, 1, 2, },
        { 2, 2, 4, },
        { 3, 2, 4, },
@@ -597,9 +598,6 @@ START_TEST(test_calc_power_of_two)
        const struct power_case_t *tcase;
        int ret;
 
-       ret = sr_next_power_of_two(0, NULL, NULL);
-       fail_unless(ret != SR_OK, "invalid value, did not fail");
-
        for (case_idx = 0; case_idx < ARRAY_SIZE(power_cases); case_idx++) {
                tcase = &power_cases[case_idx];
                ret = sr_next_power_of_two(tcase->value, &bits, &power);