# Ревью

1. Дублирование кейсов в двух папках, или так и должно быть?:

* Дополнительные параметры (малая карточка сервера).
* Дополнительные параметры (малая карточка сервера) - соответствие прототипу.
* Дополнительные параметры - общая проверка формы редактирования.
* Настройка отображения таблицы серверов.

2. Заголовки не очень понятные у многих кейсов, может, лучше их изменить на более информативные?

#### **[Дополнительные параметры - общая проверка формы редактирования](http://testit.ispsystem.net/projects/277/tests/24743?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05)**

1. Что за форма? Где эта форма? Есть прототип? При первом открытии нет понимания, что имеется ввиду, где и куда смотреть.
2. Не совсем понимаю логику, в общих шагах описана проверка смены типа поля, а далее проверяется кнопка сохранения с введенной информацией, что подразумевает проверку ввода разных значений, сохранение обязательного/необязательного поля и тд.
4. Может, стоит вынести проверку баннера в отдельный кейс? Баннер всегда есть или в каких-то случаях пропадает?

#### [Дополнительные параметры - в форме добавления сущности](http://testit.ispsystem.net/projects/277/tests/25014?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b)

1. Шаг 1 - что в ожидаемом поведении?

#### [Дополнительное поле - числовое](http://testit.ispsystem.net/projects/277/tests/24747?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b)

1. Нет проверки ввода строки, спецсимволов, даты,
2. Сколько чисел можно ввести после запятой? Отрицательные числа можно указать?

#### [Дополнительное поле - ссылка](http://testit.ispsystem.net/projects/277/tests/24751?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b)

1. Шаг 3 - Что значит "Ввод обычной строки" и ожидаемый результат "Успешно"?
2. Если точку в конце ссылки укажу?

#### Проверка обязательности
В обоих кейсах шаг 1 - мне нужно выполнить проверку для 7 полей в одном шаге? Выглядит так, что мне нужно это проверить для каждого типа поля, значит должно быть 7 проверок :point_down:
* [Необязательное поле](http://testit.ispsystem.net/projects/277/tests/24973?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b)
* [Обязательное поле](http://testit.ispsystem.net/projects/277/tests/24974?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b)

#### [Таблица серверов/оборудования](http://testit.ispsystem.net/projects/277/tests/25015?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b)

1. Не увидела проверки сворачивания длинного заголовка кастомного поля в три точки.
2. Не увидела проверки появления хинта с полным названием кастомного поля по наведению на свернутый заголовок, и по наведению на значения в строке таблицы.
3. Дата в каком формате отображается в таблице? дд.мм.гггг или дд мес гггг?
4. Проверка сортировки и фильтрации?
5. Проверка отображения длинных значений, сворачивание в три точки.

#### Стойка

[Стойка - соответствие прототипу](http://testit.ispsystem.net/projects/277/tests/25093?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b) - -тут написано, что по умолчанию вкладка доп параметров в свернутом состоянии, а в этом кейсе [Модуль "Визуализация стоек" - дополнительные поля](http://testit.ispsystem.net/projects/277/tests/25167?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b) - написано, что по умолчанию вкладка доп параметров развернута. Как в итоге должно быть?

#### [Проверка смены типов дополнительных полей](http://testit.ispsystem.net/projects/277/tests/25263?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b)

Стоит более подробно описать, нет явного описания проверки, что тип действительно изменился.

#### [Настройка отображения таблицы серверов](http://testit.ispsystem.net/projects/277/tests/25044?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b)

1. Шаг 11 - описаны две проверки в одном шаге, в ожидаемом поведении.
2. Шаг 12 - может, лучше написать "кнопка задизейблена"? Я сначала подумала, что "Отключена" - значит её выключили :thinking_face: и пошла смотреть, как она отключена в dci.

#### [Дополнительные параметры (малая карточка сервера)](http://testit.ispsystem.net/projects/277/tests/25088?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b)

В шаге 4 в ожидаемом поведении написано "Кликабельно", потом в следующем шаге 5 - написано, что происходит по клику. То есть в двух шагах проверяется клик на ссылку. Мне кажется, можно объединить эти два шага, тк тут фактически выполняется одно действие - переход по ссылке. Визуал можно попутно проверить, не выделяя в отдельный шаг. Тк больше времени тратится просто на прокликивание этих шагов при тестировании.

#### Формы добавления сущностей
Мне кажется, будет удобнее, если написать в формах добавления, в предусловия - какие кастомные поля нужно добавить для проверки добавления этой сущности.

#### [Форма добавления распределителя питания](http://testit.ispsystem.net/projects/277/tests/25229?isolatedSection=90359e12-b8dd-44b1-9ee3-016d379ae05b)
Шаг 154 - дубль шага 152.

#### Дополнительно

Проверка удаления кастомных полей?