Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[FEAT] Добавлен параметр ibsrv для запуска команд с использование автономного сервера #645

Closed
wants to merge 34 commits into from
Closed

[FEAT] Добавлен параметр ibsrv для запуска команд с использование автономного сервера #645

wants to merge 34 commits into from

Conversation

yukon39
Copy link
Contributor

@yukon39 yukon39 commented Nov 13, 2024

Добавлен параметр ibsrv для запуска команд с использование автономного сервера для команд:

  • run
  • vanessa
  • xunit
  • designer

Closes #619

Требует изменений в библиотеке ibcmdrunner PR #3

@yukon39 yukon39 mentioned this pull request Nov 13, 2024
Closed
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 13, 2024 •
edited
Loading

TestsPassed ✅Skipped ⏭️Failed ❌
Результаты тестов. ОС: windows-latest. Версия 1С: 8.3.23.1997. Версия OneScript: stable1223 ran1223 passed0 skipped0 failed
No test annotations available

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 13, 2024 •
edited
Loading

TestsPassed ✅Skipped ⏭️Failed ❌
Результаты тестов. ОС: ubuntu-latest. Версия 1С: 8.3.23.1997. Версия OneScript: stable1057 ran1057 passed0 skipped0 failed
No test annotations available

@sonar-openbsl-ru-qa-bot sonar.openbsl.ru qa-bot
Copy link

Failed

  • C Reliability Rating on New Code (is worse than A)
  • 102 New Issues (is greater than 0)

Analysis Details

102 Issues

  • Bug 1 Bug
  • Vulnerability 0 Vulnerabilities
  • Code Smell 101 Code Smells

Coverage and Duplications

  • Coverage 67.70% Coverage (62.10% Estimated after merge)
  • Duplications 0.61% Duplicated Code (4.10% Estimated after merge)

Project ID: vanessa-runner

View in SonarQube

@sonar-openbsl-ru-qa-bot sonar.openbsl.ru qa-bot
Copy link

Failed

  • C Reliability Rating on New Code (is worse than A)
  • 102 New Issues (is greater than 0)

Analysis Details

102 Issues

  • Bug 1 Bug
  • Vulnerability 0 Vulnerabilities
  • Code Smell 101 Code Smells

Coverage and Duplications

  • Coverage 67.70% Coverage (62.10% Estimated after merge)
  • Duplications 0.59% Duplicated Code (4.10% Estimated after merge)

Project ID: vanessa-runner

View in SonarQube

@sonar-openbsl-ru-qa-bot sonar.openbsl.ru qa-bot
Copy link

Failed

  • 57 New Issues (is greater than 0)

Analysis Details

57 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 57 Code Smells

Coverage and Duplications

  • Coverage 68.00% Coverage (62.10% Estimated after merge)
  • Duplications 0.57% Duplicated Code (4.00% Estimated after merge)

Project ID: vanessa-runner

View in SonarQube

@artbear artbear mentioned this pull request Nov 14, 2024
Merged
@sonar-openbsl-ru-qa-bot sonar.openbsl.ru qa-bot
Copy link

Failed

  • 57 New Issues (is greater than 0)

Analysis Details

57 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 57 Code Smells

Coverage and Duplications

  • Coverage 67.90% Coverage (62.10% Estimated after merge)
  • Duplications 0.58% Duplicated Code (4.00% Estimated after merge)

Project ID: vanessa-runner

View in SonarQube

@artbear artbear added this to the 2.3 milestone Nov 15, 2024
artbear
artbear requested changes Nov 16, 2024
View reviewed changes
src/Классы/МенеджерIbsrv.os
@@ -0,0 +1,276 @@

#Использовать ibcmdrunner
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yukon39 пройдись, пожалуйста, по Сонару, поправь замечания

например, вот это важно, похоже, ошибка копипаста

src/Классы/МенеджерIbsrv.os Outdated Show resolved Hide resolved
src/Классы/МенеджерIbsrv.os Outdated
@@ -245,7 +245,7 @@
РазрешитьРегламентныеЗадания = Ложь;

КонфигурацияАС = Новый КонфигурационныйФайлАС;
КонфигурацияАС.ПараметрыСервера(ПортHttpДоступа, СетевойИнтерфейс);
КонфигурацияАС.ПараметрыСервера(СетевойИнтерфейс, ПортHttpДоступа);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

вместо КонфигурацияАС = Новый КонфигурационныйФайлАС;

стоит написать КонфигурационныйФайлАС = Новый КонфигурационныйФайлАС;

тогда плагин bsl ls для ВСКод сможет выдавать подсказки по методам и их параметрам для этой переменной

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

исправлено

src/Модули/ОбщиеМетоды.os Show resolved Hide resolved
src/Модули/ОбщиеМетоды.os Outdated Show resolved Hide resolved
@Segate-ekb
Copy link
Collaborator

Segate-ekb commented Nov 19, 2024 •
edited
Loading

У нас в конвейере начали падать тесты.
У меня локально вот так:
image
Подозреваю, что внутри примерно то же самое, потому все висит и не завершается...

Надо с этим что-то решать )

image
Вот с этого момента тесты начали падать, т.е. по факту, с тех пор, как пошли попытки запустить АС

@Segate-ekb
Copy link
Collaborator

@yukon39 Обновили ci, добавив другие платформы, ребейзни пожалуйста )

@yukon39
Copy link
Contributor Author

yukon39 commented Nov 23, 2024

@artbear Добавил шаг сборки проекта в ci - собственно ошибка у @Segate-ekb возникает, т.к. проект банально не собран еще, и указанных обработок действительно нет в указанном месте.

Реализовать отдельным PR?

@Segate-ekb
Copy link
Collaborator

@artbear Добавил шаг сборки проекта в ci - собственно ошибка у @Segate-ekb возникает, т.к. проект банально не собран еще, и указанных обработок действительно нет в указанном месте.

Реализовать отдельным PR?

Вот я не знаю... у меня паттерн разработки - далеко не всегда зависит от предварительной сборки проекта. Мне кажется, что когда у тебя есть кнопка "Запустить тест", она должна выполнять тест...
Тем более, что мы не тестируем обработку, которая у нас в требованиях, и значит, в теории, вообще не должны ее собирать.
Не совсем понимаю, если мы не тестируем в данном тесте обработку, то почему не засунуть ее в фикстуры? Ну или в конце концов не собирать отдельно, если не нашли по нужному пути?

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 3, 2024 •
edited
Loading

TestsPassed ✅Skipped ⏭️Failed ❌
Результаты тестов. ОС: windows-latest. Версия 1С: 8.3.24.1548. Версия OneScript: stable1223 ran1223 passed0 skipped0 failed
No test annotations available

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 3, 2024 •
edited
Loading

TestsPassed ✅Skipped ⏭️Failed ❌
Результаты тестов. ОС: windows-latest. Версия 1С: 8.3.25.1445. Версия OneScript: stable1223 ran1223 passed0 skipped0 failed
No test annotations available

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 3, 2024 •
edited
Loading

TestsPassed ✅Skipped ⏭️Failed ❌
Результаты тестов. ОС: ubuntu-latest. Версия 1С: 8.3.24.1548. Версия OneScript: stable1057 ran1057 passed0 skipped0 failed
No test annotations available

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 3, 2024 •
edited
Loading

TestsPassed ✅Skipped ⏭️Failed ❌
Результаты тестов. ОС: ubuntu-latest. Версия 1С: 8.3.25.1445. Версия OneScript: stable1057 ran1057 passed0 skipped0 failed
No test annotations available

@artbear
Copy link
Collaborator

artbear commented Dec 4, 2024

ПР фактически смержен в d4ec54b

  • закрываю его

@yukon39 большое спасибо тебе за доработку и за терпение!
в описании релиза отмечу тебя как автора!

@artbear artbear closed this Dec 4, 2024
@yukon39 yukon39 deleted the issue-619-1 branch December 4, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@artbear artbear artbear requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.3
Development

Successfully merging this pull request may close these issues.

[FEATURE] Добавить возможность запуска платформы через ibsrv
3 participants
@yukon39 @Segate-ekb @artbear