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

Separated TextFormatter #17

Merged
merged 10 commits into from
Dec 15, 2023
Merged

Separated TextFormatter #17

merged 10 commits into from
Dec 15, 2023

Conversation

AllDmeat
Copy link
Contributor

@AllDmeat AllDmeat commented Dec 15, 2023

Готовлюсь к тому, чтобы добавить другие форматтеры:

  1. Отпилил текущий форматтер
  2. Скрыл его за протокол
  3. Вынес код который относится к доменной модели репорта прямо в неё, чтобы было доступно во всех форматтеррах.
  4. Убрал сущность фильтров. Вместо неё заюзал существующую сущность Статуса теста.

@AllDmeat AllDmeat requested a review from a team as a code owner December 15, 2023 14:16
@AllDmeat AllDmeat requested a review from Vespen December 15, 2023 14:16
@AllDmeat AllDmeat enabled auto-merge (squash) December 15, 2023 14:17
@AllDmeat AllDmeat disabled auto-merge December 15, 2023 14:24
@AllDmeat AllDmeat merged commit e76051b into main Dec 15, 2023
2 checks passed
@AllDmeat AllDmeat deleted the feature/separated-text-format branch December 15, 2023 14:24
AllDmeat added a commit that referenced this pull request Dec 19, 2023
* Separated text formatter

* Formatter → TextFormatter

* Removed duplicated tests

* Removed Parser

* Introduced FormatterProtocol

* Renamed Filters to TestResults

* Replaced TestResult with ReportModel.Module.File.RepeatableTest.Test.Status

* Moved common code out of formatter

* Removed unused code

* Minr change
AllDmeat added a commit that referenced this pull request Dec 19, 2023
* Separated text formatter

* Formatter → TextFormatter

* Removed duplicated tests

* Removed Parser

* Introduced FormatterProtocol

* Renamed Filters to TestResults

* Replaced TestResult with ReportModel.Module.File.RepeatableTest.Test.Status

* Moved common code out of formatter

* Removed unused code

* Minr change
AllDmeat added a commit that referenced this pull request Dec 19, 2023
* Separated text formatter

* Formatter → TextFormatter

* Removed duplicated tests

* Removed Parser

* Introduced FormatterProtocol

* Renamed Filters to TestResults

* Replaced TestResult with ReportModel.Module.File.RepeatableTest.Test.Status

* Moved common code out of formatter

* Removed unused code

* Minr change
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.

1 participant