OSDN Git Service

supplicant(vts): Remove start/stop framework
authorRoshan Pius <rpius@google.com>
Mon, 24 Jul 2017 21:13:48 +0000 (14:13 -0700)
committerIliyan Malchev <malchev@google.com>
Tue, 25 Jul 2017 04:36:00 +0000 (04:36 +0000)
This is already done by the test harness on the host side. So, no need
to invoke this from within the gtest.

Bug: 63914399
Test: `make vts -j30 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 &&
vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module
VtsHalWifiSupplicantV1_0Target -l INFO`

Change-Id: I43edc03492bf5e03f9a4236265c43baa5efcd61d
(cherry picked from commit 145cce98b56674404a6148835f008df44086db2e)
Merged-Id: I43edc03492bf5e03f9a4236265c43baa5efcd61d

wifi/supplicant/1.0/vts/functional/VtsHalWifiSupplicantV1_0TargetTest.cpp
wifi/supplicant/1.0/vts/functional/supplicant_hidl_test_utils.cpp

index a69d14d..33f3049 100644 (file)
 class SupplicantHidlEnvironment : public ::testing::Environment {
    public:
     virtual void SetUp() override {
-        stopWifiFramework();
         stopSupplicant();
     }
     virtual void TearDown() override {
-        startWifiFramework();
-        // Framework will start wpa_supplicant.
     }
 };
 
index df4bfa9..79be2b0 100644 (file)
@@ -135,19 +135,6 @@ class ServiceNotificationListener : public IServiceNotification {
     std::condition_variable condition_;
 };
 
-void stopWifiFramework() {
-    ASSERT_EQ(std::system("stop"), 0);
-    // TODO: Use some other mechanism to wait for the framework to
-    // finish disabling.
-    sleep(5);
-}
-
-void startWifiFramework() {
-    ASSERT_EQ(std::system("start"), 0);
-    // These tests don't care whether the framework
-    // finished enabling or not.
-}
-
 void stopSupplicant() {
     DriverTool driver_tool;
     SupplicantManager supplicant_manager;