From edbd4cb7f526897b48bf396e6cf3fccc3d74b378 Mon Sep 17 00:00:00 2001 From: Daniel Bevenius Date: Thu, 1 May 2025 08:12:48 +0200 Subject: [PATCH] ruby : ignore "Downloading" output in test_log_suppress (#3106) This commit adds a temporary fix to the `test_log_suppress` test in the Ruby bindings. The motivation for this changes is that I suspect that the recent migration of the models to HuggingFace Xet has changed the way HTTP caching works for the models. This is causing the test in question to fail. This is a temporary fix so that CI is not broken while we investigate this further. --- bindings/ruby/tests/test_whisper.rb | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/bindings/ruby/tests/test_whisper.rb b/bindings/ruby/tests/test_whisper.rb index 2754ab06..dfc4e0e0 100644 --- a/bindings/ruby/tests/test_whisper.rb +++ b/bindings/ruby/tests/test_whisper.rb @@ -118,7 +118,18 @@ class TestWhisper < TestBase dev = StringIO.new("") $stderr = dev Whisper::Context.new("base.en") - assert_empty dev.string + + # Filter out any lines starting with "Downloading" or containing only dots. + # The reason for this is that I think the recent migration to Huggingface + # Xet might have changed the downloading behavior. There is now a redirect + # to a different URL, which causes the download to be retried even if the + # file is already downloaded. + # TODO(danbev) Remove this when a proper fix is in place. + filtered_output = dev.string.lines.reject do |line| + line.start_with?("Downloading") || line.strip =~ /^\.+$/ + end.join + + assert_empty filtered_output, "Expected no output, but got: #{filtered_output.inspect}" ensure $stderr = stderr end