fix: fix signlescan option

Signed-off-by: Mykola Rudyk <m.rudyk@samsung.com>

src/main/java/com/lpvs/entity/LPVSDetectedLicense.java

@@ -17,7 +17,7 @@
  * This class is mapped to the "detected_license" table in the "lpvs" schema.
  */
 @Entity
-@Table(name = "detected_license", schema = "${lpvs.schema:lpvs}")
+@Table(name = "detected_license")
 @Getter
 @Setter
 public class LPVSDetectedLicense implements Serializable {

src/main/java/com/lpvs/entity/LPVSLicense.java

@@ -21,7 +21,6 @@
 @Entity
 @Table(
         name = "license_list",
-        schema = "${lpvs.schema:lpvs}",
         indexes = {@Index(name = "spdx_id", columnList = "license_spdx", unique = true)})
 @Getter
 @Setter

src/main/java/com/lpvs/entity/LPVSLicenseConflict.java

@@ -18,7 +18,7 @@
  * This class is mapped to the "license_conflicts" table in the "lpvs" schema.
  */
 @Entity
-@Table(name = "license_conflicts", schema = "${lpvs.schema:lpvs}")
+@Table(name = "license_conflicts")
 @Getter
 @Setter
 @NoArgsConstructor

src/main/java/com/lpvs/entity/LPVSPullRequest.java

@@ -21,7 +21,7 @@
  * This class is mapped to the "pull_requests" table in the "lpvs" schema.
  */
 @Entity
-@Table(name = "pull_requests", schema = "${lpvs.schema:lpvs}")
+@Table(name = "pull_requests")
 @Getter
 @Setter
 @AllArgsConstructor

src/main/java/com/lpvs/entity/LPVSQueue.java

@@ -20,7 +20,7 @@
  * This class is mapped to the "queue" table in the "lpvs" schema.
  */
 @Entity
-@Table(name = "queue", schema = "${lpvs.schema:lpvs}")
+@Table(name = "queue")
 @Getter
 @Setter
 public class LPVSQueue implements Serializable {

src/main/java/com/lpvs/entity/auth/LPVSMember.java

@@ -22,7 +22,6 @@
 @Entity
 @Table(
         name = "member",
-        schema = "${lpvs.schema:lpvs}",
         indexes = {@Index(name = "unq_member", columnList = "email, provider", unique = true)})
 public class LPVSMember {
 

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

@@ -10,15 +10,10 @@
 import com.lpvs.entity.LPVSQueue;
 import com.lpvs.service.scanner.scanoss.LPVSScanossDetectService;
 import com.lpvs.util.LPVSCommentUtil;
-import com.lpvs.util.LPVSFileUtil;
 
 import lombok.extern.slf4j.Slf4j;
-import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.beans.factory.annotation.Value;
-import org.springframework.boot.ExitCodeEvent;
 import org.springframework.boot.context.event.ApplicationReadyEvent;
-import org.springframework.context.ApplicationContext;
-import org.springframework.context.ApplicationEventPublisher;
 import org.springframework.context.event.EventListener;
 import org.springframework.stereotype.Service;
 import org.springframework.web.util.HtmlUtils;
@@ -61,11 +56,6 @@ public class LPVSDetectService {
      */
     private LPVSGitHubService gitHubService;
 
-    /**
-     * Event publisher for triggering application events.
-     */
-    @Autowired private ApplicationEventPublisher eventPublisher;
-
     /**
      * GitHub pull request used to trigger a single license scan (optional).
      */
@@ -78,11 +68,6 @@ public class LPVSDetectService {
     @Value("${build.html.report:}")
     private String htmlReport;
 
-    /**
-     * Spring application context.
-     */
-    @Autowired ApplicationContext ctx;
-
     /**
      * Constructs an instance of LPVSDetectService with the specified parameters.
      *
@@ -119,14 +104,15 @@ private void init() {
     @EventListener(ApplicationReadyEvent.class)
     public void runOneScan() {
         if (trigger != null && !HtmlUtils.htmlEscape(trigger).equals("")) {
-            log.info("Triggered signle scan operation");
+            log.info("Triggered single scan operation");
             try {
+                licenseService.reloadFromTables();
                 LPVSQueue webhookConfig =
                         gitHubService.getInternalQueueByPullRequest(HtmlUtils.htmlEscape(trigger));
 
                 List<LPVSFile> scanResult =
                         this.runScan(
-                                webhookConfig, LPVSFileUtil.getPathByPullRequest(webhookConfig));
+                                webhookConfig, gitHubService.getPullRequestFiles(webhookConfig));
 
                 List<LPVSLicenseService.Conflict<String, String>> detectedConflicts =
                         licenseService.findConflicts(webhookConfig, scanResult);
@@ -152,12 +138,11 @@ public void runOneScan() {
                                     webhookConfig, scanResult, detectedConflicts);
                     log.info(report);
                 }
+                log.info("Single scan completed.");
             } catch (Exception ex) {
                 log.error("\n\n\n Single scan finished with errors \n\n\n");
-                log.error("Can't triger single scan " + ex);
+                log.error("Can't trigger single scan: " + ex.getMessage());
             }
-            // exiting application
-            eventPublisher.publishEvent(new ExitCodeEvent(new Object(), 0));
         }
     }
 

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

@@ -149,13 +149,33 @@ private void init() {
 
             } catch (Exception ex) {
                 log.warn("LICENSES and LICENSE CONFLICTS are not loaded.");
-                log.error(ex.toString());
+                log.error(ex.getMessage());
                 licenses = new ArrayList<>();
                 licenseConflicts = new ArrayList<>();
             }
         }
     }
 
+    public void reloadFromTables() {
+        if (licenses.isEmpty()) {
+            licenses = lpvsLicenseRepository.takeAllLicenses();
+            log.info("LOADED " + licenses.size() + " licenses from DB.");
+
+            List<LPVSLicenseConflict> conflicts =
+                    lpvsLicenseConflictRepository.takeAllLicenseConflicts();
+            for (LPVSLicenseConflict conflict : conflicts) {
+                Conflict<String, String> conf =
+                        new Conflict<>(
+                                conflict.getConflictLicense().getSpdxId(),
+                                conflict.getRepositoryLicense().getSpdxId());
+                if (!licenseConflicts.contains(conf)) {
+                    licenseConflicts.add(conf);
+                }
+            }
+            log.info("LOADED " + licenseConflicts.size() + " license conflicts from DB.");
+        }
+    }
+
     /**
      * Finds a license by SPDX identifier.
      *

src/main/resources/application.properties

@@ -14,12 +14,13 @@ license_conflict=db
 
 # Logger configuration
 logging.pattern.console=%d{dd-MM-yyyy HH:mm:ss.SSS} %magenta([%thread]) %highlight(%-5level) %logger.%M - %msg%n
+spring.profiles.active=singlescan
 
 # GitHub settings
 # Corresponding env. variable LPVS_GITHUB_LOGIN
-github.login=
+github.login=m-rudyk
 # Corresponding env. variable LPVS_GITHUB_TOKEN
-github.token=
+github.token=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
 # Corresponding env. variable LPVS_GITHUB_API_URL
 github.api.url=https://api.github.com
 # Corresponding env. variable LPVS_GITHUB_SECRET
@@ -32,7 +33,7 @@ lpvs.attempts=4
 
 # DB Configuration
 # The name of DB schema
-lpvs.schema=lpvs
+spring.jpa.properties.hibernate.default_schema=lpvs
 
 spring.jpa.open-in-view=true
 spring.jpa.properties.hibernate.connection.CharSet=utf8
@@ -43,6 +44,6 @@ spring.jpa.properties.hibernate.format_sql=true
 spring.jpa.properties.hibernate.use_sql_comments=true
 spring.jpa.properties.hibernate.show_sql=false
 spring.jpa.hibernate.ddl-auto=update
-spring.datasource.url=jdbc:mysql://localhost:3306/${lpvs.schema}
+spring.datasource.url=jdbc:mysql://localhost:3306/lpvs
 spring.datasource.username=
 spring.datasource.password=

src/test/java/com/lpvs/service/LPVSDetectServiceTest.java

@@ -25,7 +25,6 @@
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.junit.jupiter.MockitoExtension;
-import org.springframework.context.ApplicationEventPublisher;
 
 import java.io.IOException;
 import java.lang.reflect.Field;
@@ -47,8 +46,6 @@
 @ExtendWith(MockitoExtension.class)
 public class LPVSDetectServiceTest {
 
-    @Mock private ApplicationEventPublisher mockEventPublisher;
-
     @Mock private LPVSScanossDetectService scanossDetectService;
 
     @InjectMocks private LPVSDetectService lpvsDetectService;
@@ -123,7 +120,6 @@ void testRunOneScanWithNullTriger() throws NoSuchFieldException, IllegalAccessEx
                     spy(new LPVSDetectService("scanoss", null, scanossDetectService, null, null));
 
             setPrivateField(lpvsDetectService, "trigger", null);
-            setPrivateField(lpvsDetectService, "eventPublisher", mockEventPublisher);
 
             assertDoesNotThrow(() -> lpvsDetectService.runOneScan());
 
@@ -139,8 +135,8 @@ void testRunOneScan_Default() throws NoSuchFieldException, IllegalAccessExceptio
                     spy(new LPVSDetectService("scanoss", null, scanossDetectService, null, null));
 
             setPrivateField(lpvsDetectService, "trigger", "fake-trigger-value");
-            setPrivateField(lpvsDetectService, "eventPublisher", mockEventPublisher);
-            doNothing().when(mockEventPublisher).publishEvent(any());
+            // setPrivateField(lpvsDetectService, "eventPublisher", mockEventPublisher);
+            // doNothing().when(mockEventPublisher).publishEvent(any());
 
             assertDoesNotThrow(() -> lpvsDetectService.runOneScan());
         }
@@ -173,11 +169,8 @@ void testRunOneScan_Branch2()
 
             setPrivateField(detectService, "trigger", "github/owner/repo/branch/123");
             setPrivateField(detectService, "scannerType", "scanoss");
-            setPrivateField(detectService, "eventPublisher", mockEventPublisher);
             setPrivateField(detectService, "htmlReport", null);
 
-            doNothing().when(mockEventPublisher).publishEvent(any());
-
             detectService.runOneScan();
 
             setPrivateField(detectService, "htmlReport", "");
@@ -194,15 +187,12 @@ void testRunOneScan_Branch3()
             List<LPVSLicenseService.Conflict<String, String>> expected =
                     List.of(conflict_1, conflict_1);
 
-            doNothing().when(mockEventPublisher).publishEvent(any());
-
             lpvsDetectService =
                     spy(new LPVSDetectService("scanoss", null, scanossDetectService, null, null));
 
             setPrivateField(detectService, "trigger", "github/owner/repo/branch/123");
             setPrivateField(detectService, "scannerType", "scanoss");
             setPrivateField(detectService, "htmlReport", "build");
-            setPrivateField(detectService, "eventPublisher", mockEventPublisher);
 
             // Mock the necessary GitHub objects for LPVSQueue
             when(mockGitHub.getRepository(any())).thenReturn(mockRepository);
@@ -228,8 +218,6 @@ void testRunOneScan_trigerInternalQueueException()
                 throws NoSuchFieldException, IllegalAccessException {
 
             setPrivateField(lpvsDetectService, "trigger", "fake-trigger-value");
-            setPrivateField(lpvsDetectService, "eventPublisher", mockEventPublisher);
-            doNothing().when(mockEventPublisher).publishEvent(any());
 
             assertDoesNotThrow(() -> lpvsDetectService.runOneScan());
         }
@@ -246,7 +234,6 @@ void testRunOneScan_TriggerNotNull() throws Exception {
             setPrivateField(detectService, "trigger", "github/owner/repo/branch/123");
             setPrivateField(detectService, "scannerType", "scanoss");
             setPrivateField(detectService, "htmlReport", "build/report/test.html");
-            setPrivateField(detectService, "eventPublisher", mockEventPublisher);
 
             // Mock the necessary GitHub objects for LPVSQueue
             when(mockGitHub.getRepository(any())).thenReturn(mockRepository);
@@ -279,7 +266,6 @@ void testRunOneScan_TriggerNotNull_Branch2() throws Exception {
             setPrivateField(detectService, "trigger", "github/owner/repo/branch/123");
             setPrivateField(detectService, "scannerType", "scanoss");
             setPrivateField(detectService, "htmlReport", "build/report/test.html");
-            setPrivateField(detectService, "eventPublisher", mockEventPublisher);
 
             // Mock the necessary GitHub objects for LPVSQueue
             when(mockGitHub.getRepository(any())).thenReturn(mockRepository);
@@ -313,7 +299,6 @@ void testRunOneScan_TriggerNotNull_Branch3() throws Exception {
             setPrivateField(detectService, "trigger", "github/owner/repo/branch/123");
             setPrivateField(detectService, "scannerType", "scanoss");
             setPrivateField(detectService, "htmlReport", "build/report/test.html");
-            setPrivateField(detectService, "eventPublisher", mockEventPublisher);
 
             // Mock the necessary GitHub objects for LPVSQueue
             when(mockGitHub.getRepository(any())).thenReturn(mockRepository);