Skip to content

Commit 72a018c

Browse files
committed
[test] do not fail test due missing --add-opens
test is relying on internals and is kind of 'optional'
1 parent 2796fee commit 72a018c

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

src/test/ruby/test_security_helper.rb

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,15 @@ def test_cert_factory_provider_leak # GH-94
1717
assert_equal 'BC', factory2.provider.name
1818
# assert_same factory1.getProvider, factory2.getProvider
1919

20-
java.security.cert.CertificateFactory.class_eval do
21-
field_reader :certFacSpi
22-
end
20+
begin
21+
java.security.cert.CertificateFactory.class_eval do
22+
field_reader :certFacSpi
23+
end
2324

24-
spi1 = factory1.certFacSpi; spi2 = factory2.certFacSpi
25+
spi1 = factory1.certFacSpi; spi2 = factory2.certFacSpi
26+
rescue SecurityError => e
27+
return skip "#{__method__} probably needs --add-opens (#{e.message})"
28+
end
2529

2630
if spi1.is_a? org.bouncycastle.jcajce.provider.asymmetric.x509.CertificateFactory
2731
org.bouncycastle.jcajce.provider.asymmetric.x509.CertificateFactory.class_eval do

0 commit comments

Comments
 (0)