v test-fmt: use .skip_files instead of filtering

pull/6615/head
Delyan Angelov 2020-10-14 23:58:31 +03:00
parent 5dd34c3a8a
commit e36f11750b
1 changed files with 32 additions and 34 deletions

View File

@ -5,37 +5,37 @@ import testing
const ( const (
known_failing_exceptions = [ known_failing_exceptions = [
'./examples/vweb/vweb_example.v', 'examples/vweb/vweb_example.v',
'./cmd/tools/gen_vc.v', 'cmd/tools/gen_vc.v',
'./cmd/tools/modules/vgit/vgit.v', // generics 'cmd/tools/modules/vgit/vgit.v', // generics
'./cmd/tools/preludes/live_main.v', 'cmd/tools/preludes/live_main.v',
'./cmd/tools/preludes/live_shared.v', 'cmd/tools/preludes/live_shared.v',
'./cmd/tools/preludes/tests_assertions.v', 'cmd/tools/preludes/tests_assertions.v',
'./cmd/tools/preludes/tests_with_stats.v', 'cmd/tools/preludes/tests_with_stats.v',
'./cmd/tools/performance_compare.v', // generics 'cmd/tools/performance_compare.v', // generics
'./cmd/tools/oldv.v', // generics 'cmd/tools/oldv.v', // generics
'./tutorials/code/blog/article.v', 'tutorials/code/blog/article.v',
'./tutorials/code/blog/blog.v', 'tutorials/code/blog/blog.v',
'./vlib/arrays/arrays.v', 'vlib/arrays/arrays.v',
'./vlib/arrays/arrays_test.v', 'vlib/arrays/arrays_test.v',
'./vlib/builtin/js/hashmap.v', 'vlib/builtin/js/hashmap.v',
'./vlib/v/tests/fn_variadic_test.v', 'vlib/v/tests/fn_variadic_test.v',
'./vlib/v/tests/generic_test.v', 'vlib/v/tests/generic_test.v',
'./vlib/crypto/aes/aes.v', 'vlib/crypto/aes/aes.v',
'./vlib/crypto/aes/aes_cbc.v', 'vlib/crypto/aes/aes_cbc.v',
'./vlib/crypto/aes/block_generic.v', 'vlib/crypto/aes/block_generic.v',
'./vlib/crypto/aes/const.v', 'vlib/crypto/aes/const.v',
'./vlib/crypto/aes/cypher_generic.v', 'vlib/crypto/aes/cypher_generic.v',
'./vlib/crypto/rc4/rc4.v', 'vlib/crypto/rc4/rc4.v',
'./vlib/eventbus/eventbus_test.v', 'vlib/eventbus/eventbus_test.v',
'./vlib/os/bare/bare_example_linux.v', 'vlib/os/bare/bare_example_linux.v',
'./vlib/szip/szip.v', 'vlib/szip/szip.v',
'./vlib/uiold/examples/users_gui/users.v', 'vlib/uiold/examples/users_gui/users.v',
'./vlib/vweb/assets/assets.v', 'vlib/vweb/assets/assets.v',
'./vlib/vweb/vweb.v', 'vlib/vweb/vweb.v',
'./vlib/v/gen/js/tests/life.v', 'vlib/v/gen/js/tests/life.v',
'./vlib/builtin/bare/linuxsys_bare.v', 'vlib/builtin/bare/linuxsys_bare.v',
'./vlib/os/os.v', 'vlib/os/os.v',
] ]
) )
@ -50,6 +50,7 @@ fn v_test_formatting(vargs string) {
testing.eheader('Run "v fmt" over all .v files') testing.eheader('Run "v fmt" over all .v files')
mut vfmt_test_session := testing.new_test_session('$vargs fmt -worker') mut vfmt_test_session := testing.new_test_session('$vargs fmt -worker')
vfmt_test_session.files << all_v_files vfmt_test_session.files << all_v_files
vfmt_test_session.skip_files << known_failing_exceptions
vfmt_test_session.test() vfmt_test_session.test()
eprintln(vfmt_test_session.benchmark.total_message('running vfmt over V files')) eprintln(vfmt_test_session.benchmark.total_message('running vfmt over V files'))
if vfmt_test_session.benchmark.nfail > 0 { if vfmt_test_session.benchmark.nfail > 0 {
@ -62,9 +63,6 @@ fn v_files() []string {
mut files_that_can_be_formatted := []string{} mut files_that_can_be_formatted := []string{}
all_test_files := os.walk_ext('.', '.v') all_test_files := os.walk_ext('.', '.v')
for tfile in all_test_files { for tfile in all_test_files {
if tfile in known_failing_exceptions {
continue
}
if tfile.starts_with('./vlib/v/cgen/tests') { if tfile.starts_with('./vlib/v/cgen/tests') {
continue continue
} }