Skip to content

Conversation

@tikkss
Copy link
Contributor

@tikkss tikkss commented Oct 18, 2025

To fix the following error:

$ rake
/Users/zzz/.rbenv/versions/3.4.2/bin/ruby test/run-test.rb
Loaded suite test
Started
F
========================================================================
Failure: test: too many redirection(DownloaderTest::#download)
/Users/zzz/src/github.com/red-data-tools/red-remote-input/test/test-downloader.rb:61:in 'block (2 levels) in <class:DownloaderTest>'
     58:         raise RemoteInput::Downloader::TooManyRedirects, "too many redirections: #{last_url}"
     59:       end
     60:
  => 61:       assert_raise(RemoteInput::Downloader::TooManyRedirects.new(expected_message)) do
     62:         downloader.download(output_path)
     63:       end
     64:     end

<RemoteInput::Downloader::TooManyRedirects(<too many redirections: https://example.com/file .. https://example.com/last_redirection>)> expected but was
<NameError(<uninitialized constant RemoteInput::Downloader::VERSION>)
(snip)

To fix the following error:

```console
$ rake
/Users/zzz/.rbenv/versions/3.4.2/bin/ruby test/run-test.rb
Loaded suite test
Started
F
========================================================================
Failure: test: too many redirection(DownloaderTest::#download)
/Users/zzz/src/github.com/red-data-tools/red-remote-input/test/test-downloader.rb:61:in 'block (2 levels) in <class:DownloaderTest>'
     58:         raise RemoteInput::Downloader::TooManyRedirects, "too many redirections: #{last_url}"
     59:       end
     60:
  => 61:       assert_raise(RemoteInput::Downloader::TooManyRedirects.new(expected_message)) do
     62:         downloader.download(output_path)
     63:       end
     64:     end

<RemoteInput::Downloader::TooManyRedirects(<too many redirections: https://example.com/file .. https://example.com/last_redirection>)> expected but was
<NameError(<uninitialized constant RemoteInput::Downloader::VERSION>)
(snip)
```
@kojix2 kojix2 merged commit 67afaa3 into red-data-tools:main Oct 19, 2025
12 checks passed
@kojix2
Copy link
Member

kojix2 commented Oct 19, 2025

Thanks!

@tikkss tikkss deleted the add-missing-require branch October 19, 2025 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants