Ошибки в тестах

[ni032mas]

Привет!

В тестах ниже кажется есть ошибки.

1. Тут мы создаем таски, закрываем их, но не удаляем, а в конце проверяем что у пользователя нет тасок.

mts-teta-entrance-exam-java-middle-2022/src/test/java/com/example/demo/ServerTest.java

Line 221 in 8c461d7

assertListTask(user, emptyList());

4. Тут ошибка аффектит два теста, в одном ожидается ERROR, в другом REOPENED. Но если мы следуем правилу, что у нас все регистрозависимое, то у нас должна быть проверка на ACCESS_DENIED, так как первый параметр %S превращает user в строку в верхнем регистре, а создавались и закрывались таски без преобразований user.

mts-teta-entrance-exam-java-middle-2022/src/test/java/com/example/demo/ServerTest.java

Line 233 in 8c461d7

return sendMessage(format("%S REOPEN_TASK %s", user, task));

[SimonHarmonicMinor]

Привет! Спасибо за замечания. Поправил ошибки в тестах. Можешь смержиться с master локально чт, 30 июн. 2022 г. в 22:42, Aleksandr Marmyshev ***@***.***>:

…

Привет! В тестах ниже кажется есть ошибки. 1. https://github.com/SimonHarmonicMinor/mts-teta-entrance-exam-java-middle-2022/blob/8c461d7dcc817ff66492f193bcd9c98934e5940f/src/test/java/com/example/demo/ServerTest.java#L213 Тут мы создаем таски, закрываем их, но не удаляем, а в конце проверяем что у пользователя нет тасок. ***@***.*** @CsvSource({ "queen,one_vision", "jackals,legacy" }) void shouldFailIfUserHasNoRightsToDeleteTask(String user, String task) { assertEquals("CREATED", createTask(user, task)); assertListTask(user, List.of(task)); assertEquals("CLOSED", closeTask(user, task)); assertListTask(user, List.of(task)); assertEquals("ACCESS_DENIED", deleteTask(user + "_IMPOSTER", task)); assertListTask(user, emptyList()); }` 1. https://github.com/SimonHarmonicMinor/mts-teta-entrance-exam-java-middle-2022/blob/8c461d7dcc817ff66492f193bcd9c98934e5940f/src/test/java/com/example/demo/ServerTest.java#L233 Тут ошибка аффектит два теста, в одном ожидается ERROR, в другом REOPENED. Но если мы следуем правилу, что у нас все регистрозависимое, то у нас должна быть проверка на ACCESS_DENIED, так как первый параметр %S превращает user в строку в верхнем регистре, а создавались и закрывались таски без преобразований user. private String reopenTask(String user, String task) { return sendMessage(format("%S REOPEN_TASK %s", user, task)); } — Reply to this email directly, view it on GitHub <#58>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AGOFLKIMU5RZLRQEPJQFQDDVRX2AXANCNFSM52KOKD7A> . You are receiving this because you are subscribed to this thread.Message ID: <SimonHarmonicMinor/mts-teta-entrance-exam-java-middle-2022/issues/58@ github.com>

-- С уважением, Киреков Семен Алексеевич.