fix 1cedcli behaviour, fix testing in PRs, qa, refactoring

Author: ovcharenko-di

.github/workflows/qa.yml

@@ -3,7 +3,7 @@
 # All rights reserved.
 
 name: Контроль качества
-# Любой пуш и pr в проекте но с фильтром по основному проекту
+
 on: [push, pull_request]
 jobs:
   build:
@@ -16,35 +16,29 @@ jobs:
         oscript_version: ['1.9.2']
 
     steps:
-      # Загрузка проекта
       - name: Актуализация
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4.2.2
 
       # https://stackoverflow.com/questions/58033366/how-to-get-current-branch-within-github-actions
       - name: Извлечение имени текущей ветки
         shell: bash
         run: echo "##[set-output name=branch;]$(echo ${GITHUB_REF#refs/heads/})"
         id: extract_branch
 
-      # Установка OneScript конкретной версии
       - name: Установка OneScript
-        uses: otymko/[email protected]
-        env:
-          ACTIONS_ALLOW_UNSECURE_COMMANDS: true
+        uses: otymko/[email protected]
         with:
           version: ${{ matrix.oscript_version }}
 
-      # Установка зависимостей пакета
       - name: Установка зависимостей
         run: |
           opm install opm
           opm install --dev
-          opm install 1testrunner
-          opm install 1bdd 
-          opm install notify
-          opm install coverage
+          opm install gitsync # TODO: должно быть необязательно
+
+      - name: Установка gitsync локально
+        run: opm run install-gitsync
 
-      # Запуск тестов и сбор покрытия кода
       - name: Покрытие кода
         run: oscript ./tasks/coverage.os true
 
@@ -54,9 +48,8 @@ jobs:
         id: extract_version
 
       - name: Установка Sonar-scanner
-        uses: warchant/setup-sonar-scanner@v1
+        uses: warchant/setup-sonar-scanner@v7
 
-      # Анализ проекта в SonarQube (ветка)
       - name: Анализ в SonarQube (branch)
         if: github.event_name == 'push'
         run: sonar-scanner
@@ -65,7 +58,6 @@ jobs:
             -Dsonar.branch.name=${{ steps.extract_branch.outputs.branch }}
             -Dsonar.projectVersion=${{ steps.extract_version.outputs.version }}
 
-      # Анализ проекта в SonarQube (PR)
       # https://docs.sonarqube.org/latest/analysis/pull-request/
       - name: Анализ в SonarQube (pull-request)
         if: github.event_name == 'pull_request'

.github/workflows/testing.yml

@@ -3,8 +3,15 @@
 # All rights reserved.
 
 name: Тестирование
-# Любой пуш и pr в проекте
-on: [push, pull_request]
+
+on:
+  push:
+  pull_request_target:
+  workflow_dispatch:
+
+permissions:
+  checks: write
+  pull-requests: write
 
 jobs:
   test:
@@ -28,8 +35,6 @@ jobs:
 
       - name: Актуализация
         uses: actions/[email protected]
-        with:
-          fetch-depth: 0
 
       # Установка OneScript конкретной версии
       - name: Установка OneScript
@@ -51,14 +56,14 @@ jobs:
             sudo DEBIAN_FRONTEND=noninteractive apt-get install -y locales libwebkit2gtk-4.0-37
             sudo localedef -i ${{ matrix.locale }} -c -f UTF-8 -A /usr/share/locale/locale.alias ${{ matrix.locale }}.UTF-8
 
-      - name: Установка libenchant1c2a (Linux)
+      - name: Установка libenchant1c2a для 8.3.21 (Linux)
         if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.v8_version, '8.3.21')
         run: |
             sudo echo "deb http://cz.archive.ubuntu.com/ubuntu focal main universe" | sudo tee -a /etc/apt/sources.list
             sudo apt-get update
             sudo DEBIAN_FRONTEND=noninteractive apt-get install -y libenchant1c2a
 
-      - name: Установка wine (требуется для Tool1CD)
+      - name: Установка wine для Tool1CD (Linux)
         if: startsWith(matrix.os, 'ubuntu')
         run: |
           sudo dpkg --add-architecture i386
@@ -76,16 +81,24 @@ jobs:
           ONEC_USERNAME: ${{ secrets.ONEC_USERNAME }}
           ONEC_PASSWORD: ${{ secrets.ONEC_PASSWORD }}
 
-      - name: Установка Java
+      - name: Установка Java 11
+        if: startsWith(matrix.edt_version, '2023')
         uses: actions/setup-java@v4
         with:
           distribution: 'temurin'
           java-version: '11'
 
+      - name: Установка Java 17
+        if: startsWith(matrix.edt_version, '2024')
+        uses: actions/setup-java@v4
+        with:
+          distribution: 'temurin'
+          java-version: '17'
+
       - name: Установка EDT
         uses: ovcharenko-di/onec-setup-action@fix/cache-ring-commands-cfg
         with:
-          type: edt # Тип устанавливаемого приложения
+          type: edt
           edt_version: ${{ matrix.edt_version }}
           cache: true
           cache_distr: true
@@ -117,10 +130,7 @@ jobs:
         env:
             ONEC_LICENCE: ${{ secrets.ONEC_LICENCE }}
 
-      - name: Сборка пакета
-        run: opm build
-
-      - name: Установка gitsync (локально)
+      - name: Установка gitsync локально
         run: opm run install-gitsync
 
       - name: Тестирование
@@ -132,10 +142,10 @@ jobs:
           run: oscript ./tasks/test.os
 
       - name: Публикация отчета
-        if: always()
+        if: success() || failure()
         uses: mikepenz/action-junit-report@v5
         with:
           report_paths: '**/build/reports/*.xml'
           fail_on_failure: true
           comment: true
-          check_name: 'Результаты тестов. ОС: ${{ matrix.os }}. Версия 1С: ${{ matrix.v8_version }}. Версия OneScript: ${{ matrix.oscript_version }}'
+          check_name: 'Результаты тестов. ОС: ${{ matrix.os }}. Версия 1С: ${{ matrix.v8_version }}. Версия OneScript: ${{ matrix.oscript_version }}. Версия EDT: ${{ matrix.edt_version }}'

CONTRIBUTING.md

@@ -21,8 +21,6 @@
 
   По умолчанию используется версия 2022.2.5
 
-- Cобрать пакет
-  - `opm build`
 - Выполнить команду, которая устанавливает gitsync в каталог bin_gitsync
   - `opm run install-gitsync`
 - Запустить тесты

features/edtExport.feature

@@ -23,17 +23,27 @@
     # И Я добавляю параметр "-v" для команды "gitsync"
     И Я добавляю параметр "sync" для команды "gitsync"
     И Я добавляю параметр "--PN test" для команды "gitsync"
-    И Я вычисляю имя модуля edt по версии из переменной окружения "EDT_VERSION" в переменную "ИмяМодуляEDT"
-    И Я добавляю параметр "--module" для команды "gitsync" из переменной "ИмяМодуляEDT"
     И Я выключаю все плагины
     И Я включаю плагин "edtExport"
 
-Сценарий: Cинхронизация с использованием edtExport
+Сценарий: Cинхронизация с использованием edtExport без явного указания версии EDT
     Допустим Я добавляю позиционный параметр для команды "gitsync" из переменной "КаталогХранилища1С"
     И Я добавляю позиционный параметр для команды "gitsync" из переменной "ПутьКаталогаИсходников"
     Когда Я выполняю команду "gitsync"
     Тогда Вывод команды "gitsync" содержит "Завершена синхронизации с git"
     И Вывод команды "gitsync" не содержит "Внешнее исключение"
+    И Вывод команды "gitsync" содержит "Версия EDT не передана через параметры, выполняется поиск максимальной установленной версии."
+    И Код возврата команды "gitsync" равен 0
+
+Сценарий: Cинхронизация с использованием edtExport с явным указанием версии EDT
+    Дано Я получаю переменную окружения "EDT_VERSION" в переменную "ВерсияEDT"
+    Дано Я добавляю параметр "--edt-version" для команды "gitsync" из переменной "ВерсияEDT"
+    Допустим Я добавляю позиционный параметр для команды "gitsync" из переменной "КаталогХранилища1С"
+    И Я добавляю позиционный параметр для команды "gitsync" из переменной "ПутьКаталогаИсходников"
+    Когда Я выполняю команду "gitsync"
+    Тогда Вывод команды "gitsync" содержит "Завершена синхронизации с git"
+    И Вывод команды "gitsync" не  содержит "Внешнее исключение"
+    И Вывод команды "gitsync" не содержит "Версия EDT не передана через параметры, выполняется поиск максимальной установленной версии."
     И Код возврата команды "gitsync" равен 0
 
 Сценарий: Cинхронизация хранилища расширения с использованием edtExport без указания базового проекта

features/step_definitions/edtExport.os

@@ -15,8 +15,6 @@
 	ВсеШаги = Новый Массив;
 
 	ВсеШаги.Добавить("ЯСкопировалКаталогРабочегоОкруженияEDTВКаталогИзПеременной");
-	ВсеШаги.Добавить("ЯВычисляюИмяМодуляEdtПоВерсииИзПеременнойОкруженияВПеременную");
-
 	Возврат ВсеШаги;
 КонецФункции
 
@@ -40,20 +38,6 @@
 	ФС.КопироватьСодержимоеКаталога(ПутьКРабочемуОкружениюEDT(), РабочееОкружениеEDT);
 КонецПроцедуры
 
-//я вычисляю имя модуля edt по версии из переменной окружения "EDT_VERSION" в переменную "ИмяМодуляEDT"
-Процедура ЯВычисляюИмяМодуляEdtПоВерсииИзПеременнойОкруженияВПеременную(Знач ИмяПеременнойОкружения, Знач ИмяПеременной) Экспорт
-
-	ВерсияEDT = ПолучитьПеременнуюСреды(ИмяПеременнойОкружения);
-
-	Если Не ЗначениеЗаполнено(ВерсияEDT) Тогда
-		ВерсияEDT = "2022.2.5"; // значение по умолчанию
-	КонецЕсли;
-	
-	ИмяМодуля = СтрШаблон("edt@%1:x86_64", ВерсияEDT);
-	БДД.СохранитьВКонтекст(ИмяПеременной, ИмяМодуля);
-
-КонецПроцедуры
-
 Функция ПутьКРабочемуОкружениюEDT()
 
 	Возврат ОбъединитьПути(КаталогFixtures(), "edtWorkspace");