Implement "author" command argument for diffChangelog command

[vitaliyg2]

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

Currently there is no way to specify changesets author for changesets generated by the diffChangelog command.
This implements support of the author command argument for the diffChangelog command, same way as it's implemented for the generateChangelog command.

This allows to specify author for diffChangelog in properties file or from the command line, also implemented changeSetAuthor property for Maven plugin (same way as it is implemented for generateChangelog command).

Fixes #1610

Things to be aware of

Implementation is based on the generateChangelog command.

[nvoxland]

Initial/Pre-Review Thoughts

Changes make sense to me. Thanks!

Questions I have:

• None

Potential risks:

• None

What could make the full review difficult:

• Nothing

[filipelautert]

Hello @vitaliyg2 , thanks for the improvement!
I fixed the current tests, but do you think you could add a new test for this change at diffChangelog.test.groovy ?

[vitaliyg2]

Hi @filipelautert,
I'm not sure if it worth creating new test for this (it will be practically identical copy of the existing test) , for now I just updated one of the existing tests to also check for if author argument is working correctly.

[filipelautert]

Thanks @vitaliyg2 , Adding the parameter to a test seems enough.

[XDelphiGrl]

The author property for generate-changelog is hidden from the help. Given that we're expanding the usage of author to diff-changelog, the author property should be visible in help for both commands.

A quick test did verify that despite not appearing in the help, generate-changelog accepts and uses the author property.

CHANGES REQUESTED

[filipelautert]

Thanks @vitaliyg2 for unhiding author field for generate-changelog!

[XDelphiGrl]

Functional tests started here: https://github.com/liquibase/liquibase-pro-tests/actions/runs/4993639743

[XDelphiGrl]

Did you ever wish you could generate a changelog or diff-changelog and specify an author name, instead of relying on the autogenerated author name? Now you can! This PR introduces a new diff-changelog command property author that Liquibase uses to popluate the author attribute on generated changesets. Generate-changelog already implemented the author command property, but it was hidden from the help. The property is now correctly documented in the CLI help for both diff-changelog and generate-changelog.

• Integration tests expanded to validate the new functionality.
• Light manual testing of diff-changelog --author customAuth for all changelog formats passing.
• Test harness failures are unrelated to this PR and can be disregarded.
• Functional tests passing.

APPROVED

[unlimitedsola]

Hi All, big thanks for adding this feature ❤️. In my original feature request, I only mentioned that the author was missing as an argument for the generated changelogs, but I can see supporting the context option could also be a useful addition too. Would you mind if I follow the steps of this PR and also add the support for context?

[filipelautert]

Hello @unlimitedsola ! Sure, please feel free to create a new pr. The more features the merrier :)