[LTP] [PATCH 2/2] net/sendfile01.sh: Rewrite into new API

[Alexey Kodanev]

On 16.04.2020 13:45, Petr Vorel wrote:
> * reduce sleep to 1s (from 10s)
> * continue testing on size failure
> 

Hi Petr,

> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
>  .../network/tcp_cmds/sendfile/sendfile01.sh   | 125 ++++++------------
>  1 file changed, 38 insertions(+), 87 deletions(-)
> 
> diff --git a/testcases/network/tcp_cmds/sendfile/sendfile01.sh b/testcases/network/tcp_cmds/sendfile/sendfile01.sh
> index 2cbde0ab7..4e4cc161b 100755
> --- a/testcases/network/tcp_cmds/sendfile/sendfile01.sh
> +++ b/testcases/network/tcp_cmds/sendfile/sendfile01.sh
> @@ -1,111 +1,62 @@
>  #!/bin/sh
> +# SPDX-License-Identifier: GPL-2.0-or-later
> +# Copyright (c) 2020 Petr Vorel <pvorel@suse.cz>
>  # Copyright (c) 2014 Oracle and/or its affiliates. All Rights Reserved.
> +# Copyright (c) Linux Test Project, 2001-2013
> +# Copyright (c) Manoj Iyer <manjo@mail.utexas.edu> 2003
> +# Copyright (c) Robbie Williamson <robbiew@us.ibm.com> 2002-2003
>  # Copyright (c) International Business Machines  Corp., 2000
> -#
> -# This program is free software; you can redistribute it and/or
> -# modify it under the terms of the GNU General Public License as
> -# published by the Free Software Foundation; either version 2 of
> -# the License, or (at your option) any later version.
> -#
> -# This program is distributed in the hope that it would be useful,
> -# but WITHOUT ANY WARRANTY; without even the implied warranty of
> -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> -# GNU General Public License for more details.
> -#
> -# You should have received a copy of the GNU General Public License
> -# along with this program; if not, write the Free Software Foundation,
> -# Inc.,  51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
> -#
> -#  PURPOSE: Copy files from server to client using the sendfile()
> -#           function.
> -#
> -#
> -#  SETUP: The home directory of root on the machine exported as "RHOST"
> -#         MUST have a ".rhosts" file with the hostname of the client
> -#         machine, where the test is executed.
> -#
> -#  HISTORY:
> -#    06/09/2003 Manoj Iyer manjo@mail.utexas.edu
> -#    - Modified to use LTP APIs, and added check to if commands used in test
> -#    exists.
> -#    03/01 Robbie Williamson (robbiew@us.ibm.com)
> -#      -Ported
> -#
> -#
> -#***********************************************************************
>  
> -TST_TOTAL=1
> -TCID="sendfile01"
> +TST_SETUP=do_setup
>  TST_CLEANUP=do_cleanup
> +TST_TESTFUNC=do_test
> +TST_NEEDS_TMPDIR=1
> +TST_NEEDS_CMDS="diff stat"
> +. tst_net.sh
>  
>  do_setup()
>  {
> -	TCdat=${TCdat:-$LTPROOT/testcases/bin/datafiles}
>  
> -	CLIENT="testsf_c${TST_IPV6}"
> -	SERVER="testsf_s${TST_IPV6}"
> +	tst_res TINFO "copy files from server to client using the sendfile() on IPv$TST_IPVER"
>  
> -	FILES=${FILES:-"ascii.sm ascii.med ascii.lg ascii.jmb"}
> +	client="testsf_c${TST_IPV6}"
> +	server="testsf_s${TST_IPV6}"
>  
> -	tst_require_cmds diff stat
> +	port=$(tst_rhost_run -s -c "tst_get_unused_port ipv$TST_IPVER stream")
> +	[ -z "$port" ] && tst_brk TBROK "failed to get unused port"
>  
> -	tst_tmpdir
> +	tst_rhost_run -s -b -c "$server $(tst_ipaddr rhost) $port"
> +	server_started=1
> +	tst_res TINFO "wait for the server to start"
> +	sleep 1

It would be better to replace 'sleep' with a wait-loop and check that the
port in the listen state there, e.g with ss -ltp | grep "$port.*testsf"

>  }
>  
>  do_test()
>  {
> -	tst_resm TINFO "Doing $0."
> -
> -	local ipv="ipv${TST_IPV6:-"4"}"
> -	local ipaddr=$(tst_ipaddr rhost)
> -	local port=$(tst_rhost_run -s -c "tst_get_unused_port $ipv stream")
> -	[ -z "$port" ] && tst_brkm TBROK "failed to get unused port"
> -
> -	tst_rhost_run -s -b -c "$SERVER $ipaddr $port"
> -	server_started=1
> -	sleep 10
> -
> -	for clnt_fname in $FILES; do
> -		serv_fname=${TCdat}/$clnt_fname
> -		local size=$(stat -c '%s' $serv_fname)
> -
> -		tst_resm TINFO \
> -			"$CLIENT ip '$ipaddr' port '$port' file '$clnt_fname'"
> -
> -		$CLIENT $ipaddr $port $clnt_fname $serv_fname $size >\
> -			/dev/null 2>&1
> -
> -		local ret=$?
> -		if [ $ret -ne 0 ]; then
> -			tst_resm TFAIL "$CLIENT returned error '$ret'"
> -			return;
> -		fi
> -
> -		diff $serv_fname $clnt_fname > /dev/null 2>&1
> -		local diff_res=$?
> -		if [ $diff_res -gt 1 ]; then
> -			tst_resm TFAIL "ERROR: Cannot compare files"
> -			return
> -		fi
> -
> -		if [ $diff_res -eq 1 ]; then
> -			tst_resm TFAIL "The file copied differs from the original"
> -			return
> +	local file lfile ret size
> +
> +	for file in $(ls $TST_NET_DATAROOT/ascii.*); do
> +		lfile="$(basename $file)"
> +		size=$(stat -c '%s' $file)
> +
> +		tst_res TINFO "test IP: $(tst_ipaddr rhost), port: $port, file: $lfile"
> +		ROD $client $(tst_ipaddr rhost) $port $lfile $file $size \> /dev/null
> +
> +		diff $file $lfile > /dev/null
> +		ret=$?
> +		if [ $ret -eq 0 ]; then
> +			tst_res TPASS "coped file is the same as the original"
> +		elif [ $ret -gt 1 ]; then
> +			tst_brk TFAIL "cannot compare files"
> +		else
> +			tst_res TFAIL "the copied file differs from the original"


EXPECT_PASS diff $lfile

>  		fi
>  	done
> -	tst_resm TPASS "test finished successfully"
>  }
>  
>  do_cleanup()
>  {
> -	[ -n "$server_started" ] && tst_rhost_run -c "pkill $SERVER"
> -	tst_rmdir
> +	[ -n "$server_started" ] && tst_rhost_run -c "pkill $server"
>  }
>  
> -TST_USE_LEGACY_API=1
> -. tst_net.sh
> -
> -do_setup
> -do_test
> -
> -tst_exit
> +tst_run
> 

The rest looks good!