Re: [PATCH blktests] Restore support for running tests without prior test results

[Omar Sandoval <osandov@osandov.com>]

On Tue, Jun 09, 2020 at 07:08:45AM -0700, Bart Van Assche wrote:
> On 2020-05-20 09:52, Bart Van Assche wrote:
> > This patch fixes the following runtime error:
> > 
> > ./check: line 245: LAST_TEST_RUN: unbound variable
> > 
> > Fixes: 203b5723a28e ("Show last run for skipped tests")
> > Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> > ---
> >  check | 8 +++++++-
> >  1 file changed, 7 insertions(+), 1 deletion(-)
> > 
> > diff --git a/check b/check
> > index 0a4e539a5cd9..5151d01995ac 100755
> > --- a/check
> > +++ b/check
> > @@ -240,9 +240,15 @@ _output_last_test_run() {
> >  }
> >  
> >  _output_test_run() {
> > +	local param_count
> >  	if [[ -t 1 ]]; then
> >  		# Move the cursor back up to the status.
> > -		tput cuu $((${#LAST_TEST_RUN[@]} + 1))
> > +		if [ -n "${LAST_TEST_RUN+set}" ]; then
> > +			param_count=${#LAST_TEST_RUN[@]}
> > +		else
> > +			param_count=0
> > +		fi
> > +		tput cuu $((param_count + 1))
> >  	fi
> >  
> >  	local status=${TEST_RUN["status"]}
> > 
> 
> Omar, ping?

Sorry Bart, I had a family emergency so I've been away for a few weeks.
This fix didn't work for me for the same reason that Li Zhijian reported
in https://github.com/osandov/blktests/pull/64. For now I'm going to
apply that fix instead. If you'd like blktests to work with set -u,
please do so by adding it to the main check script and modifying the
rest of the framework.