fixes

Author: theshadowco

build.gradle.kts

@@ -19,7 +19,6 @@ repositories {
     mavenLocal()
     mavenCentral()
     maven("https://central.sonatype.com/repository/maven-snapshots")
-    maven("https://repo.maven.apache.org/maven2")
 }
 
 val sonarQubeVersion = "25.4.0.105899"

src/main/java/com/github/_1c_syntax/bsl/sonar/ext_issues/RulesFileReader.java

@@ -53,7 +53,6 @@ public RulesFileReader(String[] filePaths) {
    *
    * @param resourceName Имя ресурса-файла
    * @param filePaths    Массив путей к загружаемым файлам
-   *
    * @return Список прочитанных файлов-описаний
    */
   public static List<RulesFile> getRulesFiles(String resourceName, String[] filePaths) {
@@ -88,21 +87,21 @@ private static Optional<RulesFile> getRulesFile(String json) {
       .configure(MapperFeature.ACCEPT_CASE_INSENSITIVE_PROPERTIES, true)
       .build();
     try {
-      return Optional.of(objectMapper.readValue(json, RulesFile.class));
+      var data = objectMapper.readValue(json, RulesFile.class);
+      if (data != null && data.rules() == null) { // пустой конфиг может прочитаться, но нам это содержимое не нужно
+        data = null;
+      }
+      return Optional.ofNullable(data);
     } catch (IOException e) {
       LOGGER.error("Can't serialize json rules to object", e);
       return Optional.empty();
     }
   }
 
   private Optional<RulesFile> getNext() {
-    if (hasMore()) {
-      Optional<RulesFile> rules = getRulesFromFile();
-      current++;
-      return rules;
-    }
-
-    return Optional.empty();
+    Optional<RulesFile> rules = getRulesFromFile();
+    current++;
+    return rules;
   }
 
   private boolean hasMore() {

src/test/java/com/github/_1c_syntax/bsl/sonar/ext_issues/RuleDefinitionTest.java

@@ -89,4 +89,27 @@ void testExternalFile() {
     assertThat(repository.rules()).hasSize(285);
     assertThat(repository.rules()).allMatch(rule -> rule.name().length() < 200);
   }
+
+  @Test
+  void testExternalFileWithErrors() {
+    // все должно отработать, несмотря на отсутствующие и битые файлы
+    var fakeRulePath = new File("fake.json");
+    var noRulePath = new File("src/test/resources/examples/.bsl-language-server.json");
+    var noConfigPath = new File("src/test/resources/examples/highlight.bsl");
+    var config = new MapSettings()
+      .setProperty(reporter.getEnabledKey(), true)
+      .setProperty(reporter.getRulesPathsKey(), fakeRulePath.getAbsolutePath()
+        + "," + noRulePath.getAbsolutePath()
+        + "," + noConfigPath.getAbsolutePath())
+      .asConfig();
+    var ruleDefinition = new RuleDefinitionsContainer(config);
+    var context = new RulesDefinition.Context();
+    ruleDefinition.define(context);
+
+    assertThat(context.repositories()).hasSize(1);
+    var repository = context.repository(reporter.getRepositoryKey());
+    assertThat(repository).isNotNull();
+    assertThat(repository.rules()).hasSize(281);
+    assertThat(repository.rules()).allMatch(rule -> rule.name().length() < 200);
+  }
 }