Merge pull request #15150 from gustavoharff/patch-1

Apply Generated annotation

Author: matrei

grails-datamapping-async/src/main/groovy/grails/gorm/async/AsyncEntity.groovy

@@ -20,7 +20,7 @@
 package grails.gorm.async
 
 import groovy.transform.CompileStatic
-
+import groovy.transform.Generated
 import org.grails.datastore.gorm.GormEnhancer
 import org.grails.datastore.gorm.GormEntity
 import org.grails.datastore.gorm.async.GormAsyncStaticApi
@@ -37,6 +37,7 @@ trait AsyncEntity<D> extends GormEntity<D> {
     /**
      * @return The async version of the GORM static API
      */
+    @Generated
     static GormAsyncStaticApi<D> getAsync() {
         return new GormAsyncStaticApi(GormEnhancer.findStaticApi(this))
     }

grails-datamapping-core-test/src/test/groovy/org/grails/datastore/gorm/schemaless/ImplementsDynamicAttributes.groovy

@@ -18,9 +18,12 @@
  */
 package org.grails.datastore.gorm.schemaless
 
+import groovy.transform.Generated
 import spock.lang.Issue
 import spock.lang.Specification
 
+import java.lang.reflect.Method
+
 
 class DynamicDomainSpec extends Specification {
 
@@ -41,6 +44,12 @@ class DynamicDomainSpec extends Specification {
         entity.attributes().foo == 123
     }
 
+    void "test that all DynamicAttributes trait methods are marked as Generated"() {
+        expect: "all DynamicAttributes methods are marked as Generated on implementation class"
+        DynamicAttributes.getMethods().each { Method traitMethod ->
+            assert DynamicEntity.class.getMethod(traitMethod.name, traitMethod.parameterTypes).isAnnotationPresent(Generated)
+        }
+    }
 }
 
 class DynamicEntity implements DynamicAttributes {
@@ -51,4 +60,4 @@ class DynamicEntity implements DynamicAttributes {
         "foo"
     }
 
-}
+}

grails-datamapping-core/src/main/groovy/grails/gorm/MultiTenant.groovy

@@ -20,7 +20,7 @@
 package grails.gorm
 
 import groovy.transform.CompileStatic
-
+import groovy.transform.Generated
 import grails.gorm.api.GormAllOperations
 import org.grails.datastore.gorm.GormEnhancer
 import org.grails.datastore.mapping.core.connections.ConnectionSource
@@ -41,6 +41,7 @@ trait MultiTenant<D> extends Entity {
      * @param callable The closure
      * @return The result of the closure
      */
+    @Generated
     static <T> T withTenant(Serializable tenantId, Closure<T> callable) {
         GormEnhancer.findStaticApi(this).withTenant(tenantId, callable)
     }
@@ -51,6 +52,7 @@ trait MultiTenant<D> extends Entity {
      * @param callable The closure
      * @return The result of the closure
      */
+    @Generated
     static <D> GormAllOperations eachTenant(Closure callable) {
         GormEnhancer.findStaticApi(this, ConnectionSource.DEFAULT).eachTenant(callable)
     }
@@ -61,6 +63,7 @@ trait MultiTenant<D> extends Entity {
      * @param tenantId The tenant id
      * @return The operations
      */
+    @Generated
     static <D> GormAllOperations<D> withTenant(Serializable tenantId) {
         (GormAllOperations<D>) GormEnhancer.findStaticApi(this).withTenant(tenantId)
     }

grails-datamapping-core/src/main/groovy/grails/gorm/time/InstantConverter.groovy

@@ -22,6 +22,7 @@ package grails.gorm.time
 import java.time.Instant
 
 import groovy.transform.CompileStatic
+import groovy.transform.Generated
 
 /**
  * A trait to convert a {@link java.time.Instant} to and from a long
@@ -31,10 +32,12 @@ import groovy.transform.CompileStatic
 @CompileStatic
 trait InstantConverter {
 
+    @Generated
     Long convert(Instant value) {
         value.toEpochMilli()
     }
 
+    @Generated
     Instant convert(Long value) {
         Instant.ofEpochMilli(value)
     }

grails-datamapping-core/src/main/groovy/grails/gorm/time/LocalDateConverter.groovy

@@ -27,6 +27,7 @@ import java.time.ZoneId
 import java.time.ZoneOffset
 
 import groovy.transform.CompileStatic
+import groovy.transform.Generated
 
 /**
  * A trait to convert a {@link LocalDate} to and from a long
@@ -37,12 +38,14 @@ import groovy.transform.CompileStatic
 trait LocalDateConverter extends TemporalConverter<LocalDate> {
 
     @Override
+    @Generated
     Long convert(LocalDate value) {
         LocalDateTime localDateTime = LocalDateTime.of(value, LocalTime.MIN)
         localDateTime.toInstant(ZoneOffset.UTC).toEpochMilli()
     }
 
     @Override
+    @Generated
     LocalDate convert(Long value) {
         Instant instant = Instant.ofEpochMilli(value)
         LocalDateTime.ofInstant(instant, ZoneId.of('UTC')).toLocalDate()

grails-datamapping-core/src/main/groovy/grails/gorm/time/LocalDateTimeConverter.groovy

@@ -25,6 +25,7 @@ import java.time.ZoneId
 import java.time.ZoneOffset
 
 import groovy.transform.CompileStatic
+import groovy.transform.Generated
 
 /**
  * A trait to convert a {@link java.time.LocalDateTime} to and from a long
@@ -35,11 +36,13 @@ import groovy.transform.CompileStatic
 trait LocalDateTimeConverter implements TemporalConverter<LocalDateTime> {
 
     @Override
+    @Generated
     Long convert(LocalDateTime value) {
         value.toInstant(ZoneOffset.UTC).toEpochMilli()
     }
 
     @Override
+    @Generated
     LocalDateTime convert(Long value) {
         LocalDateTime.ofInstant(Instant.ofEpochMilli(value), ZoneId.of('UTC'))
     }

grails-datamapping-core/src/main/groovy/grails/gorm/time/LocalTimeConverter.groovy

@@ -22,6 +22,7 @@ package grails.gorm.time
 import java.time.LocalTime
 
 import groovy.transform.CompileStatic
+import groovy.transform.Generated
 
 /**
  * A trait to convert a {@link LocalTime} to and from a long
@@ -32,11 +33,13 @@ import groovy.transform.CompileStatic
 trait LocalTimeConverter implements TemporalConverter<LocalTime> {
 
     @Override
+    @Generated
     Long convert(LocalTime value) {
         value.toNanoOfDay()
     }
 
     @Override
+    @Generated
     LocalTime convert(Long value) {
         LocalTime.ofNanoOfDay(value)
     }

grails-datamapping-core/src/main/groovy/grails/gorm/time/OffsetDateTimeConverter.groovy

@@ -23,6 +23,7 @@ import java.time.Instant
 import java.time.OffsetDateTime
 
 import groovy.transform.CompileStatic
+import groovy.transform.Generated
 
 /**
  * A trait to convert a {@link java.time.OffsetDateTime} to and from a long
@@ -33,11 +34,13 @@ import groovy.transform.CompileStatic
 trait OffsetDateTimeConverter implements TemporalConverter<OffsetDateTime> {
 
     @Override
+    @Generated
     Long convert(OffsetDateTime value) {
         value.toInstant().toEpochMilli()
     }
 
     @Override
+    @Generated
     OffsetDateTime convert(Long value) {
         OffsetDateTime.ofInstant(Instant.ofEpochMilli(value), systemOffset)
     }

grails-datamapping-core/src/main/groovy/grails/gorm/time/OffsetTimeConverter.groovy

@@ -24,6 +24,7 @@ import java.time.OffsetTime
 import java.time.ZoneOffset
 
 import groovy.transform.CompileStatic
+import groovy.transform.Generated
 
 /**
  * A trait to convert a {@link java.time.OffsetTime} to and from a long
@@ -34,11 +35,13 @@ import groovy.transform.CompileStatic
 trait OffsetTimeConverter implements TemporalConverter<OffsetTime> {
 
     @Override
+    @Generated
     Long convert(OffsetTime value) {
         value.withOffsetSameInstant(ZoneOffset.UTC).toLocalTime().toNanoOfDay()
     }
 
     @Override
+    @Generated
     OffsetTime convert(Long value) {
         OffsetTime.of(LocalTime.ofNanoOfDay(value), ZoneOffset.UTC).withOffsetSameInstant(systemOffset)
     }

grails-datamapping-core/src/main/groovy/grails/gorm/time/PeriodConverter.groovy

@@ -22,6 +22,7 @@ package grails.gorm.time
 import java.time.Period
 
 import groovy.transform.CompileStatic
+import groovy.transform.Generated
 
 /**
  * A trait to convert a {@link java.time.Period} to and from a String
@@ -31,10 +32,12 @@ import groovy.transform.CompileStatic
 @CompileStatic
 trait PeriodConverter {
 
+    @Generated
     String convert(Period value) {
         value.toString()
     }
 
+    @Generated
     Period convert(String value) {
         Period.parse(value)
     }