FAB-16233 fail test with names instead of brutal panic
FAB-16233 fail test with names instead of brutal panic

It's better to call t.FailNow() or t.Fatal(), instead of panic()
in tests, so we could have more info, i.e. which exact test has failed.

This is to aid debugging flaky UT described in FAB-16233, and should be
a good practice overall.

Signed-off-by: Jay Guo <91d7eeb83d08be0dc56ea674009c4b53383e15b3@gmail.com>

manish-sethi/fabric-sidedb,jimthematrix/fabric,manish-sethi/fabric,hyperledger/fabric,manish-sethi/fabric-sidedb,stemlending/fabric,manish-sethi/fabric-sidedb,jimthematrix/fabric,stemlending/fabric,manish-sethi/fabric-sidedb,stemlending/fabric,manish-sethi/fabric,hyperledger/fabric,stemlending/fabric,manish-sethi/fabric-sidedb,manish-sethi/fabric-sidedb,manish-sethi/fabric-sidedb