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.

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

[WIP] Покрытие через внешние тесты #1357

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

dmpas
Copy link
Collaborator

@dmpas dmpas commented Sep 29, 2023

@nixel2007 покрытие для сонара.

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

Passed

Analysis Details

0 Issues

  • Bug0 Bugs
  • Vulnerability0 Vulnerabilities
  • Code Smell0 Code Smells

Coverage and Duplications

  • No coverage informationNo coverage information (34.30% Estimated after merge)
  • No duplication informationNo duplication information (3.40% Estimated after merge)

View in SonarQube

@nixel2007
Copy link
Collaborator

+4 процента код кавереджа. Это запуск всех тестов такое дал или только какой-то один?

@dmpas
Copy link
Collaborator Author

dmpas commented Sep 30, 2023

+4 процента код кавереджа. Это запуск всех тестов такое дал или только какой-то один?

@nixel2007 только typedescription, как проверка концепции
если @EvilBeaver одобрит сам формат, то накидаю остальные

@nixel2007
Copy link
Collaborator

nixel2007 commented Sep 30, 2023

@dmpas А можно каждый тест в отдельный fact засунуть? Ну, типа динамически список тестов формировать?

Плюс интересно, почему пришлось делать новый враппер. В старом что-то не так в работе под дотнет?

@dmpas
Copy link
Collaborator Author

dmpas commented Sep 30, 2023

@nixel2007 с ходу я не нагуглил, но не особо-то и гуглил. Я так пониммю, в 2.0 часть тестов пока заведомо красные, потому не стал делать автоматом все тесты.

В старом nUnit, в новом xUnit
Ну и сам принцип создания движка поменялся. Теперь билдер и вот это вот вс.ё.

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

Passed

Analysis Details

0 Issues

  • Bug0 Bugs
  • Vulnerability0 Vulnerabilities
  • Code Smell0 Code Smells

Coverage and Duplications

  • No coverage informationNo coverage information (38.90% Estimated after merge)
  • No duplication informationNo duplication information (3.40% Estimated after merge)

View in SonarQube

@nixel2007
Copy link
Collaborator

+8!

@nixel2007
Copy link
Collaborator

@EvilBeaver
Copy link
Owner

Не, чет выглядит как костыль. Тесты надо прописывать руками, дискавери тоже негибкое. Я начинал исследование кастомного тест-раннера для NUnit, даже ветку завел. Тогда скриптовые тесты были бы видны прямо в RIDER / VS и штатно прогонялись бы на CI

var engine = Instance;

var rootPath = Path.GetDirectoryName(Assembly.GetExecutingAssembly().Location)!;
var filepath = Path.Combine(rootPath, "..", "..", "..", "..", "..", "..", "tests", filename);
Copy link
Owner

Choose a reason for hiding this comment

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

бррр

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

я с ходу не нашёл, откуда вытащить корень исходника.

src/Tests/TestRunner/TestRunnerHelper.cs Outdated Show resolved Hide resolved
@nixel2007
Copy link
Collaborator

@EvilBeaver дискавери вроде как можно и настроить (см ссылку выше, при чем она десятилетней давности, может что-то лучше уже придумали).

@nixel2007
Copy link
Collaborator

А что за ветка?

@EvilBeaver
Copy link
Owner

А что за ветка?

Она не опубликована, потому что в ней нет ничего вменяемого. Поделиться нечем

@EvilBeaver
Copy link
Owner

@dmpas
Copy link
Collaborator Author

dmpas commented Oct 1, 2023

Не, чет выглядит как костыль. Тесты надо прописывать руками, дискавери тоже негибкое. Я начинал исследование кастомного тест-раннера для NUnit, даже ветку завел. Тогда скриптовые тесты были бы видны прямо в RIDER / VS и штатно прогонялись бы на CI

@EvilBeaver а думал, ты NUnit выкинул и перешёл на xUnit. Так-то давай NUnit вернём к жизни.

@dmpas
Copy link
Collaborator Author

dmpas commented Oct 1, 2023

@EvilBeaver дискавери вроде как можно и настроить (см ссылку выше, при чем она десятилетней давности, может что-то лучше уже придумали).

@nixel2007 там по ссылке грусть какая-то. С динамическим формированием классов на коленке.

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

Passed

Analysis Details

0 Issues

  • Bug0 Bugs
  • Vulnerability0 Vulnerabilities
  • Code Smell0 Code Smells

Coverage and Duplications

  • No coverage informationNo coverage information (38.90% Estimated after merge)
  • No duplication informationNo duplication information (3.40% Estimated after merge)

View in SonarQube

@nixel2007
Copy link
Collaborator

nixel2007 commented Oct 2, 2023

Я понимаю, нет ничего более постоянного и так далее, но раз пока нет красивой реализации, поживём на некрасивой?

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.

3 participants