Enhances the GitHub verify workflow to run Ruby, Java and Go implementations of the recog fingerprint verification tool on any modified XML fingerprint files. This will help contributors and pull request reviewers verify at a minimum that each fingerprint example is matched and all parameters that are defined by capture groups have the expected value. Note, warnings are currently disabled in the tools that provide warnings (Ruby and Java), and will be enabled once all of the warnings in the existing fingerprints are corrected.
Motivation and Context
Help contributors and pull request reviewers catch concerns.
How Has This Been Tested?
Opened PR against my recog fork
Patched xml/ fingerprints to introduce issues
Committed and pushed changes
Confirmed Ruby Verify, Java Verify, and Go Verify workflow jobs all failed
Committed and pushed changes to undo the changes applied by the patch
Confirmed all Verify workflows were not run since xml files under xml/ were no longer different
Types of changes
New feature (non-breaking change which adds functionality)
Checklist:
[x] I have updated the documentation accordingly (or changes are not required).
[x] I have added tests to cover my changes (or new tests are not required).
Description
Enhances the GitHub verify workflow to run Ruby, Java and Go implementations of the recog fingerprint verification tool on any modified XML fingerprint files. This will help contributors and pull request reviewers verify at a minimum that each fingerprint example is matched and all parameters that are defined by capture groups have the expected value. Note, warnings are currently disabled in the tools that provide warnings (Ruby and Java), and will be enabled once all of the warnings in the existing fingerprints are corrected.
Motivation and Context
Help contributors and pull request reviewers catch concerns.
How Has This Been Tested?
xml/
fingerprints to introduce issuesxml/
were no longer differentTypes of changes
Checklist: