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 | |
| 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')
| -rw-r--r-- | library/cpp/testing/unittest/registar.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/library/cpp/testing/unittest/registar.h b/library/cpp/testing/unittest/registar.h index 2e0a9637b8..e3d2fec867 100644 --- a/library/cpp/testing/unittest/registar.h +++ b/library/cpp/testing/unittest/registar.h @@ -488,7 +488,7 @@ public: \ //general #define UNIT_ASSERT_EQUAL_C(A, B, C) \ do { \ - if (!((A) == (B))) { \ + if (!((A) == (B))) { /* NOLINT(readability-container-size-empty) */ \ UNIT_FAIL_IMPL("equal assertion failed", Sprintf("%s == %s %s", #A, #B, (::TStringBuilder() << C).data())); \ } \ } while (false) @@ -497,7 +497,7 @@ public: \ #define UNIT_ASSERT_UNEQUAL_C(A, B, C) \ do { \ - if ((A) == (B)) { \ + if ((A) == (B)) { /* NOLINT(readability-container-size-empty) */ \ UNIT_FAIL_IMPL("unequal assertion failed", Sprintf("%s != %s %s", #A, #B, (::TStringBuilder() << C).data()));\ } \ } while (false) |