diff options
author | Maciej Żenczykowski <maze@google.com> | 2021-07-01 02:22:30 +0000 |
---|---|---|
committer | Maciej Zenczykowski <maze@google.com> | 2021-07-01 03:10:30 +0000 |
commit | aee2cbe30f5097ec85daa2cfabfe7377dfd5e542 (patch) | |
tree | ec36741078d8f126a549803ffa7a4a85e17f30e5 | |
parent | 2d4975f0c055ed9ee6ef9050c03fb19214d648b5 (diff) | |
download | netd-aee2cbe30f5097ec85daa2cfabfe7377dfd5e542.tar.gz |
Revert "Disable IdletimerAddRemoveInterface test on >=5.10"
This reverts commit ac67f9348b78ad6a9876e72cf57c1818177a9bfd.
The Android Common Kernels [5.10+] now supports this.
Test: atest NetdBinderTest IdletimerControllerTest
Bug: 175745224
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Original-Change: https://android-review.googlesource.com/1673846
Merged-In: I4ff791cd455dc14106e329e7c29811495ce7bae7
Change-Id: I4ff791cd455dc14106e329e7c29811495ce7bae7
-rw-r--r-- | tests/binder_test.cpp | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/tests/binder_test.cpp b/tests/binder_test.cpp index 69d1f9b5..1658abcb 100644 --- a/tests/binder_test.cpp +++ b/tests/binder_test.cpp @@ -43,7 +43,6 @@ #include <openssl/base64.h> #include <sys/socket.h> #include <sys/types.h> -#include <sys/utsname.h> #include <android-base/file.h> #include <android-base/format.h> @@ -1229,16 +1228,6 @@ void expectIdletimerInterfaceRuleNotExists(const std::string& ifname, int timeou } // namespace TEST_F(NetdBinderTest, IdletimerAddRemoveInterface) { - // TODO(b/175745224): Temporarily disable idletimer test on >5.10 kernels - utsname u; - if (!uname(&u)) { - unsigned long major, minor; - char *p; - major = strtoul(u.release, &p, 10); - minor = strtoul(++p, NULL, 10); - if (major > 5 || (major == 5 && minor >= 10)) return; - } - // TODO: We will get error in if expectIdletimerInterfaceRuleNotExists if there are the same // rule in the table. Because we only check the result after calling remove function. We might // check the actual rule which is removed by our function (maybe compare the results between |