Bring update summary options to all update-related goals of the maven plugin

[KeepItSimpleStupid]

Impact

• Bug fix (non-breaking change which fixes expected existing functionality)
• Enhancement/New feature (adds functionality without impacting existing logic)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

#4395 and #5077 brought the options showSummary and showSummaryOutput to the update goal of the maven plugin.

These options seem to be relevant for the updateSQL and updateTestingRollback goals as well, so this PR brings them to the parent abstract class of all these goals

Things to be aware of

Things to worry about

Additional Context

[MalloD12]

Approved.

Code change looks good to me. This is not applicable for updateSQL, but it is for update and upateTestingRollback. I manually tested the change with these last two commands and worked correctly.

Thanks for the PR,
Daniel.

[KeepItSimpleStupid]

Thanks @MalloD12, the pleasure is mine : liquibase is a great tool ! :)

Since the showSummary and showSummaryOutput are not applicable for the updateSQL goal, do you want me to rework the PR to acknowledge that ?
i.e. duplicate the declaration & logic of these 2 fields in LiquibaseUpdateTestingRollback instead of moving it to AbstractLiquibaseUpdateMojo ?

[MalloD12]

Thanks @MalloD12, the pleasure is mine : liquibase is a great tool ! :)

Since the showSummary and showSummaryOutput are not applicable for the updateSQL goal, do you want me to rework the PR to acknowledge that ? i.e. duplicate the declaration & logic of these 2 fields in LiquibaseUpdateTestingRollback instead of moving it to AbstractLiquibaseUpdateMojo ?

Hi @KeepItSimpleStupid,

I think that's not needed. Let's leave it as it is. Having it in the AbstractLiquibaseUpdateMojo is ok and that's pretty much how we handle it for CLI having all the showSummary logic in the AbtractUpdateCommandStep class.

Thanks,
Daniel.