refactor: Rename repository methods for name-based query derivation

Signed-off-by: Oleg Kopysov <[email protected]>

src/main/java/com/lpvs/controller/LPVSWebController.java

@@ -242,7 +242,7 @@ public ResponseEntity<HistoryEntity> newHistoryPageByUser(
                 // Validate and sanitize user inputs to prevent XSS attacks
                 sanitizeUserInputs(pr);
 
-                Boolean hasIssue = detectedLicenseRepository.existsIssue(pr);
+                Boolean hasIssue = detectedLicenseRepository.existsByIssueIsTrueAndPullRequest(pr);
 
                 lpvsHistories.add(
                         new LPVSHistory(
@@ -289,11 +289,11 @@ public ResponseEntity<LPVSResult> resultPage(
             LPVSPullRequest pr = prOpt.get();
 
             List<LPVSLicense> distinctByLicense =
-                    detectedLicenseRepository.findDistinctByLicense(pr);
+                    detectedLicenseRepository.findDistinctByPullRequestAndLicense(pr);
             List<String> detectedLicenses = new ArrayList<>();
             Map<String, Integer> licenseCountMap = new HashMap<>();
 
-            List<String> allSpdxId = licenseRepository.takeAllSpdxId();
+            List<String> allSpdxId = licenseRepository.findAllSpdxId();
             for (String spdxId : allSpdxId) {
                 licenseCountMap.put(spdxId, 0);
             }
@@ -313,7 +313,7 @@ public ResponseEntity<LPVSResult> resultPage(
                     detectedLicenseRepository.findByPullRequest(pr, pageable);
             List<LPVSDetectedLicense> dlList = detectedLicenseRepository.findByPullRequest(pr);
             List<LPVSResultFile> lpvsResultFileList = new ArrayList<>();
-            Boolean hasIssue = detectedLicenseRepository.existsIssue(pr);
+            Boolean hasIssue = detectedLicenseRepository.existsByIssueIsTrueAndPullRequest(pr);
 
             String licenseSpdxId;
             String status;
@@ -343,7 +343,7 @@ public ResponseEntity<LPVSResult> resultPage(
                 }
             }
 
-            Long count = detectedLicenseRepository.CountByDetectedLicenseWherePullRequestId(pr);
+            Long count = detectedLicenseRepository.countByPullRequestAndLicenseIsNotNull(pr);
             String[] tempPullNumber = pr.getPullRequestUrl().split("/");
             LPVSResult lpvsResult =
                     new LPVSResult(

src/main/java/com/lpvs/repository/LPVSDetectedLicenseRepository.java

@@ -12,7 +12,6 @@
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
 import org.springframework.data.jpa.repository.JpaRepository;
-import org.springframework.data.jpa.repository.Query;
 import org.springframework.data.repository.query.Param;
 
 import java.util.List;
@@ -28,10 +27,7 @@ public interface LPVSDetectedLicenseRepository extends JpaRepository<LPVSDetecte
      * @param pr The {@link LPVSPullRequest} to check.
      * @return {@code true} if at least one detected license with an issue is associated with the pull request, {@code false} otherwise.
      */
-    @Query(
-            value =
-                    "select count(dl)>0 from LPVSDetectedLicense dl where dl.issue = True and dl.pullRequest = :pr")
-    Boolean existsIssue(@Param("pr") LPVSPullRequest pr);
+    Boolean existsByIssueIsTrueAndPullRequest(@Param("pr") LPVSPullRequest pr);
 
     /**
      * Find all detected licenses associated with a specific pull request.
@@ -56,30 +52,21 @@ public interface LPVSDetectedLicenseRepository extends JpaRepository<LPVSDetecte
      * @param pr The {@link LPVSPullRequest} to count detected licenses for.
      * @return The count of detected licenses where the license is not null.
      */
-    @Query(
-            value =
-                    "select count(*) from LPVSDetectedLicense dl where dl.pullRequest = :pr and dl.license is not null")
-    Long CountByDetectedLicenseWherePullRequestId(@Param("pr") LPVSPullRequest pr);
+    Long countByPullRequestAndLicenseIsNotNull(@Param("pr") LPVSPullRequest pr);
 
     /**
      * Find distinct licenses associated with a specific pull request.
      *
      * @param pr The {@link LPVSPullRequest} to retrieve distinct licenses for.
      * @return List of distinct {@link LPVSLicense} associated with the pull request.
      */
-    @Query(
-            value =
-                    "select distinct dl.license from LPVSDetectedLicense dl where dl.pullRequest = :pr")
-    List<LPVSLicense> findDistinctByLicense(@Param("pr") LPVSPullRequest pr);
+    List<LPVSLicense> findDistinctByPullRequestAndLicense(@Param("pr") LPVSPullRequest pr);
 
     /**
      * Find detected licenses associated with a specific pull request where the license is not null.
      *
      * @param pr The {@link LPVSPullRequest} to retrieve detected licenses for.
      * @return List of {@link LPVSDetectedLicense} associated with the pull request where the license is not null.
      */
-    @Query(
-            value =
-                    "select dl from LPVSDetectedLicense dl where dl.pullRequest = :pr and dl.license is not null")
-    List<LPVSDetectedLicense> findNotNullDLByPR(@Param("pr") LPVSPullRequest pr);
+    List<LPVSDetectedLicense> findByPullRequestAndLicenseIsNotNull(@Param("pr") LPVSPullRequest pr);
 }

src/main/java/com/lpvs/repository/LPVSLicenseConflictRepository.java

@@ -12,7 +12,6 @@
 import org.springframework.data.repository.query.Param;
 import org.springframework.stereotype.Repository;
 
-import java.util.List;
 
 /**
  * Repository interface for managing {@link LPVSLicenseConflict} entities.
@@ -21,14 +20,6 @@
 @Repository
 public interface LPVSLicenseConflictRepository extends JpaRepository<LPVSLicenseConflict, Long> {
 
-    /**
-     * Retrieve all license conflicts from the database.
-     *
-     * @return List of {@link LPVSLicenseConflict} entities representing license conflicts.
-     */
-    @Query("SELECT lc FROM LPVSLicenseConflict lc")
-    List<LPVSLicenseConflict> takeAllLicenseConflicts();
-
     /**
      * Find a specific license conflict between two licenses.
      *

src/main/java/com/lpvs/repository/LPVSLicenseRepository.java

@@ -21,22 +21,13 @@
 @Repository
 public interface LPVSLicenseRepository extends JpaRepository<LPVSLicense, Long> {
 
-    /**
-     * Retrieve all licenses from the database.
-     *
-     * @return List of {@link LPVSLicense} entities representing licenses.
-     */
-    @Query("SELECT l FROM LPVSLicense l")
-    List<LPVSLicense> takeAllLicenses();
-
     /**
      * Search for a license by SPDX identifier.
      *
      * @param spdxId The SPDX identifier of the license.
      * @return The latest {@link LPVSLicense} entity with the specified SPDX identifier.
      */
-    @Query("SELECT l FROM LPVSLicense l WHERE l.spdxId = :spdxId ORDER BY l.id DESC LIMIT 1")
-    LPVSLicense searchBySpdxId(@Param("spdxId") String spdxId);
+    LPVSLicense findFirstBySpdxIdOrderByIdDesc(@Param("spdxId") String spdxId);
 
     /**
      * Search for a license by alternative license names.
@@ -54,6 +45,5 @@ public interface LPVSLicenseRepository extends JpaRepository<LPVSLicense, Long>
      *
      * @return List of SPDX identifiers as Strings.
      */
-    @Query(value = "select licenses.spdxId from LPVSLicense licenses")
-    List<String> takeAllSpdxId();
+    List<String> findAllSpdxId();
 }

src/main/java/com/lpvs/repository/LPVSPullRequestRepository.java

@@ -60,7 +60,6 @@ LPVSPullRequest findLatestByPullRequestInfo(
      * @param pageable The pagination information.
      * @return Page of {@link LPVSPullRequest} entities with the specified base name.
      */
-    @Query(value = "select pr from LPVSPullRequest pr where pr.pullRequestBase = :name")
     Page<LPVSPullRequest> findByPullRequestBase(@Param("name") String name, Pageable pageable);
 
     /**
@@ -69,7 +68,6 @@ LPVSPullRequest findLatestByPullRequestInfo(
      * @param name The name of the pull request base.
      * @return List of {@link LPVSPullRequest} entities with the specified base name.
      */
-    @Query(value = "select pr from LPVSPullRequest pr where pr.pullRequestBase = :name")
     List<LPVSPullRequest> findByPullRequestBase(@Param("name") String name);
 
     /**
@@ -78,8 +76,7 @@ LPVSPullRequest findLatestByPullRequestInfo(
      * @param name The name of the pull request base.
      * @return The count of pull requests with the specified base name.
      */
-    @Query(value = "select count(*) from LPVSPullRequest pr where pr.pullRequestBase = :name")
-    Long CountByPullRequestBase(@Param("name") String name);
+    Long countByPullRequestBase(@Param("name") String name);
 
     /**
      * Find all pull requests with the specified sender or pull request head, paginated.
@@ -88,9 +85,6 @@ LPVSPullRequest findLatestByPullRequestInfo(
      * @param pageable The pagination information.
      * @return Page of {@link LPVSPullRequest} entities with the specified sender or pull request head.
      */
-    @Query(
-            value =
-                    "select pr from LPVSPullRequest pr where pr.sender = :name or pr.pullRequestHead = :name")
     Page<LPVSPullRequest> findBySenderOrPullRequestHead(
             @Param("name") String name, Pageable pageable);
 
@@ -100,9 +94,6 @@ Page<LPVSPullRequest> findBySenderOrPullRequestHead(
      * @param name The name of the sender or pull request head.
      * @return List of {@link LPVSPullRequest} entities with the specified sender or pull request head.
      */
-    @Query(
-            value =
-                    "select pr from LPVSPullRequest pr where pr.sender = :name or pr.pullRequestHead = :name")
     List<LPVSPullRequest> findBySenderOrPullRequestHead(@Param("name") String name);
 
     /**
@@ -111,8 +102,5 @@ Page<LPVSPullRequest> findBySenderOrPullRequestHead(
      * @param name The name of the sender or pull request head.
      * @return The count of pull requests with the specified sender or pull request head.
      */
-    @Query(
-            value =
-                    "select count(*) from LPVSPullRequest pr where pr.sender = :name or pr.pullRequestHead = :name")
-    Long CountBySenderOrPullRequestHead(@Param("name") String name);
+    Long countBySenderOrPullRequestHead(@Param("name") String name);
 }

src/main/java/com/lpvs/repository/LPVSQueueRepository.java

@@ -8,21 +8,10 @@
 
 import com.lpvs.entity.LPVSQueue;
 import org.springframework.data.jpa.repository.JpaRepository;
-import org.springframework.data.jpa.repository.Query;
 
-import java.util.List;
 
 /**
  * Repository interface for managing {@link LPVSQueue} entities.
  * Extends {@link org.springframework.data.jpa.repository.JpaRepository} for basic CRUD operations.
  */
-public interface LPVSQueueRepository extends JpaRepository<LPVSQueue, Long> {
-
-    /**
-     * Retrieve the list of all entities from the "lpvs_queue" table.
-     *
-     * @return List of {@link LPVSQueue} entities representing the queue.
-     */
-    @Query("SELECT q FROM LPVSQueue q")
-    List<LPVSQueue> getQueueList();
-}
+public interface LPVSQueueRepository extends JpaRepository<LPVSQueue, Long> {}

src/main/java/com/lpvs/service/LPVSGitHubService.java

@@ -307,13 +307,19 @@ public void commentResults(
                 commitCommentBuilder.append("<li>" + conflict.l1 + " and " + conflict.l2 + "</li>");
                 LPVSDetectedLicense detectedIssue = new LPVSDetectedLicense();
                 detectedIssue.setPullRequest(lpvsPullRequest);
-                Long l1 = lpvsLicenseRepository.searchBySpdxId(conflict.l1).getLicenseId();
-                Long l2 = lpvsLicenseRepository.searchBySpdxId(conflict.l2).getLicenseId();
+                Long l1 =
+                        lpvsLicenseRepository
+                                .findFirstBySpdxIdOrderByIdDesc(conflict.l1)
+                                .getLicenseId();
+                Long l2 =
+                        lpvsLicenseRepository
+                                .findFirstBySpdxIdOrderByIdDesc(conflict.l2)
+                                .getLicenseId();
                 detectedIssue.setLicenseConflict(
                         lpvsLicenseConflictRepository.findLicenseConflict(l1, l2));
                 if (webhookConfig.getRepositoryLicense() != null) {
                     LPVSLicense repoLicense =
-                            lpvsLicenseRepository.searchBySpdxId(
+                            lpvsLicenseRepository.findFirstBySpdxIdOrderByIdDesc(
                                     webhookConfig.getRepositoryLicense());
                     if (repoLicense == null) {
                         repoLicense =

src/main/java/com/lpvs/service/LPVSLicenseService.java

@@ -118,8 +118,7 @@ public LPVSLicenseService(
      * Load all license conflicts from the database.
      */
     private void loadLicenseConflicts() {
-        List<LPVSLicenseConflict> conflicts =
-                lpvsLicenseConflictRepository.takeAllLicenseConflicts();
+        List<LPVSLicenseConflict> conflicts = lpvsLicenseConflictRepository.findAll();
         for (LPVSLicenseConflict conflict : conflicts) {
             Conflict<String, String> conf =
                     new Conflict<>(
@@ -155,7 +154,7 @@ private void init() {
         } else {
             try {
                 // 1. Load licenses from DB
-                licenses = lpvsLicenseRepository.takeAllLicenses();
+                licenses = lpvsLicenseRepository.findAll();
                 // print info
                 log.info("LICENSES: loaded " + licenses.size() + " licenses from DB.");
 
@@ -186,7 +185,7 @@ private void init() {
      */
     public void reloadFromTables() {
         if (licenses.isEmpty()) {
-            licenses = lpvsLicenseRepository.takeAllLicenses();
+            licenses = lpvsLicenseRepository.findAll();
             log.info("RELOADED " + licenses.size() + " licenses from DB.");
 
             loadLicenseConflicts();
@@ -376,7 +375,8 @@ public List<Conflict<String, String>> findConflicts(
         String repositoryLicense = webhookConfig.getRepositoryLicense();
 
         if (repositoryLicense != null) {
-            LPVSLicense repoLicense = lpvsLicenseRepository.searchBySpdxId(repositoryLicense);
+            LPVSLicense repoLicense =
+                    lpvsLicenseRepository.findFirstBySpdxIdOrderByIdDesc(repositoryLicense);
             if (repoLicense == null) {
                 repoLicense =
                         lpvsLicenseRepository.searchByAlternativeLicenseNames(repositoryLicense);

src/main/java/com/lpvs/service/LPVSLoginCheckService.java

@@ -119,10 +119,10 @@ public HistoryPageEntity pathCheck(
         if ((type.equals("own") && findNickName.equals(name))
                 || (type.equals("org") && findOrganization.equals(name))) {
             prPage = lpvsPullRequestRepository.findByPullRequestBase(name, pageable);
-            count = lpvsPullRequestRepository.CountByPullRequestBase(name);
+            count = lpvsPullRequestRepository.countByPullRequestBase(name);
         } else if (type.equals("send") && findNickName.equals(name)) {
             prPage = lpvsPullRequestRepository.findBySenderOrPullRequestHead(name, pageable);
-            count = lpvsPullRequestRepository.CountBySenderOrPullRequestHead(name);
+            count = lpvsPullRequestRepository.countBySenderOrPullRequestHead(name);
         } else {
             throw new WrongAccessException("WrongAccessException");
         }

src/main/java/com/lpvs/service/LPVSQueueService.java

@@ -150,7 +150,7 @@ public BlockingDeque<LPVSQueue> getQueue() {
     public void checkForQueue() throws InterruptedException {
         QUEUE.clear();
         log.debug("Checking for previous queue");
-        List<LPVSQueue> webhookConfigList = queueRepository.getQueueList();
+        List<LPVSQueue> webhookConfigList = queueRepository.findAll();
         for (LPVSQueue webhook : webhookConfigList) {
             log.info("Add WebHook id = " + webhook.getId() + " to the queue.");
             QUEUE.putFirst(webhook);

src/main/java/com/lpvs/service/LPVSStatisticsService.java

@@ -128,7 +128,7 @@ public Dashboard getDashboardEntity(String type, String name, Authentication aut
         List<DashboardElementsByDate> dashboardByDates = new ArrayList<>();
         Map<LocalDate, List<LPVSPullRequest>> datePrMap = new HashMap<>();
 
-        List<String> allSpdxId = lpvsLicenseRepository.takeAllSpdxId();
+        List<String> allSpdxId = lpvsLicenseRepository.findAllSpdxId();
         for (String spdxId : allSpdxId) {
             licenseCountMap.put(spdxId, 0);
         }
@@ -159,7 +159,7 @@ public Dashboard getDashboardEntity(String type, String name, Authentication aut
             Set<String> senderSet = new HashSet<>();
             for (LPVSPullRequest pr : entry.getValue()) {
                 List<LPVSDetectedLicense> dlList =
-                        lpvsDetectedLicenseRepository.findNotNullDLByPR(pr);
+                        lpvsDetectedLicenseRepository.findByPullRequestAndLicenseIsNotNull(pr);
                 if (!(pr.getRepositoryName() == null || pr.getRepositoryName().isEmpty())) {
                     senderSet.add(pr.getSender());
                 }

src/test/java/com/lpvs/controller/LPVSWebControllerTest.java

@@ -167,7 +167,8 @@ public void testNewHistoryPageByUser() {
 
         when(loginCheckService.pathCheck(type, name, pageable, authentication))
                 .thenReturn(new HistoryPageEntity(prPage, 1L));
-        when(detectedLicenseRepository.existsIssue(pullRequest)).thenReturn(false);
+        when(detectedLicenseRepository.existsByIssueIsTrueAndPullRequest(pullRequest))
+                .thenReturn(false);
 
         HistoryEntity result =
                 webController.new WebApiEndpoints()
@@ -229,12 +230,13 @@ public void testResultPage() {
 
         when(loginCheckService.getMemberFromMemberMap(authentication)).thenReturn(new LPVSMember());
         when(lpvsPullRequestRepository.findById(prId)).thenReturn(Optional.of(pullRequest));
-        when(detectedLicenseRepository.findDistinctByLicense(pullRequest)).thenReturn(licenses);
-        when(licenseRepository.takeAllSpdxId()).thenReturn(List.of("MIT", "Apache-2.0"));
+        when(detectedLicenseRepository.findDistinctByPullRequestAndLicense(pullRequest))
+                .thenReturn(licenses);
+        when(licenseRepository.findAllSpdxId()).thenReturn(List.of("MIT", "Apache-2.0"));
         when(detectedLicenseRepository.findByPullRequest(pullRequest, pageable))
                 .thenReturn(new PageImpl<>(detectedLicenses));
         when(detectedLicenseRepository.findByPullRequest(pullRequest)).thenReturn(detectedLicenses);
-        when(detectedLicenseRepository.CountByDetectedLicenseWherePullRequestId(pullRequest))
+        when(detectedLicenseRepository.countByPullRequestAndLicenseIsNotNull(pullRequest))
                 .thenReturn(1L);
 
         LPVSResult result =