tools: add a vet_known_failing_windows skip list to `v test-cleancode`
parent
dfa2d63616
commit
65d9c8fa6f
|
@ -5,52 +5,67 @@ import testing
|
||||||
import v.util
|
import v.util
|
||||||
import arrays
|
import arrays
|
||||||
|
|
||||||
const (
|
const vet_known_failing = [
|
||||||
vet_known_failing_exceptions = []string{}
|
'do_not_delete_this',
|
||||||
vet_folders = [
|
]
|
||||||
'vlib/sqlite',
|
|
||||||
'vlib/v',
|
|
||||||
'vlib/x/json2',
|
|
||||||
'vlib/x/ttf',
|
|
||||||
'cmd/v',
|
|
||||||
'cmd/tools',
|
|
||||||
'examples/2048',
|
|
||||||
'examples/tetris',
|
|
||||||
'examples/term.ui',
|
|
||||||
]
|
|
||||||
verify_known_failing_exceptions = [
|
|
||||||
// Handcrafted meaningful formatting of code parts (mostly arrays)
|
|
||||||
'examples/sokol/02_cubes_glsl/cube_glsl.v',
|
|
||||||
'examples/sokol/03_march_tracing_glsl/rt_glsl.v',
|
|
||||||
'examples/sokol/04_multi_shader_glsl/rt_glsl.v',
|
|
||||||
'examples/sokol/05_instancing_glsl/rt_glsl.v',
|
|
||||||
'examples/sokol/06_obj_viewer/show_obj.v',
|
|
||||||
'vlib/v/checker/tests/modules/deprecated_module/main.v' /* adds deprecated_module. module prefix to imports, even though the folder has v.mod */,
|
|
||||||
'vlib/gg/m4/graphic.v',
|
|
||||||
'vlib/gg/m4/m4_test.v',
|
|
||||||
'vlib/gg/m4/matrix.v',
|
|
||||||
'vlib/builtin/int_test.v' /* special number formatting that should be tested */,
|
|
||||||
// TODOs and unfixed vfmt bugs
|
|
||||||
'vlib/v/gen/js/tests/js.v', /* local `hello` fn, gets replaced with module `hello` aliased as `hl` */
|
|
||||||
]
|
|
||||||
vfmt_verify_list = [
|
|
||||||
'cmd/',
|
|
||||||
'examples/',
|
|
||||||
'tutorials/',
|
|
||||||
'vlib/',
|
|
||||||
]
|
|
||||||
vfmt_known_failing_exceptions = arrays.merge(verify_known_failing_exceptions, [
|
|
||||||
'vlib/regex/regex_test.v' /* contains meaningfull formatting of the test case data */,
|
|
||||||
'vlib/crypto/sha512/sha512block_generic.v' /* formatting of large constant arrays wraps to too many lines */,
|
|
||||||
'vlib/crypto/aes/const.v' /* formatting of large constant arrays wraps to too many lines */,
|
|
||||||
])
|
|
||||||
)
|
|
||||||
|
|
||||||
const (
|
const vet_known_failing_windows = [
|
||||||
vexe = os.getenv('VEXE')
|
'do_not_delete_this',
|
||||||
vroot = os.dir(vexe)
|
'vlib/v/gen/js/tests/testdata/byte_is_space.v',
|
||||||
is_fix = '-fix' in os.args
|
'vlib/v/gen/js/tests/testdata/compare_ints.v',
|
||||||
)
|
'vlib/v/gen/js/tests/testdata/hw.v',
|
||||||
|
'vlib/v/gen/js/tests/testdata/string_methods.v',
|
||||||
|
'vlib/v/tests/project_with_modules_having_submodules/bin/main.vsh',
|
||||||
|
'vlib/v/tests/valgrind/simple_interpolation_script_mode.v',
|
||||||
|
'vlib/v/tests/valgrind/simple_interpolation_script_mode_more_scopes.v',
|
||||||
|
]
|
||||||
|
|
||||||
|
const vet_folders = [
|
||||||
|
'vlib/sqlite',
|
||||||
|
'vlib/v',
|
||||||
|
'vlib/x/json2',
|
||||||
|
'vlib/x/ttf',
|
||||||
|
'cmd/v',
|
||||||
|
'cmd/tools',
|
||||||
|
'examples/2048',
|
||||||
|
'examples/tetris',
|
||||||
|
'examples/term.ui',
|
||||||
|
]
|
||||||
|
|
||||||
|
const verify_known_failing_exceptions = [
|
||||||
|
// Handcrafted meaningful formatting of code parts (mostly arrays)
|
||||||
|
'examples/sokol/02_cubes_glsl/cube_glsl.v',
|
||||||
|
'examples/sokol/03_march_tracing_glsl/rt_glsl.v',
|
||||||
|
'examples/sokol/04_multi_shader_glsl/rt_glsl.v',
|
||||||
|
'examples/sokol/05_instancing_glsl/rt_glsl.v',
|
||||||
|
'examples/sokol/06_obj_viewer/show_obj.v',
|
||||||
|
'vlib/v/checker/tests/modules/deprecated_module/main.v' /* adds deprecated_module. module prefix to imports, even though the folder has v.mod */,
|
||||||
|
'vlib/gg/m4/graphic.v',
|
||||||
|
'vlib/gg/m4/m4_test.v',
|
||||||
|
'vlib/gg/m4/matrix.v',
|
||||||
|
'vlib/builtin/int_test.v' /* special number formatting that should be tested */,
|
||||||
|
// TODOs and unfixed vfmt bugs
|
||||||
|
'vlib/v/gen/js/tests/js.v', /* local `hello` fn, gets replaced with module `hello` aliased as `hl` */
|
||||||
|
]
|
||||||
|
|
||||||
|
const vfmt_verify_list = [
|
||||||
|
'cmd/',
|
||||||
|
'examples/',
|
||||||
|
'tutorials/',
|
||||||
|
'vlib/',
|
||||||
|
]
|
||||||
|
|
||||||
|
const vfmt_known_failing_exceptions = arrays.merge(verify_known_failing_exceptions, [
|
||||||
|
'vlib/regex/regex_test.v' /* contains meaningfull formatting of the test case data */,
|
||||||
|
'vlib/crypto/sha512/sha512block_generic.v' /* formatting of large constant arrays wraps to too many lines */,
|
||||||
|
'vlib/crypto/aes/const.v' /* formatting of large constant arrays wraps to too many lines */,
|
||||||
|
])
|
||||||
|
|
||||||
|
const vexe = os.getenv('VEXE')
|
||||||
|
|
||||||
|
const vroot = os.dir(vexe)
|
||||||
|
|
||||||
|
const is_fix = '-fix' in os.args
|
||||||
|
|
||||||
fn main() {
|
fn main() {
|
||||||
args_string := os.args[1..].join(' ')
|
args_string := os.args[1..].join(' ')
|
||||||
|
@ -76,8 +91,12 @@ fn tsession(vargs string, tool_source string, tool_cmd string, tool_args string,
|
||||||
|
|
||||||
fn v_test_vetting(vargs string) {
|
fn v_test_vetting(vargs string) {
|
||||||
expanded_vet_list := util.find_all_v_files(vet_folders) or { return }
|
expanded_vet_list := util.find_all_v_files(vet_folders) or { return }
|
||||||
|
mut vet_known_exceptions := vet_known_failing.clone()
|
||||||
|
if os.user_os() == 'windows' {
|
||||||
|
vet_known_exceptions << vet_known_failing_windows
|
||||||
|
}
|
||||||
vet_session := tsession(vargs, 'vvet', '${os.quoted_path(vexe)} vet', 'vet', expanded_vet_list,
|
vet_session := tsession(vargs, 'vvet', '${os.quoted_path(vexe)} vet', 'vet', expanded_vet_list,
|
||||||
vet_known_failing_exceptions)
|
vet_known_exceptions)
|
||||||
//
|
//
|
||||||
fmt_cmd, fmt_args := if is_fix {
|
fmt_cmd, fmt_args := if is_fix {
|
||||||
'${os.quoted_path(vexe)} fmt -w', 'fmt -w'
|
'${os.quoted_path(vexe)} fmt -w', 'fmt -w'
|
||||||
|
|
Loading…
Reference in New Issue