[Tarantool-patches] [PATCH] test: adjust the test name related to PAIRSMM flag
Igor Munkin
imun at tarantool.org
Wed Nov 27 00:10:47 MSK 2019
Vlad,
Thanks for your remark. On second thought I totally agree with you, so
I'll move this chunk one more time. Furthermore, I guess we ought to
move all results of this discussion to Lua style guide[1]:
On 26.11.19, Vladislav Shpilevoy wrote:
> Hi!
>
> On 26/11/2019 17:54, Alexander Turenko wrote:
> > On Tue, Nov 26, 2019 at 06:47:38PM +0300, Igor Munkin wrote:
> >> The prior name doesn't respect the naming convention used by test-run
> >> thus this chunk was ignored by our testing machinery.
> >>
> >> Part-of tarantool/tarantool#4560
> >>
> >> Signed-off-by: Igor Munkin <imun at tarantool.org>
> >> ---
> >>
> >> Branch: https://github.com/tarantool/luajit/tree/imun/test-name-fixup
> >> Issue: https://github.com/tarantool/tarantool/issues/4560
> >>
> >>
> >> test/{mmpairs_test.lua => mmpairs-tarantool-4560.test.lua} | 0
> >> 1 file changed, 0 insertions(+), 0 deletions(-)
> >> rename test/{mmpairs_test.lua => mmpairs-tarantool-4560.test.lua} (100%)
> >>
> >> diff --git a/test/mmpairs_test.lua b/test/mmpairs-tarantool-4560.test.lua
> >> similarity index 100%
> >> rename from test/mmpairs_test.lua
> >> rename to test/mmpairs-tarantool-4560.test.lua
>
> We have already agreed on, and use a different test name pattern:
>
> gh-####-description.test.lua
>
> Not
>
> <description>-####.test.lua.
[1]: https://www.tarantool.io/en/doc/2.2/dev_guide/lua_style_guide/#testing
--
Best regards,
IM
More information about the Tarantool-patches
mailing list