borinquenkid opened a new pull request, #16054:
URL: https://github.com/apache/grails-core/pull/16054

   Stacked on #16003.
   
   Addresses review comments from @jdaugherty:
   - Renames the Object-typed field to `untypedProperty` and rewords test names 
— the property is excluded because it is raw Object-typed 
(`DefaultASTDatabindingHelper#shouldFieldBeInWhiteList`), not because it is 
generically "unlisted".
   - Drops the misleading `hasOne` association, which added nothing to the 
tested contract and declares an incomplete mapping (no `belongsTo` 
back-reference on `WhitelistAddress`).
   - Trims the domain test to its non-duplicated assertions and 
cross-references `DefaultASTDatabindingHelperDomainClassSpecialPropertiesSpec`, 
which already pins id/version/dateCreated/lastUpdated exclusion for domain 
classes.
   
   ## Verification
   - `:grails-test-suite-web:test --tests 
DefaultDatabindingWhitelistBehaviorSpec`
   - `./gradlew codeStyle`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to