CPP-KT / bimap-task

0 stars 0 forks source link

test diff_lower_and_upper_bound #9

Closed Nomad192 closed 1 year ago

Nomad192 commented 1 year ago

Добавил к исходным тестам проверку различия upper_bound и lower_bound.

upper_bound -> строго больше (Источник: https://en.cppreference.com/w/cpp/algorithm/upper_bound) lower_bound -> больше или равно (Источник: https://en.cppreference.com/w/cpp/algorithm/lower_bound)

По хорошему, можно было бы просто добавить ешё несколько проверок в тест upper_bound, тк в нынешнем состоянии он не проверяет ситуацию на ключ который уже есть в bimap

Dogzik commented 1 year ago

Пожалуйста следуй стилю, определённому конфигом форматтера в проекте

lejabque commented 1 year ago

а зачем это выносить в отдельный тест? выше есть два теста с такими же наборами значений, добавь просто необходимые проверки туда. и лучше сравнивать результат upper/lower_bound с конкретным ожидаемым значением, а не между собой