New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Skip tests that do not run on JRuby #969
Conversation
https://circleci.com/gh/lostisland/faraday/355
in the RubyParser.java. I have seen this failure mode before! Example
https://github.com/igrigorik/em-http-request/blob/v1.1.5/lib/em-http/http_connection.rb#L115 The
I had seen it before: tmm1/http_parser.rb#42 |
Ah, good ole |
OK, the failing CI run on jruby92 is upload to Coverage website. Hm!
Finished in 21.12 seconds (files took 3.88 seconds to load) Randomized with seed 40914 Coverage report generated for RSpec to /home/circleci/project/coverage. 1834 / 2203 LOC (83.25%) covered. Coverage (83.25%) is below the expected minimum coverage (84.00%). Exited with code 2 |
e695ff1
to
2b025f9
Compare
cf8a097
to
0bfbb34
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree we won't be able to officially support it, but it would still be good to have JRuby covered!
@olleolleolle I guess next step here is to add JRuby to the GitHub Actions matrix? |
One thing about GH actions, they don't take Allow Failure easily |
5296e03
to
31b123a
Compare
Add RSpec skips to test cases that can not run on JRuby.
Description
There are adapters that do not support JRuby.
This is about investigating a JRuby build for the CI.
We do not have a maintainer which can work focused with JRuby, right now, so we want to keep it in there, but not as a required job.
Todos
Invalid header type