Skip to content

Develop1#1266

Open
Maria-Simonova wants to merge 4 commits intoyandex-praktikum:mainfrom
Maria-Simonova:develop1
Open

Develop1#1266
Maria-Simonova wants to merge 4 commits intoyandex-praktikum:mainfrom
Maria-Simonova:develop1

Conversation

@Maria-Simonova
Copy link

Дополнительная информация по отчётам:

  1. Изначально JaCoCo отчёт был успешно сформирован в проекте:
    Initial PR commit Maria-Simonova/Task_1Jacoco#1

  2. Позже я обратила внимание, что выполнила проект не полностью
    в соответствии с инструкцией, и решила переделать работу.

  3. При повторной попытке сгенерировать JaCoCo отчёт в новом проекте
    возникли проблемы, связанные с корпоративными настройками ноутбука
    (ограничения безопасности, невозможность настроить JAVA_HOME и т.д.).

  4. В связи с этим в основном решении (данный PR) был добавлен
    Allure-отчёт, который служит подтверждением того, что тесты
    выполняются успешно и покрытие соответствует требованиям.

Основная работа (код + Allure отчёт) находится в этом PR.
Ссылка на JaCoCo отчёт приведена для демонстрации того,
что изначально он был успешно сформирован.

@@ -0,0 +1,113 @@
package praktikum;

Choose a reason for hiding this comment

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

Весь пакет test должен лежать в пакете src, нужно перенести. После переноса удали лишние импорты.

@Maria-Simonova
Copy link
Author

@mkouznetsov-git Добрый день! Исправила замечания по Task1.

  1. Тесты перенесены в src/test
  2. Удалены лишние импорты в BurgerTest
  3. Добавлен Allure отчёт.

Готово к повторной проверке!

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.

2 participants