[AUTHZ] Check Authz plugin's spec json files in UT

[packyan]

Why are the changes needed?

to close #4715

How was this patch tested?

• Add some test cases that check the changes thoroughly including negative and positive cases if possible

• Add screenshots for manual tests if appropriate

• Run test locally before make a pull request

[codecov-commenter]

Codecov Report

Merging #4717 (88e70da) into master (2ed0990) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4717      +/-   ##
============================================
- Coverage     58.06%   58.02%   -0.04%     
  Complexity       13       13              
============================================
  Files           581      581              
  Lines         32297    32297              
  Branches       4313     4313              
============================================
- Hits          18753    18741      -12     
- Misses        11746    11755       +9     
- Partials       1798     1801       +3     

see 7 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[bowenliang123]

Hi, the spec json file should not be regenerated in each build, by design. But yes, it should be checked during ci builds.

[packyan]

Hi, the spec json file should not be regenerated in each build, by design. But yes, it should be checked during ci builds.

Yes, I will add a UT to check generated golden files.

[bowenliang123]

Alright, I mean change it from executing the main method to running it in test suites, for both validation in testing and regeneration, just to be clear.

[packyan]

Alright, I mean change it from executing the main method to running it in test suites, for both validation in testing and regeneration, just to be clear.

understood

[bowenliang123]

Would you like to remove redundant changes in generator and policy json file of #4716 which is already merged ? @packyan

[packyan]

Would you like to remove redundant changes in generator and policy json file of #4716 which is already merged ? @packyan

Done, wait for CI finished.

[bowenliang123]

LGTM overall. Minor comment.

[bowenliang123]

Thanks for your continuous contribution. Merged to master.