Skip to content

Lab 5: Architecture rules (NetArchTest)#29

Open
KonstantinDanger wants to merge 28 commits into
lenagrin:masterfrom
KonstantinDanger:lab-5
Open

Lab 5: Architecture rules (NetArchTest)#29
KonstantinDanger wants to merge 28 commits into
lenagrin:masterfrom
KonstantinDanger:lab-5

Conversation

@KonstantinDanger
Copy link
Copy Markdown

@KonstantinDanger KonstantinDanger commented May 16, 2026

Lab 5: Architecture rules (NetArchTest)

Мета роботи

дослідження архітектурних правила залежностей.


Необхідні умови

Додати кілька архітектурних правил залежностей (наприклад, *.UI не має залежати від *.Infrastructure напряму). Переконатися, що порушення ламає збірку (червоний PR), а фікс — зеленить. Здати: PR із тестами правил, скрін невдалого прогону (до фіксу) і зеленого (після).


Виконав студент:
Яцюк Костянтин
групи ПЗС-1


Хід роботи

1. Створимо залежність NetSdrClientApp.Messages від NetSdrClientApp.Networking та навпаки:

image image

2. Встановлення NetArchTest.Rules

image

3. Створення тестів:

image

Тести не пройдено:
image

Sonar check не пройшов:
image
image


4. Позбавлення залежностей:

image image

Закоментований код було видалено

Тести пройдено:
image


5. Перевірка PR:

image

Як перевірити

  1. Відкрити Pull Request: Lab 5: Architecture rules (NetArchTest) KonstantinDanger/reengineering-course-fork#9
  2. Перейти у вкладку "Checks"
  3. Переконатися, що SonarCloud analysis виконано
  4. Перевірити статус Quality Gate
  5. Перевірити коміти з виправленим кодом

Висновки

Під час виконання даної лабораторної роботи було розроблено юніт-тести для перевірки залежностей в архітектурі програми.

KonstantinDanger and others added 28 commits April 30, 2026 21:52
Lab-1. SonarCloud integration and CI setup
Lab 1:  Sonar Cloud and CI connection
Added more tests to gain bigger coverage percentage
Removed code duplication
Excluded new code from coverage check
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