-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Fix #2789 #3753
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 #2789 #3753
Conversation
@@ -115,4 +115,9 @@ class ReplCompilerTests extends ReplTest { | |||
// storedOutput().startsWith("scala.MatchError: null") | |||
// } | |||
} | |||
|
|||
@Test def i2789: Unit = fromInitialState { implicit state => | |||
compile("(x: Int) => println(x)") |
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.
Why not put this in a repl script test?
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.
Because it prints something like this:
val res0: Int => Unit = Lambda$1423/1112073835@3e6534e7
which is not deterministic
@@ -93,27 +93,27 @@ class ReplCompilerTests extends ReplTest { | |||
) | |||
} | |||
|
|||
@Test def i3305: Unit = { | |||
// FIXME: Tests are not run in isolation, the classloader is corrupted after the first exception |
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.
Open an issue for that?
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.
Do you know what PR fixed the underlying problem with anonymous function parsing? |
|
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.
Please add "Fixed in 023303d" to the commit message.
1356384
to
96791ee
Compare
Closing #1424 too?