diff options
author | Elliott Hughes <enh@google.com> | 2019-02-07 12:41:37 -0800 |
---|---|---|
committer | Elliott Hughes <enh@google.com> | 2019-02-11 13:15:56 -0800 |
commit | 01705e4f1d87a30734363d0b9dde10de0866baef (patch) | |
tree | 6cc739fbf1d1311c6392c7af79a5990ffacc9029 /libutils/StrongPointer_test.cpp | |
parent | ea57928f002306ad3994f5c209af9ccd185fb96b (diff) |
Clean up libcutils/libutils tests.
Move tests in the same directory as the corresponding code, so it's
easier to see what is/isn't tested.
Fix naming of libcutils_tests (plural) to match the singular that's more
common (even though the plural makes more sense to me).
Add these two to system/core/'s TEST_MAPPING.
Remove obsolete AndroidTest.xml.
Fix a flaky (timing-dependent) libcutils test.
Test: ran tests
Change-Id: I7e0a31ff45c8a152562bf66fc97161594249366e
Diffstat (limited to 'libutils/StrongPointer_test.cpp')
-rw-r--r-- | libutils/StrongPointer_test.cpp | 58 |
1 files changed, 58 insertions, 0 deletions
diff --git a/libutils/StrongPointer_test.cpp b/libutils/StrongPointer_test.cpp new file mode 100644 index 000000000..153cf9683 --- /dev/null +++ b/libutils/StrongPointer_test.cpp @@ -0,0 +1,58 @@ +/* + * Copyright (C) 2015 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#include <gtest/gtest.h> + +#include <utils/StrongPointer.h> +#include <utils/RefBase.h> + +using namespace android; + +class SPFoo : public LightRefBase<SPFoo> { +public: + explicit SPFoo(bool* deleted_check) : mDeleted(deleted_check) { + *mDeleted = false; + } + + ~SPFoo() { + *mDeleted = true; + } +private: + bool* mDeleted; +}; + +TEST(StrongPointer, move) { + bool isDeleted; + SPFoo* foo = new SPFoo(&isDeleted); + ASSERT_EQ(0, foo->getStrongCount()); + ASSERT_FALSE(isDeleted) << "Already deleted...?"; + sp<SPFoo> sp1(foo); + ASSERT_EQ(1, foo->getStrongCount()); + { + sp<SPFoo> sp2 = std::move(sp1); + ASSERT_EQ(1, foo->getStrongCount()) << "std::move failed, incremented refcnt"; + ASSERT_EQ(nullptr, sp1.get()) << "std::move failed, sp1 is still valid"; + // The strong count isn't increasing, let's double check the old object + // is properly reset and doesn't early delete + sp1 = std::move(sp2); + } + ASSERT_FALSE(isDeleted) << "deleted too early! still has a reference!"; + { + // Now let's double check it deletes on time + sp<SPFoo> sp2 = std::move(sp1); + } + ASSERT_TRUE(isDeleted) << "foo was leaked!"; +} |