Skip to content

Conversation

@naitoh
Copy link
Contributor

@naitoh naitoh commented Sep 7, 2025

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