NIFI-15960 Replaced current property in AvroSchemaRegistry with a new one which utilizes a validation strategy where it details what Avro validations occur when choosing strict validation of Avro schemas.

[dan-s1]

Summary

NIFI-15960

This PR primarily details the various validations which occur when strict validation is chosen for validating Avro schemas in the AvroSchemaRegistry. The unit tests were overhauled to use the NIFI testing framework and multiple unit tests were added especially to contrast the difference between strict and lenient validation of Avro schemas.

Tracking

Please complete the following tracking steps prior to pull request creation.

Issue Tracking

• Apache NiFi Jira issue created

Pull Request Tracking

• Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
• Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000
• Pull request contains commits signed with a registered key indicating Verified status

Pull Request Formatting

• Pull Request based on current revision of the main branch
• Pull Request refers to a feature branch with one commit containing changes

Verification

Please indicate the verification steps performed prior to pull request creation.

Build

• Build completed using ./mvnw clean install -P contrib-check
  • JDK 21
  • JDK 25

Licensing

• New dependencies are compatible with the Apache License 2.0 according to the License Policy
• New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

• Documentation formatting appears as expected in rendered files

[pvillard31]

Should removeProperty(obsoletePropertyName) be guarded by hasProperty(obsoletePropertyName) (or the loop broken after the first match), so that the migration result only reports the property that was actually present?

[dan-s1]

Javadoc on remove is
Removes the property with the given name, if it exists. If the property does not exist, this is a no-op.

Why would it be necessary then to be guarded by hasProperty(obsoletePropertyName) (or the loop broken after the first match)?

[dan-s1]

@pvillard31 Are you okay if this is left as is?

[dan-s1]

@pvillard31 Thank you very much for approving my other PRs. Can you please take another look at this PR? My teammates are interested in having this documentation so it is clear what parts of the Avro schema are validated. Thanks!