[Tarantool-patches] [PATCH 0/2] Make unit tests TAP-compliant

Sergey Bronnikov sergeyb at tarantool.org
Tue Jun 2 12:21:37 MSK 2020


Vladislav,

please review patches.

On 12:59 Thu 21 May , sergeyb at tarantool.org wrote:
> From: Sergey Bronnikov <sergeyb at tarantool.org>
> 
> 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@


More information about the Tarantool-patches mailing list