patch 8.2.1432: various inconsistencies in test files
Problem: Various inconsistencies in test files.
Solution: Add modelines where they were missing. Use Check commands instead
of silently skipping over tests. Adjust indents and comments.
(Ken Takata, closes #6695)
diff --git a/src/testdir/test_writefile.vim b/src/testdir/test_writefile.vim
index e580930..a6250c5 100644
--- a/src/testdir/test_writefile.vim
+++ b/src/testdir/test_writefile.vim
@@ -43,8 +43,9 @@
endfunc
func Test_writefile_fails_conversion()
- if !has('iconv') || has('sun')
- return
+ CheckFeature iconv
+ if has('sun')
+ throw 'Skipped: does not work on SunOS'
endif
" Without a backup file the write won't happen if there is a conversion
" error.
@@ -63,8 +64,9 @@
endfunc
func Test_writefile_fails_conversion2()
- if !has('iconv') || has('sun')
- return
+ CheckFeature iconv
+ if has('sun')
+ throw 'Skipped: does not work on SunOS'
endif
" With a backup file the write happens even if there is a conversion error,
" but then the backup file must remain