-
Notifications
You must be signed in to change notification settings - Fork 70
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
Add shorter task alias for C++ test execution #480
Comments
The same could be said for test aliases like this: task test {
dependsOn 'testRelease'
}
task testDebug {
dependsOn 'runFrcUserProgramTest' + wpi.platforms.desktop.capitalize() + 'DebugGoogleTestExe'
}
task testRelease {
dependsOn 'runFrcUserProgramTest' + wpi.platforms.desktop.capitalize() + 'ReleaseGoogleTestExe'
} |
This was resolved by the adition of simulateNative, right? |
Only the simulation part. testNativeDebug and testNativeDebug build the tests, but don't run them. There's still no equivalent of the test, testDebug, and testRelease tasks above. |
I renamed this to more fit what the issue is now. |
Are the C++ tests not run in the build task? |
They are, but the build task builds way too much; it builds the athena binary, simulation binary, and test binary (plus debug and release versions of some of those, iirc). This takes a long time in C++. A test task should only build, say, the debug version of the test binary and run it. Java already has this, so this gives C++ robot projects feature parity. |
The following in build.gradle works, but it would be nice if GradleRIO provided it like
simulateJava
.The text was updated successfully, but these errors were encountered: