-
-
Notifications
You must be signed in to change notification settings - Fork 925
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
Fix test profile #978
Comments
I've fixed a few of the outstanding tests with ee734a2. The remaining cases are:
|
https://travis-ci.org/jruby/jruby/jobs/10825790#L167 seems to creep up from time to time. |
@BanzaiMan is this an intermittent issue? I can't seem to reproduce: Running org.jruby.embed.ScriptingContainerTest |
@jvbreen1 feel free to fire up a PR where you revert the changes against a green master and we'll see :) |
@kares I'm not sure I understand. I thought the issue was fixed. Are you suggesting I revert the change in order to reproduce the issue and thus verify the fix? |
Fixed long ago, before 9k release. |
It has been a while since we moved to maven, but the main 'test-extended' target that Travis runs have never been completely fixed.
This is the last successful build with the old 'ant' way (
ant test-extended
): https://travis-ci.org/jruby/jruby/jobs/8438749And this is the first failure with the new 'mvn' way (
mvn -q -Ptest test
): https://travis-ci.org/jruby/jruby/jobs/8685411Not exactly sure how the things changed and broke, but these should be fixed.
The text was updated successfully, but these errors were encountered: