-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Partest for dotty #410
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
Partest for dotty #410
Conversation
… work yet, move all to one round
Conflicts: test/dotc/tests.scala
I have very mixed feelings about this PR. It feels that by using
I do not have enough experience with partest, so maybe those concerns are caused by lack of skill of using it. Though I assume I'm not only one, who has more experience with Based on this, I'd preffer to leave A sidenote: travis uses 1.5-core machines. There wouldn't be much win from using partest there. |
From high level view. In perfect world we want to have infrastructure that verifies:
Currently this all is done in quite a compact manner, where single file that defines tests. Performance regression is tested for them automatically. Code coverage is also easy to add: I'm actually use By using Scala to define tests we allow to reuse flags via variables, add default flags via emplicits, and a lot more. I believe that having this all defined by directory structure is too limiting, and we'll need to reinvent a wheel. What I propose instead, is to have tests defined in the way they are defined now, but allow ParTest to be a JUnit runner, and generate directory structure for it(including check files) in runtime. |
1fcdb69
to
c4a010e
Compare
Let's discuss next week when I am back. I think we probably will need to run the current tests and partest in parallel, but we can try to avoid duplication, by not having the same test in both frameworks. Partest has the advantage that it makes it far easier to move tests over from scalac, and that is important right now. But I would not want to give up all of the flexibility of the current testing setup. |
Closing this pull-request as we've decided on a different path, namely:
|
Backport "Improve checking LHS of Assign" to 3.3 LTS
Use partest to run pos and neg tests that were run by test/dotc/tests.scala before.
Compiler flags can be provided per directory by a __defaultFlags.flags files, or per test through testname.flags (in this case, __defaultFlags.flags is ignored). For neg tests, the expected number of errors can be provided in a testname.nerr file.