zydronium/google-nomulus
1
0
Fork 0
mirror of https://github.com/google/nomulus.git synced 2025-07-20 17:56:08 +02:00
Code Issues Projects Releases Packages Wiki Activity

Reenable Test Executor sharing in Gradle build

Browse source 
Combining all tests in one suite and drop the forkEvery=1 directive.
Issue was fixed by [] and []

TESTED=Run locally with maxParallelForks =1 and 5, and tested on travis
       with maxParallelForks=5

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=229414759
This commit is contained in:
weiminyu 2019-01-15 12:04:14 -08:00 • committed by jianglai
parent 30c05a6e76
commit 035b9149e0
1 changed files with 1 additions and 53 deletions
Download patch file Download diff file Expand all files Collapse all files

54
gradle/core/build.gradle
View file

@ -6,30 +6,6 @@ plugins {
// used for easy inspection. // used for easy inspection.
def generatedDir = "${project.buildDir}/generated/source/custom/main" def generatedDir = "${project.buildDir}/generated/source/custom/main"
// Tests that conflict with (mostly unidentified) members of the main test
// suite. It is unclear if they are offenders (i.e., those that pollute global
// state) or victims.
// TODO(weiminyu): identify cause and fix offending tests.
def outcastTestPatterns = [
"google/registry/batch/DeleteContactsAndHostsActionTest.*",
"google/registry/batch/RefreshDnsOnHostRenameActionTest.*",
"google/registry/flows/CheckApiActionTest.*",
"google/registry/flows/EppLifecycleHostTest.*",
"google/registry/flows/domain/DomainCreateFlowTest.*",
"google/registry/flows/domain/DomainUpdateFlowTest.*",
"google/registry/tools/CreateDomainCommandTest.*",
"google/registry/tools/server/CreatePremiumListActionTest.*",
// Conflicts with WhoisActionTest
"google/registry/whois/WhoisHttpActionTest.*",
]
// Tests that conflict with members of both the main test suite and the
// outcast suite.
// TODO(weiminyu): identify cause and fix offending tests.
def fragileTestPatterns = [
"google/registry/cron/TldFanoutActionTest.*"
]
sourceSets { sourceSets {
main { main {
java { java {
@ -428,41 +404,13 @@ artifacts {
testRuntime testJar testRuntime testJar
} }
task fragileTest(type: Test) {
// Common exclude pattern. See README in parent directory for explanation.
exclude "**/*TestCase.*", "**/*TestSuite.*"
include fragileTestPatterns
// Run every test class in a freshly started process.
forkEvery 1
// Uncomment to see test outputs in stdout.
//testLogging.showStandardStreams = true
}
task outcastTest(type: Test) {
// Common exclude pattern. See README in parent directory for explanation.
exclude "**/*TestCase.*", "**/*TestSuite.*"
include outcastTestPatterns
// Sets the maximum number of test executors that may exist at the same time.
maxParallelForks 5
}
test { test {
// Common exclude pattern. See README in parent directory for explanation. // Common exclude pattern. See README in parent directory for explanation.
exclude "**/*TestCase.*", "**/*TestSuite.*" exclude "**/*TestCase.*", "**/*TestSuite.*"
exclude fragileTestPatterns
exclude outcastTestPatterns
// Run every test class in its own process.
// TODO: Figure out why we need to do this, fix it and then stop doing it.
// This setting makes the build take 35 minutes, without it it takes about 10.
forkEvery 1
// Sets the maximum number of test executors that may exist at the same time. // Sets the maximum number of test executors that may exist at the same time.
maxParallelForks 5 maxParallelForks 5
}.dependsOn(fragileTest, outcastTest) }
task nomulus(type: Jar) { task nomulus(type: Jar) {
manifest { manifest {