OpenSSL tests fail - uninitialized constant OpenSSL about nio4r HOT 8 CLOSED

bump

from nio4r.

Do you mind making a PR? Thanks!

from nio4r.

e53e5d5 is the root cause for the issue. The question is if it should be reverted, or if the constant should be pre-defined outside of the block. But revert is probably easier solution.

from nio4r.

Boy, that was stupid (I authored the commit). Don't recall. I think the intent was to not load openssl when the spec files load, so some specs may run without openssl being loaded. Maybe change

RSpec.describe OpenSSL::SSL::SSLSocket do

to

RSpec.describe 'OpenSSL::SSL::SSLSocket' do

from nio4r.

Will this be patched in the code soon, @MSP-Greg ?

from nio4r.

@Segaja Not sure. If the above is used, other specs can be run without loading Ruby's OpenSSL. Not sure if that's helpful.

Happy to do a PR with the above...

from nio4r.

@MSP-Greg I can confirm the fix above with the quotes works. For now I patch it manually in the build for arch, but it would be nice if it would be fixed in the project itself.

from nio4r.

I've fixed this by moving the require to before the describe block.

from nio4r.