X-Git-Url: https://gerrit.fd.io/r/gitweb?p=csit.git;a=blobdiff_plain;f=tox.ini;h=2ad02e3fec595468e594ba509e55b074a78f7c3d;hp=0fd86453288c41aba9e91451efed1b629cfbb295;hb=71151aa84d7e9c2da3b22789535aefb1f381ff87;hpb=694b418272e9d7670ac69d477ed731bb7445b65a diff --git a/tox.ini b/tox.ini index 0fd8645328..2ad02e3fec 100644 --- a/tox.ini +++ b/tox.ini @@ -25,12 +25,13 @@ # will execute only checks defined in "pylint" tox environment. [tox] -envlist = linelength, autogen, pylint +envlist = new_line_length, line_length, autogen, pylint, tc_naming # The following is needed as tox requires setup.py by default. skipsdist = true +# Just a shorthand to avoid long lines. +checker_dir = ./resources/libraries/bash/entry/check -# TODO: Tox prints various warnings. -# Figure out what they are about and fix them. +# TODO: Tox prints various warnings. Figure them out and fix them. [testenv:pylint] deps = @@ -39,32 +40,39 @@ deps = whitelist_externals = /bin/bash setenv = PYTHONPATH = {toxinidir} # Run pylint, but hide its return value until python warnings are cleared. -commands = bash -c "bash resources/libraries/bash/entry/check/pylint.sh || true" +commands = bash -c "bash {[tox]checker_dir}/pylint.sh || true" # TODO: See FIXME in https://gerrit.fd.io/r/16423 -[testenv:linelength] +[testenv:line_length] whitelist_externals = /bin/bash # Fix all transgressions and remove the " || true" workaround. -commands = bash -c "bash resources/libraries/bash/entry/check/line.sh || true" +commands = bash -c "bash {[tox]checker_dir}/line.sh || true" -# It would be possible to add a check which fails -# if number of long lines increases (from parent commit value), -# and have it voting. -# But that would basically prevent us from adding new suites righ now. :( +[testenv:new_line_length] +whitelist_externals = /bin/bash +# Adding .svg, big .rst tables, .yaml and similar still requires long lines. +# As current implementation is too dumb to filter those out, +# the checker has to remain non-voting. +commands = bash -c "bash {[tox]checker_dir}/new_line.sh || true" [testenv:autogen] whitelist_externals = /bin/bash setenv = PYTHONPATH = {toxinidir} -commands = bash resources/libraries/bash/entry/check/autogen.sh +commands = bash {[tox]checker_dir}/autogen.sh + +[testenv:tc_naming] +whitelist_externals = /bin/bash +# Fix all TC namings and remove the " || true" workaround. +commands = bash -c "bash {[tox]checker_dir}/tc_naming.sh || true" # TODO: Migrate current docs check here. -# TODO: Create license checker. # TODO: Create voting "pylint violations should not increase" checker. -# TODO: Create voting "linelength violations should not increase" checker. +# TODO: Create voting checker to reject suites with Force Tags of other suite. +# TODO: Create voting checker against tags not in tag_documentation.rst file. # TODO: Create Robot suite Documentation checker (backslash if not next mark). # TODO: Create .yaml specific checker, so people can override long line check. # TODO: Create .rst specific checker, if there is one allowing # to override line check. # TODO: You get the idea, replace line check with something smarter -# wherever possible. \ No newline at end of file +# wherever possible.