search

GSMA-CPAS / BWRP-chaincode

Apache License 2.0
1 stars 0 forks source link

missing error check in chaincode tests #18

Closed sschulz-t closed 3 years ago

sschulz-t commented 3 years ago

While working on something different, i noticed that there are error checks missing in the test scripts. For example here: https://github.com/GSMA-CPAS/BWRP-chaincode/blob/3731102dc3b9a38b3af9420b4e3f7c147446e81c/hybrid/offchain_test.go#L366

Could you please add a check like require.NoError(t, err)

to those calls?

dhruvagarwal86 commented 3 years ago

@sschulz-t : I have added the missing error check and committed the code in "abacfeature-test" branch. Kindly check

sschulz-t commented 3 years ago

thanks! fixed in commit 9a708a0c and merged to master