Hi Michal

On 3/7/22 16:48, Michal Simek wrote:
> There is really no value to check empty spaces. That's why use
> ut_assert_nextlinen() instead of ut_assert_nextline().
> This change ensures that PINMUX_SIZE can be increased without changing
> tests.
> 
> Signed-off-by: Michal Simek <michal.si...@xilinx.com>
> ---
> 
> https://lore.kernel.org/all/3a6be84c8354f38754a9838670cc0319e84f29e8.1645626183.git.michal.si...@xilinx.com/
> ---
>  test/cmd/pinmux.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/test/cmd/pinmux.c b/test/cmd/pinmux.c
> index 8ae807b53779..ba338b8dce85 100644
> --- a/test/cmd/pinmux.c
> +++ b/test/cmd/pinmux.c
> @@ -16,18 +16,18 @@ static int dm_test_cmd_pinmux_status_pinname(struct 
> unit_test_state *uts)
>       /* Test that 'pinmux status <pinname>' displays the selected pin. */
>       console_record_reset();
>       run_command("pinmux status a5", 0);
> -     ut_assert_nextline("a5        : gpio input .                            
> ");
> +     ut_assert_nextlinen("a5        : gpio input .");
>       ut_assert_console_end();
>  
>       console_record_reset();
>       run_command("pinmux status P7", 0);
> -     ut_assert_nextline("P7        : GPIO2 bias-pull-down input-enable.      
> ");
> +     ut_assert_nextlinen("P7        : GPIO2 bias-pull-down input-enable.");
>       ut_assert_console_end();
>  
>       console_record_reset();
>       run_command("pinmux status P9", 0);
> -     ut_assert_nextline("single-pinctrl pinctrl-single-no-width: missing 
> register width");
> -     ut_assert_nextline("P9 not found");
> +     ut_assert_nextlinen("single-pinctrl pinctrl-single-no-width: missing 
> register width");
> +     ut_assert_nextlinen("P9 not found");
>       ut_assert_console_end();
>  
>       return 0;

Reviewed-by: Patrice Chotard <patrice.chot...@foss.st.com>

Thanks
Patrice

Reply via email to