Re: [RESEND PATCH v3] list: test: Mending tests for list_cut_position()

[David Gow]

On Tue, 1 Oct 2024 at 01:06, I Hsin Cheng <richard120310@xxxxxxxxx> wrote:
>
> Mending test for list_cut_position*() for the missing check of integer
> "i" after the second loop. The variable should be checked for second
> time to make sure both lists after the cut operation are formed as
> expected.
>
> Signed-off-by: I Hsin Cheng <richard120310@xxxxxxxxx>
> ---

Thanks. I've tested this, and it works now and makes sense.

I would recommend updating the patch description slightly, as it's a
little bit confusing as-is (partly due to the early version having
already been applied and reverted).
Could we describe this (a) in the imperative mood, and (b) focus less
on this being a "fix" or "mend" and more on what the new check does.
For example, something like:
"Check the total number of elements in both resultant lists are
correct. Previously, only the first list's size was checked, so
additional elements in the second list would not have been caught."

Otherwise, this is fine.

Reviewed-by: David Gow <davidgow@xxxxxxxxxx>

Thanks for your patience,
-- David




>  lib/list-test.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/lib/list-test.c b/lib/list-test.c
> index 37cbc33e9fdb..b4b3810c71d0 100644
> --- a/lib/list-test.c
> +++ b/lib/list-test.c
> @@ -408,6 +408,8 @@ static void list_test_list_cut_position(struct kunit *test)
>                 KUNIT_EXPECT_PTR_EQ(test, cur, &entries[i]);
>                 i++;
>         }
> +
> +       KUNIT_EXPECT_EQ(test, i, 3);
>  }
>
>  static void list_test_list_cut_before(struct kunit *test)
> @@ -436,6 +438,8 @@ static void list_test_list_cut_before(struct kunit *test)
>                 KUNIT_EXPECT_PTR_EQ(test, cur, &entries[i]);
>                 i++;
>         }
> +
> +       KUNIT_EXPECT_EQ(test, i, 3);
>  }
>
>  static void list_test_list_splice(struct kunit *test)
> --
> 2.43.0
>
> --
> You received this message because you are subscribed to the Google Groups "KUnit Development" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to kunit-dev+unsubscribe@xxxxxxxxxxxxxxxx.
> To view this discussion on the web visit https://groups.google.com/d/msgid/kunit-dev/20240930170633.42475-1-richard120310%40gmail.com.

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature