gustavoharff opened a new pull request, #15214: URL: https://github.com/apache/grails-core/pull/15214
Follow up #15150 Jacoco isn't displaying coverage lines on methods annotated with Transactional (or other annotation that apply AST transformation). I think that has been applied to display a more clean list of methods in the tree, but I guess the coverage lines is more important than that. I did some tests and below you can see the results. ### Current No coverage lines are displayed. ```groovy markAsGenerated(classNode, renamedMethodNode) ``` <img width="450" alt="image" src="https://github.com/user-attachments/assets/be779612-05d7-444c-ada3-a9bde1f5dcd0"; /> ### Marking the original method as generated I don't think this is the best way, it feels counterintuitive to me. ```groovy markAsGenerated(classNode, methodNode) ``` <img width="450" alt="image" src="https://github.com/user-attachments/assets/5d2796e7-26be-4582-8c74-cb84c79abe91"; /> ### Not marking methods as generated It has a larger methods list than other ways. <img width="450" alt="image" src="https://github.com/user-attachments/assets/6a65c113-2753-4d89-aeaa-7e76df68dbe6"; /> --- I chose not to mark the methods as generated, let me know what you think. -- 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]
