codeconsole commented on PR #15954: URL: https://github.com/apache/grails-core/pull/15954#issuecomment-4937443170
@borinquenkid done in 43e59b4 — two tests added: - The 2 missing lines were `storeAuditorAvailability`'s body, which nothing in this module's spec executed. There's now a test registering `@CreatedBy`/`@LastModifiedBy` properties via `persistentEntityAdded()` and asserting the auditor is populated on insert/update. - The 2 partial branches were the `isEmpty()` legs of the `TimestampSuppression` constructor ternaries, reachable only by capturing inside a window opened with an empty class list — now tested, along with the (previously unpinned) behavior that an empty class list disables nothing. Verified locally with JaCoCo: both lines fully executed, 4/4 branches covered on both ternaries. Codecov should report 100% patch coverage on the next run. -- 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]
