feat: Add API for single pull request scan

[o-kopysov]

Pull Request

Description

Current PR includes a new REST API for a single pull request scan.
Also, it contains code refactoring and UT updates.

Fixes #354

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Code cleanup/refactoring
• Documentation update
• This change requires a documentation update
• CI system update
• Test Coverage update

Testing

http://localhost:7896/scan/github.com/samsung/lpvs/370

Checklist:

• My code follows the style guidelines of this project
• My code meets the required code coverage for lines (90% and above)
• My code meets the required code coverage for branches (80% and above)
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

[codecov-commenter]

Codecov Report

Attention: 15 lines in your changes are missing coverage. Please review.

Comparison is base (f7ac60c) 90.94% compared to head (a660f71) 90.90%.

Files	Patch %	Lines
src/main/java/com/lpvs/util/LPVSWebhookUtil.java	40.00%	4 Missing and 5 partials ⚠️
...ain/java/com/lpvs/controller/GitHubController.java	84.37%	4 Missing and 1 partial ⚠️
.../main/java/com/lpvs/service/LPVSGitHubService.java	93.75%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #370      +/-   ##
============================================
- Coverage     90.94%   90.90%   -0.04%     
+ Complexity      487      486       -1     
============================================
  Files            47       47              
  Lines          1700     1737      +37     
  Branches        206      209       +3     
============================================
+ Hits           1546     1579      +33     
+ Misses           97       94       -3     
- Partials         57       64       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[o-kopysov]

[License Pre-Validation Service] No license issue detected

Detected licenses:

File: src/test/java/com/lpvs/controller/GitHubControllerTest.java
License(s): MIT (permitted)
Component: LPVS (test/java/com/lpvs/controller/GitHubWebhooksControllerTest.java)
Matched Lines: 9-83
Snippet Match: 47%

[o-kopysov]

JaCoCo Test Coverage Summary

• Coverage: 92.920%
• Branches: 80.442%

[m-rudyk]

Approved.

[tiokim]

LGTM
Is it possible to add the job for running the single scan along with Samsung/LPVS/{PR number} to the webhook file for ease of verification?

[o-kopysov]

LGTM Is it possible to add the job for running the single scan along with Samsung/LPVS/{PR number} to the webhook file for ease of verification?

We already have a plan to implement GitHub action for LPVS: #319
Or did you mean something else?

[tiokim]

LGTM Is it possible to add the job for running the single scan along with Samsung/LPVS/{PR number} to the webhook file for ease of verification?

We already have a plan to implement GitHub action for LPVS: #319 Or did you mean something else?

I want to see if the function works well through the workflow for every PR.

[o-kopysov]

I want to see if the function works well through the workflow for every PR.

Current single scan API is implemented for "server mode" of LPVS operation. To check its functionality, we need to run LPVS server somewhere.

[o-kopysov]

[License Pre-Validation Service] No license issue detected

Detected licenses:

File: src/main/java/com/lpvs/service/LPVSQueueService.java
License(s): MIT (permitted)
Component: LPVS (main/java/com/lpvs/service/LPVSQueueService.java)
Matched Lines: 193-218
Snippet Match: 11%

File: src/test/java/com/lpvs/entity/enums/LPVSPullRequestActionTest.java
License(s): MIT (permitted)
Component: LPVS (test/java/com/lpvs/entity/enums/LPVSPullRequestActionTest.java)
Matched Lines: 2-51
Snippet Match: 96%

[o-kopysov]

[License Pre-Validation Service] No license issue detected

Detected licenses:

File: src/main/java/com/lpvs/service/LPVSQueueService.java License(s): MIT (permitted) Component: LPVS (main/java/com/lpvs/service/LPVSQueueService.java) Matched Lines: 193-218 Snippet Match: 11%

File: src/test/java/com/lpvs/entity/enums/LPVSPullRequestActionTest.java License(s): MIT (permitted) Component: LPVS (test/java/com/lpvs/entity/enums/LPVSPullRequestActionTest.java) Matched Lines: 2-51 Snippet Match: 96%

Here is a result of single scan execution

[tiokim]

[License Pre-Validation Service] No license issue detected
Detected licenses:
File: src/main/java/com/lpvs/service/LPVSQueueService.java License(s): MIT (permitted) Component: LPVS (main/java/com/lpvs/service/LPVSQueueService.java) Matched Lines: 193-218 Snippet Match: 11%
File: src/test/java/com/lpvs/entity/enums/LPVSPullRequestActionTest.java License(s): MIT (permitted) Component: LPVS (test/java/com/lpvs/entity/enums/LPVSPullRequestActionTest.java) Matched Lines: 2-51 Snippet Match: 96%

Here is a result of single scan execution

Thanks for the hard work. It seems good to merge now!