andreacavalli/rocksdb
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Change GTEST_SKIP to BYPASS for MemoryAllocatorTest (#9340)

Browse Source 
Summary: Pull Request resolved: https://github.com/facebook/rocksdb/pull/9340

Reviewed By: riversand963

Differential Revision: D33344152

Pulled By: mrambacher

fbshipit-source-id: 283637625b86c33497571c5f52cac3ddf910b6f3
This commit is contained in:
mrambacher 2021-12-29 03:40:45 -08:00 committed by Facebook GitHub Bot
parent 26a238f5b7
commit 0a563ae278
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
memory/memory_allocator_test.cc
View File

@ -141,7 +141,7 @@ TEST_F(CreateMemoryAllocatorTest, JemallocOptionsTest) {
Status s = MemoryAllocator::CreateFromString(config_options_, id, &allocator); Status s = MemoryAllocator::CreateFromString(config_options_, id, &allocator);
if (!JemallocNodumpAllocator::IsSupported()) { if (!JemallocNodumpAllocator::IsSupported()) {
ASSERT_TRUE(s.IsNotSupported()); ASSERT_TRUE(s.IsNotSupported());
ROCKSDB_GTEST_SKIP("JEMALLOC not supported"); ROCKSDB_GTEST_BYPASS("JEMALLOC not supported");
return; return;
} }
ASSERT_OK(s); ASSERT_OK(s);
@ -193,7 +193,7 @@ TEST_F(CreateMemoryAllocatorTest, NewJemallocNodumpAllocator) {
std::string msg; std::string msg;
if (!JemallocNodumpAllocator::IsSupported(&msg)) { if (!JemallocNodumpAllocator::IsSupported(&msg)) {
ASSERT_TRUE(s.IsNotSupported()); ASSERT_TRUE(s.IsNotSupported());
ROCKSDB_GTEST_SKIP("JEMALLOC not supported"); ROCKSDB_GTEST_BYPASS("JEMALLOC not supported");
return; return;
} }
ASSERT_NOK(s); // Invalid options ASSERT_NOK(s); // Invalid options