From c567cca7ab09ecc47fd8d616ee7219a6326548d4 Mon Sep 17 00:00:00 2001 From: Nikita Fedkin Date: Wed, 8 Dec 2021 17:11:07 +0300 Subject: [PATCH] =?UTF-8?q?=D0=98=D1=81=D0=BF=D1=80=D0=B0=D0=B2=D0=BB?= =?UTF-8?q?=D0=B5=D0=BD=20=D0=BC=D0=B5=D1=80=D0=B6=20=D0=BF=D1=80=D0=B8?= =?UTF-8?q?=D0=BC=D0=B8=D1=82=D0=B8=D0=B2=D0=BE=D0=B2=20=D0=B2=20=D0=BA?= =?UTF-8?q?=D0=BE=D0=BD=D1=84=D0=B8=D0=B3=D1=83=D1=80=D0=B0=D1=86=D0=B8?= =?UTF-8?q?=D1=8F=D1=85?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../library/configuration/InitInfoBaseOptions.groovy | 2 +- .../configuration/ResultsTransformOptions.groovy | 4 ++-- .../library/configuration/SonarQubeOptions.groovy | 2 +- .../jenkins/library/configuration/StageFlags.groovy | 12 ++++++------ .../configuration/ConfigurationReaderTest.java | 4 +++- test/unit/resources/jobConfiguration.json | 3 +++ 6 files changed, 16 insertions(+), 11 deletions(-) diff --git a/src/ru/pulsar/jenkins/library/configuration/InitInfoBaseOptions.groovy b/src/ru/pulsar/jenkins/library/configuration/InitInfoBaseOptions.groovy index d01d3b9..f0874a6 100644 --- a/src/ru/pulsar/jenkins/library/configuration/InitInfoBaseOptions.groovy +++ b/src/ru/pulsar/jenkins/library/configuration/InitInfoBaseOptions.groovy @@ -17,7 +17,7 @@ class InitInfoBaseOptions implements Serializable { InitInfoBaseMethod initMethod = InitInfoBaseMethod.FROM_STORAGE; @JsonPropertyDescription("Запустить миграцию ИБ") - boolean runMigration = true + Boolean runMigration = true @JsonPropertyDescription("""Дополнительные шаги, запускаемые через vrunner. В каждой строке передается отдельная команда diff --git a/src/ru/pulsar/jenkins/library/configuration/ResultsTransformOptions.groovy b/src/ru/pulsar/jenkins/library/configuration/ResultsTransformOptions.groovy index 08168fc..f450d84 100644 --- a/src/ru/pulsar/jenkins/library/configuration/ResultsTransformOptions.groovy +++ b/src/ru/pulsar/jenkins/library/configuration/ResultsTransformOptions.groovy @@ -8,14 +8,14 @@ import com.fasterxml.jackson.annotation.JsonPropertyDescription class ResultsTransformOptions implements Serializable { @JsonPropertyDescription("Фильтровать замечания по уровню поддержки модуля. По умолчанию включено.") - boolean removeSupport = true + Boolean removeSupport = true @JsonPropertyDescription("""Настройка фильтрации замечаний по уровню поддержки. 0 - удалить файлы на замке; 1 - удалить файлы на замке и на поддержке; 2 - удалить файлы на замке, на поддержке и снятые с поддержки. """) - int supportLevel + Integer supportLevel @Override @NonCPS diff --git a/src/ru/pulsar/jenkins/library/configuration/SonarQubeOptions.groovy b/src/ru/pulsar/jenkins/library/configuration/SonarQubeOptions.groovy index 7dcbb37..cb25672 100644 --- a/src/ru/pulsar/jenkins/library/configuration/SonarQubeOptions.groovy +++ b/src/ru/pulsar/jenkins/library/configuration/SonarQubeOptions.groovy @@ -13,7 +13,7 @@ class SonarQubeOptions implements Serializable { String sonarQubeInstallation; @JsonPropertyDescription("Использовать sonar-scanner, доступный в PATH") - boolean useSonarScannerFromPath + Boolean useSonarScannerFromPath @JsonPropertyDescription( "Имя настроенной утилиты sonar-scanner.\nПрименяется, если useSonarScannerFromPath установлено в false." diff --git a/src/ru/pulsar/jenkins/library/configuration/StageFlags.groovy b/src/ru/pulsar/jenkins/library/configuration/StageFlags.groovy index 8dd2949..f52cb76 100644 --- a/src/ru/pulsar/jenkins/library/configuration/StageFlags.groovy +++ b/src/ru/pulsar/jenkins/library/configuration/StageFlags.groovy @@ -7,22 +7,22 @@ import com.fasterxml.jackson.annotation.JsonPropertyDescription @JsonIgnoreProperties(ignoreUnknown = true) class StageFlags implements Serializable { @JsonPropertyDescription("Анализ SonarQube включен") - boolean sonarqube + Boolean sonarqube @JsonPropertyDescription("Синтаксический контроль включен") - boolean syntaxCheck + Boolean syntaxCheck @JsonPropertyDescription("Валидация EDT включена") - boolean edtValidate + Boolean edtValidate @JsonPropertyDescription("Дымовые тесты включены") - boolean smoke + Boolean smoke @JsonPropertyDescription("Предварительные шаги инициализации включены") - boolean initSteps + Boolean initSteps @JsonPropertyDescription("Запуск BDD сценариев включен") - boolean bdd + Boolean bdd @Override @NonCPS diff --git a/test/unit/groovy/ru/pulsar/jenkins/library/configuration/ConfigurationReaderTest.java b/test/unit/groovy/ru/pulsar/jenkins/library/configuration/ConfigurationReaderTest.java index 43c34aa..64d2d3b 100644 --- a/test/unit/groovy/ru/pulsar/jenkins/library/configuration/ConfigurationReaderTest.java +++ b/test/unit/groovy/ru/pulsar/jenkins/library/configuration/ConfigurationReaderTest.java @@ -33,6 +33,8 @@ class ConfigurationReaderTest { assertThat(jobConfiguration.getV8version()).isEqualTo("8.3.14.1944"); assertThat(jobConfiguration.getSonarQubeOptions().getSonarScannerToolName()).isEqualTo("sonar-scanner"); + assertThat(jobConfiguration.getSonarQubeOptions().getSonarQubeInstallation()).isEqualTo("qa"); + assertThat(jobConfiguration.getSonarQubeOptions().getUseSonarScannerFromPath()).isTrue(); assertThat(jobConfiguration.getSecrets()) .hasFieldOrPropertyWithValue("storage", "1234") @@ -41,7 +43,7 @@ class ConfigurationReaderTest { assertThat(jobConfiguration.getSyntaxCheckOptions().getCheckModes()).hasSize(1); - assertThat(jobConfiguration.getResultsTransformOptions().isRemoveSupport()).isFalse(); + assertThat(jobConfiguration.getResultsTransformOptions().getRemoveSupport()).isFalse(); assertThat(jobConfiguration.getResultsTransformOptions().getSupportLevel()).isZero(); assertThat(jobConfiguration.getSmokeTestOptions().getVrunnerSettings()).contains("./tools/vrunner-smoke.json"); diff --git a/test/unit/resources/jobConfiguration.json b/test/unit/resources/jobConfiguration.json index e49e884..a4205c1 100644 --- a/test/unit/resources/jobConfiguration.json +++ b/test/unit/resources/jobConfiguration.json @@ -15,6 +15,9 @@ "vanessa --settings ./tools/vrunner.first.json" ] }, + "sonarqube": { + "sonarQubeInstallation": "qa" + }, "syntaxCheck": { "checkModes": ["-ThinClient"] },