From 9dbf64d74081f8dd571cddb55b7884f5d9e5821a Mon Sep 17 00:00:00 2001 From: Timothy Sample Date: Sat, 10 Nov 2018 21:21:47 -0500 Subject: [PATCH] Enable 'word-split' specification test * tests/spec/Makefile.am (TESTS): Add 'word-split'. * tests/spec/oil.scm: Filter out four tests from it that we cannot support yet. --- tests/spec/Makefile.am | 3 ++- tests/spec/oil.scm | 15 ++++++++++++++- 2 files changed, 16 insertions(+), 2 deletions(-) diff --git a/tests/spec/Makefile.am b/tests/spec/Makefile.am index 2e628e1..21daa3a 100644 --- a/tests/spec/Makefile.am +++ b/tests/spec/Makefile.am @@ -18,7 +18,8 @@ TEST_EXTENSIONS = .sh SH_LOG_COMPILER = ./check-spec TESTS = \ - oil/spec/smoke.test.sh + oil/spec/smoke.test.sh \ + oil/spec/word-split.test.sh clean-local: rm -rf oil diff --git a/tests/spec/oil.scm b/tests/spec/oil.scm index b94983f..f4060c0 100644 --- a/tests/spec/oil.scm +++ b/tests/spec/oil.scm @@ -75,7 +75,20 @@ (else (unless ignore? (display line out)) (loop (read-line in 'concat) ignore?)))))))) - (tests-to-remove '())) + (tests-to-remove + '(("spec/word-split.test.sh" + (;; This test requires local variables, which is + ;; a Bash extension. + "IFS is scoped" + ;; We do not do tilde expansion yet. + "Tilde sub is not split, but var sub is" + ;; This test relies on 'echo -e', which we do not + ;; have. When rewritten to avoid it, we pass. + "IFS empty doesn't do splitting" + ;; This test relies on 'unset' and 'echo -e', + ;; which we do not have. When rewritten to avoid + ;; them, we pass. + "IFS unset behaves like $' \\t\\n'"))))) (for-each (match-lambda ((file tests) (remove-tests tests file))) tests-to-remove)))))))