-
Notifications
You must be signed in to change notification settings - Fork 5
Настройка линз VSCode для запуска тестов через OneUnit #12
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.
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
base: master
Are you sure you want to change the base?
Conversation
WalkthroughДобавлена новая секция 2.3 в README.md с инструкциями по запуску тестов в VSCode через BSL Language Server CodeLens. Включен пример .bsl-language-server.json с полями для executable, аргументов тестов, опций coverage/open test report и getTestsResultPattern. Изменения носят сугубо документальный характер. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (3)
README.md (3)
690-690
: Добавьте язык к ограждённому блоку кода.Сейчас блок начинается с ``` без указания языка, из‑за чего срабатывает markdownlint MD040. Для корректной подсветки синтаксиса и прохождения линтеров укажите json.
-``` +```json
686-689
: Уточните расположение файла конфигурации и необходимость перезагрузки LSP.Стоит явно указать, что файл
.bsl-language-server.json
следует размещать в корне рабочей области VSCode (workspace root). Также полезно добавить, что после внесения изменений BSL Language Server нужно перезапустить/перегрузить (или перезагрузить окно VSCode), чтобы линзы подхватили новую конфигурацию.
697-699
: Уточните про каталог вывода отчётов.Пути
out/genericCoverage.xml
,out/otr.xml
,out/cobertura.xml
подразумевают существование каталогаout
. Еслиoneunit
не создаёт директории автоматически, выполнение из линз может завершиться ошибкой записи. Стоит:
- либо добавить примечание «Убедитесь, что каталог out существует»;
- либо заменить пути на те, что гарантированно существуют в проекте.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md
(1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
README.md
690-690: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🔇 Additional comments (1)
README.md (1)
694-701
: Проверьте корректность аргументов и регулярного выражения для извлечения имени теста.
- Плейсхолдеры
%s
для--file
и-m
должны соответствовать тому, как BSL Language Server подставляет путь к файлу и имя теста.- Текущее
getTestsResultPattern
:^[^.]+\.([^\s]+).*$
предполагает формат выводаdiscover
видаНабор.Метод ...
и выдёргивает метод как первую группу. Это может быть хрупко, если в выводе появятся дополнительные точки/префиксы, или формат локализован/изменён.Рекомендации к проверке:
- Запустите
oneunit discover --file <путь.к.файлу> --mode flat
и убедитесь, что первая группа в регулярке действительно даёт нужное имя теста.- Если потребуется чуть более устойчивый вариант, можно, например, ограничить захват до конца строки или до первого пробела после метода, в зависимости от фактического вывода. Например:
- Начало строки, до первой точки — имя набора, затем захват имени метода до первого пробела:
^[^.]+\.(\S+)
- Или, если вывод строго
Набор.Метод
без хвоста:^[^.]+\.(\S+)$
Summary by CodeRabbit