From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp40.i.mail.ru (smtp40.i.mail.ru [94.100.177.100]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 5D0B2469710 for ; Tue, 2 Jun 2020 12:22:38 +0300 (MSK) Date: Tue, 2 Jun 2020 12:21:37 +0300 From: Sergey Bronnikov Message-ID: <20200602092137.GC10824@pony.bronevichok.ru> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Subject: Re: [Tarantool-patches] [PATCH 0/2] Make unit tests TAP-compliant List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: tarantool-patches@dev.tarantool.org, Vladislav Shpilevoy Vladislav, please review patches. On 12:59 Thu 21 May , sergeyb@tarantool.org wrote: > From: Sergey Bronnikov > > Tarantool unit tests uses TAP framework but some tests haven't produce > TAP-compliant output. This patch fixes such tests. > > TAP framework contains primitives to check correctness of test functions > behaviour and produced test output contains information about passed and failed > testcases. This output processed by test-run.py on test execution. In this scheme > separate artifacts like reference output files are useless and patch removes them. > > GH branch: https://github.com/tarantool/tarantool/tree/ligurio/gh-5000-using-tap-output-in-unit-tests > GH issue: https://github.com/tarantool/tarantool/issues/5000 > > Alexander Turenko (1): > test: update unit test lib to produce TAP-compliant output > > Sergey Bronnikov (1): > test: update unit tests to make them TAP-compliant > > test/unit/CMakeLists.txt | 49 +- > test/unit/base64.result | 178 -- > test/unit/bit.c | 75 +- > test/unit/bit.result | 895 ---------- > test/unit/bitset_basic.c | 38 +- > test/unit/bitset_basic.result | 12 - > test/unit/bitset_index.c | 42 +- > test/unit/bitset_index.result | 21 - > test/unit/bitset_iterator.c | 32 + > test/unit/bitset_iterator.result | 22 - > test/unit/bloom.cc | 20 +- > test/unit/bloom.result | 6 - > test/unit/bps_tree.cc | 80 +- > test/unit/bps_tree.result | 284 --- > test/unit/bps_tree_iterator.cc | 42 +- > test/unit/bps_tree_iterator.result | 21 - > test/unit/cbus.c | 20 +- > test/unit/cbus.result | 24 - > test/unit/cbus_hang.c | 2 +- > test/unit/cbus_hang.result | 4 - > test/unit/cbus_stress.c | 53 +- > test/unit/cbus_stress.result | 2 - > test/unit/checkpoint_schedule.result | 41 - > test/unit/coio.result | 12 - > test/unit/coll.cpp | 51 +- > test/unit/coll.result | 90 - > test/unit/column_mask.result | 36 - > test/unit/crypto.result | 132 -- > test/unit/csv.result | 80 +- > test/unit/decimal.result | 700 -------- > test/unit/fiber.cc | 21 +- > test/unit/fiber.result | 19 - > test/unit/fiber_channel.cc | 2 + > test/unit/fiber_channel.result | 23 - > test/unit/fiber_channel_stress.cc | 3 + > test/unit/fiber_channel_stress.result | 2 - > test/unit/fiber_cond.result | 8 - > test/unit/fiber_stack.c | 3 + > test/unit/fiber_stack.result | 11 - > test/unit/fiber_stress.cc | 4 + > test/unit/fiber_stress.result | 0 > test/unit/find_path.c | 3 + > test/unit/find_path.result | 2 - > test/unit/guard.cc | 3 +- > test/unit/guard.result | 1 - > test/unit/guava.c | 11 + > test/unit/guava.result | 6 - > test/unit/heap.c | 38 + > test/unit/heap.result | 24 - > test/unit/heap_iterator.c | 14 + > test/unit/heap_iterator.result | 8 - > test/unit/histogram.c | 11 + > test/unit/histogram.result | 6 - > test/unit/int96.cc | 51 +- > test/unit/int96.result | 2 - > test/unit/json.result | 208 --- > test/unit/light.cc | 16 +- > test/unit/light.result | 8 - > test/unit/luaL_iterator.result | 89 - > test/unit/luaT_tuple_new.result | 22 - > test/unit/merger.result | 71 - > test/unit/mhash.result | 18 - > test/unit/mhash_bytemap.result | 18 - > test/unit/mp_error.result | 44 - > test/unit/msgpack.result | 2282 ------------------------- > test/unit/popen.result | 30 - > test/unit/queue.c | 65 +- > test/unit/queue.result | 34 - > test/unit/ratelimit.result | 13 - > test/unit/reflection_c.result | 5 - > test/unit/reflection_cxx.result | 31 - > test/unit/rmean.cc | 19 +- > test/unit/rmean.result | 11 - > test/unit/rope.c | 10 +- > test/unit/rope.result | 299 ---- > test/unit/rope_avl.c | 5 + > test/unit/rope_avl.result | 123 -- > test/unit/rope_basic.c | 19 +- > test/unit/rope_basic.result | 139 -- > test/unit/rope_common.h | 11 +- > test/unit/rope_stress.c | 8 + > test/unit/rope_stress.result | 4 - > test/unit/rtree.cc | 20 +- > test/unit/rtree.result | 8 - > test/unit/rtree_iterator.cc | 24 +- > test/unit/rtree_iterator.result | 7 - > test/unit/rtree_multidim.cc | 30 +- > test/unit/rtree_multidim.result | 15 - > test/unit/say.result | 67 - > test/unit/scramble.c | 21 +- > test/unit/scramble.result | 4 - > test/unit/sio.result | 36 - > test/unit/stailq.result | 76 - > test/unit/swim.result | 234 --- > test/unit/swim_errinj.result | 18 - > test/unit/swim_proto.result | 40 - > test/unit/tuple_bigref.result | 17 - > test/unit/unit.c | 3 + > test/unit/unit.h | 13 +- > test/unit/uri.result | 1307 -------------- > test/unit/uuid.result | 9 - > test/unit/vclock.result | 149 -- > test/unit/vy_cache.result | 9 - > test/unit/vy_mem.c | 4 +- > test/unit/vy_mem.result | 17 - > test/unit/vy_point_lookup.result | 20 - > test/unit/vy_write_iterator.result | 61 - > test/unit/xrow.result | 58 - > 108 files changed, 688 insertions(+), 8521 deletions(-) > delete mode 100644 test/unit/base64.result > delete mode 100644 test/unit/bit.result > delete mode 100644 test/unit/bitset_basic.result > delete mode 100644 test/unit/bitset_index.result > delete mode 100644 test/unit/bitset_iterator.result > delete mode 100644 test/unit/bloom.result > delete mode 100644 test/unit/bps_tree.result > delete mode 100644 test/unit/bps_tree_iterator.result > delete mode 100644 test/unit/cbus.result > delete mode 100644 test/unit/cbus_hang.result > delete mode 100644 test/unit/cbus_stress.result > delete mode 100644 test/unit/checkpoint_schedule.result > delete mode 100644 test/unit/coio.result > delete mode 100644 test/unit/coll.result > delete mode 100644 test/unit/column_mask.result > delete mode 100644 test/unit/crypto.result > delete mode 100644 test/unit/decimal.result > delete mode 100644 test/unit/fiber.result > delete mode 100644 test/unit/fiber_channel.result > delete mode 100644 test/unit/fiber_channel_stress.result > delete mode 100644 test/unit/fiber_cond.result > delete mode 100644 test/unit/fiber_stack.result > delete mode 100644 test/unit/fiber_stress.result > delete mode 100644 test/unit/find_path.result > delete mode 100644 test/unit/guard.result > delete mode 100644 test/unit/guava.result > delete mode 100644 test/unit/heap.result > delete mode 100644 test/unit/heap_iterator.result > delete mode 100644 test/unit/histogram.result > delete mode 100644 test/unit/int96.result > delete mode 100644 test/unit/json.result > delete mode 100644 test/unit/light.result > delete mode 100644 test/unit/luaL_iterator.result > delete mode 100644 test/unit/luaT_tuple_new.result > delete mode 100644 test/unit/merger.result > delete mode 100644 test/unit/mhash.result > delete mode 100644 test/unit/mhash_bytemap.result > delete mode 100644 test/unit/mp_error.result > delete mode 100644 test/unit/msgpack.result > delete mode 100644 test/unit/popen.result > delete mode 100644 test/unit/queue.result > delete mode 100644 test/unit/ratelimit.result > delete mode 100644 test/unit/reflection_c.result > delete mode 100644 test/unit/reflection_cxx.result > delete mode 100644 test/unit/rmean.result > delete mode 100644 test/unit/rope.result > delete mode 100644 test/unit/rope_avl.result > delete mode 100644 test/unit/rope_basic.result > delete mode 100644 test/unit/rope_stress.result > delete mode 100644 test/unit/rtree.result > delete mode 100644 test/unit/rtree_iterator.result > delete mode 100644 test/unit/rtree_multidim.result > delete mode 100644 test/unit/say.result > delete mode 100644 test/unit/scramble.result > delete mode 100644 test/unit/sio.result > delete mode 100644 test/unit/stailq.result > delete mode 100644 test/unit/swim.result > delete mode 100644 test/unit/swim_errinj.result > delete mode 100644 test/unit/swim_proto.result > delete mode 100644 test/unit/tuple_bigref.result > delete mode 100644 test/unit/uri.result > delete mode 100644 test/unit/uuid.result > delete mode 100644 test/unit/vclock.result > delete mode 100644 test/unit/vy_cache.result > delete mode 100644 test/unit/vy_mem.result > delete mode 100644 test/unit/vy_point_lookup.result > delete mode 100644 test/unit/vy_write_iterator.result > delete mode 100644 test/unit/xrow.result > > -- > 2.23.0 -- sergeyb@