Hi, thank you for the review!

Hi!
Please, add changelogs according to `doc/changelogs.md`.

On 3/4/21 12:15 PM, Artem Starshov wrote:

Issue: https://github.com/tarantool/tarantool/issues/5040
Branch: https://github.com/tarantool/tarantool/tree/artemreyt/gh-5040-interactive-mode-bug
CI: https://github.com/tarantool/tarantool/runs/2029377964

Changes in v3:
   * core:
     - add case-insensetive value for boolean error injection;
     - add more tests for setting error injections;
     - few refactored things (80 line width, alignment and etc.);
     - add release_disable for test;
     - add reference to test-run issue in commit message;
     - now, it is able to set errinjs via env ONLY in Debug mode.
   * lua:
     - some refactoring (name of variables, 80 line-width..);
     - add release_disable for test;
     - add reference to test-run issue in commit message;
     - add "errinj" to the name of the test;
     - add comments to source code.

Artem Starshov (2):
   core: add setting error injections via env
   lua: fix tarantool -e always enters interactive mode

  src/lib/core/errinj.c                         |  30 +++++
  src/lib/core/errinj.h                         |   6 +
  src/lua/init.c                                |  27 +++-
  src/main.cc                                   |   3 +
  ...5040-inter-mode-isatty-via-errinj.test.lua | 126 ++++++++++++++++++
  test/app-tap/suite.ini                        |   1 +
  .../errinj_set_with_enviroment_vars.test.lua  |  29 ++++
  ...errinj_set_with_enviroment_vars_script.lua |  34 +++++
  test/box-tap/suite.ini                        |   1 +
  test/box/errinj.result                        |   1 +
  10 files changed, 254 insertions(+), 4 deletions(-)
  create mode 100755 test/app-tap/gh-5040-inter-mode-isatty-via-errinj.test.lua
  create mode 100755 test/box-tap/errinj_set_with_enviroment_vars.test.lua
  create mode 100644 test/box-tap/errinj_set_with_enviroment_vars_script.lua