[tarantool-patches] Re: [PATCH 1/3] Add test on error during reconfigure

  • From: Alex Khatskevich <avkhatskevich@xxxxxxxxxxxxx>
  • To: Vladislav Shpilevoy <v.shpilevoy@xxxxxxxxxxxxx>, tarantool-patches@xxxxxxxxxxxxx
  • Date: Thu, 19 Jul 2018 23:33:20 +0300


  +-- Check that data has at least all fields as an ethalon.

1. Typo here and in other places: ethalon -> etalon.
Fixed

2. Please, describe the function in doxygen style since
it is not a trival one-liner.
Added, however, the comment is bigger than the function...
+...
+err:match('Error injection:.*')

3. P l e a s e. Again. Do not use pcall + match. Use
util.check_error.
Ok. Sorry. It is a very old commit.

Other related posts: