Merge pull request #14953 from apache/micronautTests

feature - #14017 - add back Micronaut support to Grails 7 with optional plugin

Author: jdaugherty

RENAME.md

@@ -112,7 +112,7 @@ Below is a reference of all migrated artifacts - both their old and new name.
 | org.grails.plugins                | gsp                                             | org.apache.grails                               | grails-gsp                                                    |                        |                        | grails-views           |
 | org.grails                        | grails-gsp                                      | org.apache.grails.views                         | grails-gsp-core                                               |                        |                        | grails-views           |
 | org.grails                        | grails-logging                                  | org.apache.grails                               | grails-logging                                                |                        |                        | grails-core            |
-| org.grails                        | grails-plugin-i18n                              | org.apache.grails                               | grails-i18n                                                   |                        |                        | grails-core            |
+| org.grails                        | grails-plugin-i18n                              | org.apache.grails.i18n                          | grails-i18n                                                   |                        |                        | grails-core            |
 | org.grails                        | grails-plugin-interceptors                      | org.apache.grails                               | grails-interceptors                                           |                        |                        | grails-core            |
 | org.grails                        | grails-plugin-mimetypes                         | org.apache.grails                               | grails-mimetypes                                              |                        |                        | grails-core            |
 | org.grails.plugins                | quartz                                          | org.apache.grails                               | grails-quartz                                                 |                        |                        | grails-quartz          |

etc/bin/rename_gradle_artifacts.sh

@@ -67,7 +67,7 @@ declare -a core_mappings=(
 "org[.]grails:grails-plugin-rest|org.apache.grails:grails-rest-transforms"
 "org[.]grails:grails-plugin-mimetypes|org.apache.grails:grails-mimetypes"
 "org[.]grails:grails-plugin-interceptors|org.apache.grails:grails-interceptors"
-"org[.]grails:grails-plugin-i18n|org.apache.grails:grails-i18n"
+"org[.]grails:grails-plugin-i18n|REMOVE_ME" # i18n is now included by default and should not be included explicitly
 "org[.]grails:grails-plugin-domain-class|org.apache.grails:grails-domain-class"
 "org[.]grails:grails-plugin-datasource|org.apache.grails:grails-datasource"
 "org[.]grails:grails-plugin-databinding|org.apache.grails:grails-databinding"
@@ -109,7 +109,7 @@ declare -a excluded_core_mappings=(
 "exclude[[:space:]]+module[[:space:]]*:[[:space:]]*['\"]grails-plugin-rest['\"]|exclude module:'grails-rest-transforms'"
 "exclude[[:space:]]+module[[:space:]]*:[[:space:]]*['\"]grails-plugin-mimetypes['\"]|exclude module:'grails-mimetypes'"
 "exclude[[:space:]]+module[[:space:]]*:[[:space:]]*['\"]grails-plugin-interceptors['\"]|exclude module:'grails-interceptors'"
-"exclude[[:space:]]+module[[:space:]]*:[[:space:]]*['\"]grails-plugin-i18n['\"]|exclude module:'grails-i18n'"
+"exclude[[:space:]]+module[[:space:]]*:[[:space:]]*['\"]grails-plugin-i18n['\"]|exclude module:'REMOVE_THIS_EXCLUDE'"
 "exclude[[:space:]]+module[[:space:]]*:[[:space:]]*['\"]grails-plugin-domain-class['\"]|exclude module:'grails-domain-class'"
 "exclude[[:space:]]+module[[:space:]]*:[[:space:]]*['\"]grails-plugin-datasource['\"]|exclude module:'grails-datasource'"
 "exclude[[:space:]]+module[[:space:]]*:[[:space:]]*['\"]grails-plugin-databinding['\"]|exclude module:'grails-databinding'"

gradle.properties

@@ -49,11 +49,15 @@ apacheRatVersion=0.8.1
 gradleChecksumPluginVersion=1.4.0
 gradleLicensePluginVersion=0.16.1
 
+# micronaut libraries not in the bom due to the potential for spring mismatches
+micronautPlatformVersion=4.9.2
+
 # Libraries only specific to test apps, these should not be exposed
 grailsSpringSecurityVersion=7.0.0-SNAPSHOT
 jbossTransactionApiVersion=2.0.0.Final
-micronautVersion=4.9.10
-micronautSerdeJacksonVersion=2.15.1
+# note: we do not import the micronaut bom in our tests to avoid spring version mismatches
+micronautHttpClientVersion=4.9.9
+micronautSerdeJacksonVersion=2.11.0
 
 # build dependencies for code quality checks
 checkstyleVersion=11.0.0

gradle/dependency-licenses.gradle

@@ -20,7 +20,7 @@
 apply plugin: 'com.github.hierynomus.license-report'
 
 List<String> licenseExclusions = rootProject.subprojects.collect {
-    "org.apache.grails:${it.findProperty('pomArtifactId') ?: it.name}:${rootProject.projectVersion}" as String
+    "${it.group}:${it.findProperty('pomArtifactId') ?: it.name}:${rootProject.projectVersion}" as String
 }
 
 downloadLicenses {

gradle/java-config.gradle

@@ -34,7 +34,7 @@ tasks.withType(Javadoc).configureEach { Javadoc it ->
 
 tasks.withType(GroovyCompile).configureEach {
     groovyOptions.encoding = 'UTF-8' // encoding needs to be the same since it's different across platforms
-    // Preserve method parameter names in Groovy classes for IDE parameter hints.
+    // Preserve method parameter names in Groovy/Java classes for IDE parameter hints & bean reflection metadata.
     groovyOptions.parameters = true
     options.encoding = 'UTF-8' // encoding needs to be the same since it's different across platforms
     options.fork = true

gradle/publish-root-config.gradle

@@ -24,9 +24,9 @@ version = projectVersion
 group = 'this.will.be.overridden'
 
 def publishedProjects = [
+        'grails-async',
         'grails-async-core',
         'grails-async-gpars',
-        'grails-async',
         'grails-async-rxjava',
         'grails-async-rxjava2',
         'grails-async-rxjava3',
@@ -43,15 +43,15 @@ def publishedProjects = [
         'grails-databinding',
         'grails-databinding-core',
         'grails-datasource',
-        'grails-dependencies-starter-web',
         'grails-dependencies-assets',
+        'grails-dependencies-starter-web',
         'grails-dependencies-test',
         'grails-domain-class',
         'grails-encoder',
+        'grails-events',
         'grails-events-compat',
         'grails-events-core',
         'grails-events-gpars',
-        'grails-events',
         'grails-events-rxjava',
         'grails-events-rxjava2',
         'grails-events-rxjava3',
@@ -63,14 +63,15 @@ def publishedProjects = [
         'grails-gsp-core',
         'grails-i18n',
         'grails-interceptors',
+        'grails-layout',
         'grails-logging',
+        'grails-micronaut',
         'grails-mimetypes',
         'grails-rest-transforms',
         'grails-scaffolding',
         'grails-services',
         'grails-shell-cli',
         'grails-sitemesh3',
-        'grails-layout',
         'grails-spring',
         'grails-taglib',
         'grails-test-core',
@@ -94,11 +95,11 @@ def publishedProjects = [
         'grails-web-taglib',
         'grails-web-url-mappings',
         // datastore
-        'grails-datastore-async',
-        'grails-datastore-core',
+        'grails-datamapping-async',
         'grails-datamapping-core',
         'grails-datamapping-core-test',
-        'grails-datamapping-async',
+        'grails-datastore-async',
+        'grails-datastore-core',
         //'grails-datamapping-rx',
         'grails-datamapping-support',
         'grails-datamapping-tck',
@@ -108,23 +109,23 @@ def publishedProjects = [
         // simple
         'grails-data-simple',
         // hibernate5
-        'grails-data-hibernate5-spring-boot',
-        'grails-data-hibernate5-dbmigration',
-        'grails-data-hibernate5-core',
         'grails-data-hibernate5',
+        'grails-data-hibernate5-core',
+        'grails-data-hibernate5-dbmigration',
+        'grails-data-hibernate5-spring-boot',
         // mongodb
-        'grails-data-mongodb-spring-boot',
-        'grails-data-mongodb-core',
+        'grails-data-mongodb',
         'grails-data-mongodb-bson',
+        'grails-data-mongodb-core',
         'grails-data-mongodb-ext',
-        'grails-data-mongodb',
         'grails-data-mongodb-gson-templates',
+        'grails-data-mongodb-spring-boot',
         // wrapper
         'grails-wrapper',
         // profiles
         'grails-profiles-base',
-        'grails-profiles-profile',
         'grails-profiles-plugin',
+        'grails-profiles-profile',
         'grails-profiles-rest-api',
         'grails-profiles-rest-api-plugin',
         'grails-profiles-web',

gradle/rat-root-config.gradle

@@ -21,8 +21,6 @@ tasks.named('rat') {
     def allExcludes = [
             '.asf.yaml', // ASF metadata for github integration excluded from src zip
             'CODE_OF_CONDUCT.md',
-            'LICENSE',
-            'NOTICE',
             'BUILD_DATE', // build artifact for storing the build date / verifying
             'CHECKSUMS', // build artifact for storing checksums for easy verification
             'PUBLISHED_ARTIFACTS', // build artifact for storing published artifacts coordinates & paths
@@ -34,7 +32,6 @@ tasks.named('rat') {
             'grails-web-common/src/main/groovy/org/grails/web/json/JSONElement.java',
             'grails-web-common/src/main/groovy/org/grails/web/json/JSONException.java',
             'grails-web-common/src/main/groovy/org/grails/web/json/JSONWriter.java',
-            'gradle/wrapper/gradle-wrapper.properties', // gradle wrapper excluded from src zip
             '**/.gitattributes', // git configuration isn't code
             '**/.gradle/**', '**/wrapper/**', 'gradlew*',  // gradle wrapper files excluded from src zip
             '**/*.html', // html files are only in test

grails-async/plugin/src/main/groovy/org/grails/plugins/web/async/ControllersAsyncGrailsPlugin.groovy

@@ -31,7 +31,7 @@ import org.grails.plugins.web.async.spring.PromiseFactoryBean
  */
 class ControllersAsyncGrailsPlugin extends Plugin {
 
-    def grailsVersion = '7.0.0 > *'
+    def grailsVersion = '7.0.0-SNAPSHOT > *'
     def loadAfter = ['controllers']
     Closure doWithSpring() {
         { ->

grails-cache/src/main/groovy/grails/plugin/cache/CacheGrailsPlugin.groovy

@@ -30,7 +30,7 @@ import org.grails.plugin.cache.GrailsCacheManager
 @Slf4j
 class CacheGrailsPlugin extends Plugin {
 
-    def grailsVersion = '7.0.0 > *'
+    def grailsVersion = '7.0.0-SNAPSHOT > *'
     def observe = ['controllers', 'services']
     def loadAfter = ['controllers', 'services']
     def authorEmail = '[email protected]'

grails-controllers/src/main/groovy/org/grails/plugins/web/controllers/ControllersAutoConfiguration.java

@@ -44,12 +44,16 @@
 
 import grails.config.Settings;
 import grails.core.GrailsApplication;
+import org.grails.plugins.domain.GrailsDomainClassAutoConfiguration;
 import org.grails.web.config.http.GrailsFilters;
 import org.grails.web.filters.HiddenHttpMethodFilter;
 import org.grails.web.servlet.mvc.GrailsDispatcherServlet;
 import org.grails.web.servlet.mvc.GrailsWebRequestFilter;
 
-@AutoConfiguration(before = { HttpEncodingAutoConfiguration.class, WebMvcAutoConfiguration.class })
+@AutoConfiguration(
+        before = {HttpEncodingAutoConfiguration.class, WebMvcAutoConfiguration.class},
+        after = {GrailsDomainClassAutoConfiguration.class}
+)
 @ConditionalOnWebApplication(type = ConditionalOnWebApplication.Type.SERVLET)
 public class ControllersAutoConfiguration {