Sergey,

Thanks for the review.

I comented your suggestion below and applied changes into branch

https://github.com/tarantool/tarantool/tree/opiskunov/gh-4856-integration-test-connectors

I don’t send new patch cause, as we agreed, need to use test-run for testing connectors.

Четверг, 21 мая 2020, 19:10 +03:00 от Sergey Bronnikov <sergeyb@tarantool.org>:

Hi, Oleg

Thanks for the patch! See my coments inline.

> # Release
>
> build_debian:
> - cmake . -DCMAKE_BUILD_TYPE=RelWithDebInfo -DENABLE_WERROR=ON ${CMAKE_EXTRA_PARAMS}
> + cmake . -DCMAKE_BUILD_TYPE=RelWithDebInfo -DENABLE_WERROR=ON -DENABLE_DIST=ON ${CMAKE_EXTRA_PARAMS}

Why do we need ENABLE_WERROR here?

I didn’t add ENABLE_WERROR here — it comes from release build. I have added DENABLE_DIST=ON, and we already discussed it in «vshard» thread.

> +# Integration testing
> +
> +test_connector_python_asynctnt: build_debian
> + make install

Let's create a separate target for tarantool installation and add it to
dependence. Same for targets below.