Refactor to be more in line with a standard Gradle project structure

.gitignore

@@ -74,6 +74,9 @@ local.properties
 # Auto-generated java files
 autogenerated/
 
+# IDEA
+nomulus.iml
+
 ######################################################################
 # Python Ignores
 
@@ -83,11 +86,9 @@ autogenerated/
 # Gradle Ignores
 
 # We don't want to ignore the gradle jar files
-!/gradle/gradle/wrapper/**/*.jar
+!/gradle/wrapper/**/*.jar
 .gradle/
-/gradle/.gradle
+**/build
-/gradle/**/WEB-INF
+node_modules/**
-/gradle/**/build
+!node_modules/soyutils_usegoog.js
-/gradle/node_modules/**
-!/gradle/node_modules/soyutils_usegoog.js
 /repos/

.travis.yml

@@ -55,4 +55,4 @@ env:
 # output, instead of the default 10.
 # See notes on the CREDZ and REPORT_GCS_BUCKET environment variable in the
 # comments at the top of the file.
-script: cd gradle && echo "$CREDZ" >credz.json && chmod 755 ./gradlew && travis_wait 45 ./gradlew build --continue -P uploaderDestination="$REPORT_GCS_DESTINATION" -P uploaderCredentialsFile=credz.json -P uploaderMultithreadedUpload=yes -P mavenUrl=https://storage.googleapis.com/domain-registry-maven-repository/maven -P pluginsUrl=https://storage.googleapis.com/domain-registry-maven-repository/plugins
+script: echo "$CREDZ" >credz.json && chmod 755 ./gradlew && travis_wait 45 ./gradlew build --continue -P uploaderDestination="$REPORT_GCS_DESTINATION" -P uploaderCredentialsFile=credz.json -P uploaderMultithreadedUpload=yes -P mavenUrl=https://storage.googleapis.com/domain-registry-maven-repository/maven -P pluginsUrl=https://storage.googleapis.com/domain-registry-maven-repository/plugins

appengine_war.gradle

@@ -21,14 +21,14 @@ apply plugin: 'war'
 // plugin will recognize this as an app-engine standard app (and also
 // obtains the appengine-web.xml from the correct location)
 project.convention.plugins['war'].webAppDirName =
-        "../../../java/google/registry/env/${environment}/${project.name}"
+        "../../java/google/registry/env/${environment}/${project.name}"
 
 apply plugin: 'com.google.cloud.tools.appengine'
 
 // Get the web.xml file for the service.
 war {
     webInf {
-        from "../../../java/google/registry/env/common/${project.name}/WEB-INF"
+        from "../../java/google/registry/env/common/${project.name}/WEB-INF"
     }
 }
 

build.gradle

@@ -65,9 +65,7 @@ if (!project.hasProperty('showAllOutput')) {
 gradleLint.autoLintAfterFailure = false
 
 // Paths to main and test sources.
-ext.projectRootDir = "${rootDir}/.."
+ext.projectRootDir = "${rootDir}"
-ext.javaDir = "${rootDir}/../java"
-ext.javatestsDir = "${rootDir}/../javatests"
 
 // Tasks to deploy/stage all App Engine services
 task deploy {
@@ -161,43 +159,42 @@ subprojects {
 
   def commonlyExcludedResources = ['**/*.java', '**/BUILD']
 
+  project.ext.javaDir = "${project.projectDir}/src/main/java"
+  project.ext.javaTestDir = "${project.projectDir}/src/test/java"
+
   sourceSets {
     main {
       java {
-        srcDirs = [
+        srcDirs += aptGeneratedDir
-            rootProject.javaDir,
-            aptGeneratedDir
-        ]
       }
       resources {
         srcDirs = [
-            rootProject.javaDir
+            project.ext.javaDir
         ]
         exclude commonlyExcludedResources
       }
     }
     test {
       java {
-        srcDirs = [
+        srcDirs += aptGeneratedTestDir
-            rootProject.javatestsDir,
-            aptGeneratedTestDir
-        ]
       }
       resources {
         srcDirs = [
-            rootProject.javatestsDir,
+            project.ext.javaTestDir
         ]
         exclude commonlyExcludedResources
       }
     }
   }
 
+  if (project.name == 'util') return
+  if (project.name == 'proxy') return
+  if (project.name == 'core') return
+
   test {
     testLogging.showStandardStreams = Boolean.parseBoolean(showAllOutput)
   }
 
-  if (project.name == 'core') return
-
   ext.relativePath = "google/registry/${project.name}"
 
   sourceSets.each {
@@ -208,13 +205,5 @@ subprojects {
       include "${project.relativePath}/"
     }
   }
-  project(':core').sourceSets.each {
-    it.java {
-      exclude "${project.relativePath}/"
-    }
-    it.resources {
-      exclude "${project.relativePath}/"
-    }
-  }
 }
 

buildSrc/build.gradle

@@ -41,6 +41,14 @@ repositories {
 apply from: '../dependencies.gradle'
 apply from: '../java_common.gradle'
 
+sourceSets {
+  main {
+    java {
+      srcDirs += "${project.buildDir}/generated/source/apt/main"
+    }
+  }
+}
+
 checkstyle {
   configDir file('../config/checkstyle')
 }
@@ -50,6 +58,10 @@ checkstyle {
 // - Format in place: ../gradlew spotlessApply
 spotless {
   java {
+    target project.fileTree("${project.rootDir}/") {
+      include "src/main/java/**/*.java"
+      include "src/test/java/**/*.java"
+    }
     googleJavaFormat('1.7')
   }
   format 'misc', {

config/checkstyle/suppressions.xml

@@ -8,5 +8,5 @@
   <!-- Ignore generated files in Checkstyle -->
   <suppress files="[/\\].*[/\\]generated.*[/\\]" checks="."/>
   <!-- Ignore Javadoc checks in test files -->
-  <suppress files="[/\\].*[/\\]javatests.*[/\\]" checks="JavadocType"/>
+  <suppress files="[/\\].*[/\\]src/test/java/.*[/\\]" checks="JavadocType"/>
 </suppressions>

core/.gitignore

@@ -0,0 +1 @@
+out/

core/build.gradle

@@ -10,7 +10,7 @@ def screenshotsDir = "${project.buildDir}/screenshots"
 def screenshotsForGoldensDir = "${project.buildDir}/screenshots_for_goldens"
 def newGoldensDir = "${project.buildDir}/new_golden_images"
 def goldensDir =
-  "${javatestsDir}/google/registry/webdriver/goldens/chrome-linux"
+  "${project.ext.javaTestDir}/google/registry/webdriver/goldens/chrome-linux"
 
 // 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
@@ -110,7 +110,7 @@ dependencies {
   // Custom-built objectify jar at commit ecd5165, included in Nomulus
   // release.
   implementation files(
-      "${rootDir}/../third_party/objectify/v4_1/objectify-4.1.3.jar")
+      "${rootDir}/third_party/objectify/v4_1/objectify-4.1.3.jar")
   testImplementation project(':third_party')
 
   compile deps['com.beust:jcommander']
@@ -309,10 +309,10 @@ dependencies {
 }
 
 task jaxbToJava {
-  def xsdFilesDir = "${javaDir}/google/registry/xml/xsd"
+  def xsdFilesDir = "${project.ext.javaDir}/google/registry/xml/xsd"
-  def bindingsFile = "${javaDir}/google/registry/xjc/bindings.xjb"
+  def bindingsFile = "${project.ext.javaDir}/google/registry/xjc/bindings.xjb"
-  def pkgInfoGenerator = "${javaDir}/google/registry/xjc/make_pkginfo.sh"
+  def pkgInfoGenerator = "${project.ext.javaDir}/google/registry/xjc/make_pkginfo.sh"
-  def pkgInfoTemplate = "${javaDir}/google/registry/xjc/package-info.java.in"
+  def pkgInfoTemplate = "${project.ext.javaDir}/google/registry/xjc/package-info.java.in"
   def outputDir = "${generatedDir}/google/registry/xjc"
 
   inputs.dir xsdFilesDir
@@ -374,7 +374,7 @@ task soyToJava {
           spec11SoyDir, toolsSoyDir, uiSoyDir, registrarSoyDir,
   ]
   soyRelativeDirs.each {
-    inputs.dir "${javaDir}/${it}"
+    inputs.dir "${project.ext.javaDir}/${it}"
     outputs.dir "${generatedDir}/${it}"
   }
 
@@ -394,19 +394,19 @@ task soyToJava {
     soyToJava('google.registry.tools.soy',
             "${generatedDir}/${toolsSoyDir}",
             fileTree(
-                    dir: "${javaDir}/${toolsSoyDir}",
+                    dir: "${project.ext.javaDir}/${toolsSoyDir}",
                     include: ['**/*.soy']))
 
     soyToJava('google.registry.ui.soy.registrar',
             "${generatedDir}/${registrarSoyDir}",
             fileTree(
-                    dir: "${javaDir}/${registrarSoyDir}",
+                    dir: "${project.ext.javaDir}/${registrarSoyDir}",
                     include: ['**/*.soy']))
 
     soyToJava('google.registry.ui.soy',
             "${generatedDir}/${uiSoyDir}",
             files {
-              file("${javaDir}/${uiSoyDir}").listFiles()
+              file("${project.ext.javaDir}/${uiSoyDir}").listFiles()
             }.filter {
               it.name.endsWith(".soy")
             })
@@ -414,13 +414,13 @@ task soyToJava {
     soyToJava('google.registry.reporting.spec11.soy',
             "${generatedDir}/${spec11SoyDir}",
             fileTree(
-                    dir: "${javaDir}/${spec11SoyDir}",
+                    dir: "${project.ext.javaDir}/${spec11SoyDir}",
                     include: ['**/*.soy']))
   }
 }
 
 task soyToJS {
-  def rootSoyDirectory = "${javaDir}/google/registry/ui/soy"
+  def rootSoyDirectory = "${project.ext.javaDir}/google/registry/ui/soy"
   def outputSoyDirectory = "${generatedDir}/google/registry/ui/soy"
   inputs.dir rootSoyDirectory
   outputs.dir outputSoyDirectory
@@ -434,7 +434,7 @@ task soyToJS {
           "--allowExternalCalls", "false",
           "--srcs", "${soyFiles.join(',')}",
           "--shouldProvideRequireSoyNamespaces", "true",
-          "--compileTimeGlobalsFile", "${javaDir}/google/registry/ui/globals.txt",
+          "--compileTimeGlobalsFile", "${project.ext.javaDir}/google/registry/ui/globals.txt",
           "--deps", "${deps.join(',')}"
     }
   }
@@ -456,7 +456,7 @@ task soyToJS {
 }
 
 task stylesheetsToJavascript {
-  def cssSourceDir = "${javaDir}/google/registry/ui/css"
+  def cssSourceDir = "${project.ext.javaDir}/google/registry/ui/css"
   def outputDir = "${resourcesDir}/google/registry/ui/css"
   inputs.dir cssSourceDir
   outputs.dir outputDir
@@ -509,8 +509,8 @@ task compileProdJS(type: JavaExec) {
   def outputDir = "${resourcesDir}/google/registry/ui"
   def nodeModulesDir = "${rootDir}/node_modules"
   def cssSourceDir = "${resourcesDir}/google/registry/ui/css"
-  def jsSourceDir = "${javaDir}/google/registry/ui/js"
+  def jsSourceDir = "${project.ext.javaDir}/google/registry/ui/js"
-  def externsDir = "${javaDir}/google/registry/ui/externs"
+  def externsDir = "${project.ext.javaDir}/google/registry/ui/externs"
   def soySourceDir = "${generatedDir}/google/registry/ui/soy"
 
   [nodeModulesDir, cssSourceDir, jsSourceDir, externsDir, soySourceDir].each {