GHA-82 Add UTs for SubmitReview

[pavel-mikula-sonarsource]

GHA-82

Part of

[sonarqubecloud]

🤖 Pull Request summary

Adds comprehensive unit tests for SubmitReview functionality.

• New test file: Creates complete test suite for SubmitReview with 5 test scenarios
• Review state handling: Tests different PR review states (commented, approved, changes_requested)
• JIRA integration: Verifies proper JIRA issue transitions and reviewer tracking for eng-xp-squad
• Test infrastructure: Updates JiraClientStub with new addReviewedBy method

Focus areas for review:

• Test coverage completeness for all review state paths
• Mock setup accuracy, especially the GitHub context payload structure
• Email lookup handling in the eng-xp-squad scenario edge cases

💬 Please send your feedback

Quality Gate passed

Issues
0 New issues
3 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[mary-georgiou-sonarsource]

These tests are valid if the dev has asked review from more than one person?
I mean one approval, request for changes should be enough to trigger the action right?

Just sanity checking the logic.

[pavel-mikula-sonarsource]

It's triggered one by one, probably far apart in time. First comes is first served. The last review wins the state.

So if one reviewer approves, one requests changes, it depends on what happened last. And it will be moved there. And can be fixed manually if needed.