From e030713910ce0b1964e863216cb570c16bbddd5f Mon Sep 17 00:00:00 2001 From: Gleb Mazovetskiy Date: Mon, 25 Mar 2019 00:34:23 +0000 Subject: [PATCH] Fix --test-probe 1. Changes `folder` to `dir.to_s` (`folder` no longer exists). 2. Changes `passing.any?` to `!passing.empty?`, as `any?` has pitfalls: Took me a few minutes to debug when I tried with `input_path` that was returning `nil` (`[nil].any? #=> false`). --- lib/sass_spec/runner.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/sass_spec/runner.rb b/lib/sass_spec/runner.rb index 537e2dfa10..0d5e5fa5f2 100644 --- a/lib/sass_spec/runner.rb +++ b/lib/sass_spec/runner.rb @@ -60,10 +60,10 @@ def run passing = [] test_cases.each do |test_case| if test_case.todo? && test_case.result? - passing << test_case.folder + passing << test_case.dir.to_s end end - if passing.any? + if !passing.empty? puts "The following tests pass but were marked as TODO for #{@options[:engine_adapter].describe}:" puts passing.join("\n") else