diff options
| author | yakubovskiy03 <yakubovskiy03@yandex-team.com> | 2022-10-25 16:34:43 +0300 |
|---|---|---|
| committer | yakubovskiy03 <yakubovskiy03@yandex-team.com> | 2022-10-25 16:34:43 +0300 |
| commit | 70dc14cf15259cb9aec11c59e26cc56848640dcc (patch) | |
| tree | 360b7e642fce0b3574f24cdc7e49f14ca3d22910 /library/cpp/testing/README.md | |
| parent | d18928ecee746c0e009d8504d94c374aafe681af (diff) | |
| download | ydb-70dc14cf15259cb9aec11c59e26cc56848640dcc.tar.gz | |
add NOLINT for clang-tidy
Сейчас диагностика [readability-container-size-empty](https://releases.llvm.org/14.0.0/tools/clang/tools/extra/docs/clang-tidy/checks/readability-container-size-empty.html) ругается на %%UNIT_ASSERT_EQUAL(s, "")%%. Можно поменять на %%UNIT_ASSERT(s.empty())%%, но у этого изменения есть как минимум 2 недостатка (дальше со слов @sobols):
- Если тест сломается и строчка станет непустой, мы не увидим красивого сообщения об ошибке, содержащего новое значение, а будет безликое сообщение про assertion failed.
- Выглядит неоднородно с соседним кодом, который проверяет на непустые строчки.
Diffstat (limited to 'library/cpp/testing/README.md')
0 files changed, 0 insertions, 0 deletions