-
Notifications
You must be signed in to change notification settings - Fork 842
cleanup and simplify test runner logic, remove RunTests.cmd #1628
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
Conversation
|
please don't .... I use the script. It may be easier when doing a full pass to not have the script. But the script is usefull when running a subset of tests. |
|
@KevinRansom I can make it so that you can do whatever you used to do via direct invocations of build.cmd (we can add a "nobuild" option if needed). |
|
@KevinRansom I'll make sure you can do the following from build.cmd
|
|
@KevinRansom I've extended this with a major cleanup of the test runner scripts under tests\fsharp (mainly with a view to working out how we're going to run these on CoreCLR and cross-platform) The logic of RunTests.cmd is now available directly from Build: Build then test (auto-chooses all applicable tests): Build then test a particular suite: Build then run tests marked with Conformance tag: Run tests excluding those marked with Slow tag (skip the build step) |
|
Unrelated failure: |
|
@KevinRansom I've merged this. Please let me know if the new |
The script "RunTests.cmd" is not worth much - it basically gets in the way. Moving this stuff into
build.cmdseems a better way to make progress on simplifying the build/test process and gradually reducing the use of batch files.As an aside, I have these ultimate aims in this cleanup work
tests\fsharp