Skip to content

Conversation

@yukon39
Copy link
Contributor

@yukon39 yukon39 commented Nov 13, 2024

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

  • run
  • vanessa
  • xunit
  • designer

Closes #619

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

@github-actions
Copy link

github-actions bot commented Nov 13, 2024

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

@github-actions
Copy link

github-actions bot commented Nov 13, 2024

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
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
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
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

@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
Copy link
Collaborator

Choose a reason for hiding this comment

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

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

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

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.

исправлено

@Segate-ekb
Copy link
Collaborator

Segate-ekb commented Nov 19, 2024

У нас в конвейере начали падать тесты.
У меня локально вот так:
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
Copy link

github-actions bot commented Dec 3, 2024

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

@github-actions
Copy link

github-actions bot commented Dec 3, 2024

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

@github-actions
Copy link

github-actions bot commented Dec 3, 2024

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

@github-actions
Copy link

github-actions bot commented Dec 3, 2024

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Добавить возможность запуска платформы через ibsrv

3 participants