Skip to content

The Zlib::GzipReader in JRuby does not behave as expected with REXML, so the test is skipped#292

Merged
kou merged 1 commit into
ruby:masterfrom
naitoh:skip_zlib_GzipReader
Sep 7, 2025
Merged

The Zlib::GzipReader in JRuby does not behave as expected with REXML, so the test is skipped#292
kou merged 1 commit into
ruby:masterfrom
naitoh:skip_zlib_GzipReader

Conversation

@naitoh

@naitoh naitoh commented Sep 7, 2025

Copy link
Copy Markdown
Contributor

Why?

JRuby's Zlib::GzipReader#readline does not support arguments, so when using Zlib::GzipReader.open, it always returns an empty string and does not function. We will temporarily exclude Zlib::GzipReader from test coverage.

$ test/run.rb --location test/test_order.rb:49

Failure: test_more_ordering(REXMLTests::OrderTester)
/Users/naitoh/ghq/github.com/naitoh/rexml/test/test_order.rb:110:in `test_more_ordering'
     107:          count += 1
     108:        }
     109:
  => 110:        assert_equal( actual.size, count )
     111:      end if defined?(Zlib::GzipReader)
     112:   end
     113: end
<46> expected but was
<0>

See: jruby/jruby#8997

… so the test is skipped

## Why?

JRuby's `Zlib::GzipReader#readline` does not support arguments, so when using `Zlib::GzipReader.open`,
it always returns an empty string and does not function.
We will temporarily exclude `Zlib::GzipReader` from test coverage.

```
$ test/run.rb --location test/test_order.rb:49

Failure: test_more_ordering(REXMLTests::OrderTester)
/Users/naitoh/ghq/github.com/naitoh/rexml/test/test_order.rb:110:in `test_more_ordering'
     107:          count += 1
     108:        }
     109:
  => 110:        assert_equal( actual.size, count )
     111:      end if defined?(Zlib::GzipReader)
     112:   end
     113: end
<46> expected but was
<0>
```

See: jruby/jruby#8997
@kou kou merged commit b5b148e into ruby:master Sep 7, 2025
67 checks passed
@naitoh naitoh deleted the skip_zlib_GzipReader branch September 7, 2025 07:13
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