feat: jobs check enhancement

[acelebanski]

Description

I modified the method check_non_finished_jobs in CheckFirewall class and renamed it to check_jobs. Now it allows to pass some parameters (job_type, job_status, job_result) to the method and now we can query any of those 3 keys values.

So we can not only check whether the status is FIN but also if the type is Commit or the result is OK (or any different combination). Default parameters check only for status of FIN, so method's default behaviour remains the same.

Motivation and Context

#161

How Has This Been Tested?

Unit tests within the repo.

Types of changes

• New feature (non-breaking change which adds functionality)

Checklist

• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes if appropriate.
• All new and existing tests passed.

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
panos_upgrade_assurance
check_firewall.py					1072-1074, 1091
Project Total

This report was generated by python-coverage-comment-action

[github-actions]

A Preview PR in PanDev repo has been created. You can view it here.

[acelebanski]

We figured out that original check_non_finished_jobs method actually does the job (we don't have to check the result for PEND value because that equals to status other than FIN).