apmanager: support for WiFi devices and interfaces
Added DeviceInfo for enumerating WiFi devices (PHYs) and detecting
WiFi interfaces. Also Created a Device class for abstracting WiFi PHYs,
which will maintain the device's capabilities and manage interfaces
created on the device.
Integrate Device with Manager and Config to allow user to start
an AP service without providing interface information.
Next up, will focus on parsing HT/VHT capabilities for each band and
integrate it into ap configuration. Also adding shill support for
claim/release interface.
BUG=chromium:431763
TEST=unittest
Change-Id: I62f4251064483b57e264d1278fde68022a737aea
Reviewed-on: https://chromium-review.googlesource.com/231257
Reviewed-by: Peter Qiu <zqiu@chromium.org>
Commit-Queue: Peter Qiu <zqiu@chromium.org>
Tested-by: Peter Qiu <zqiu@chromium.org>
diff --git a/service_unittest.cc b/service_unittest.cc
index 3788258..5f99f7f 100644
--- a/service_unittest.cc
+++ b/service_unittest.cc
@@ -14,6 +14,7 @@
#include <gtest/gtest.h>
#include "apmanager/mock_config.h"
+#include "apmanager/mock_manager.h"
using chromeos::ProcessMock;
using ::testing::_;
@@ -30,7 +31,7 @@
class ServiceTest : public testing::Test {
public:
- ServiceTest() : service_(kServiceIdentifier) {}
+ ServiceTest() : service_(&manager_, kServiceIdentifier) {}
void StartDummyProcess() {
service_.hostapd_process_.reset(new chromeos::ProcessImpl);
@@ -46,6 +47,7 @@
protected:
Service service_;
+ MockManager manager_;
};
MATCHER_P(IsServiceErrorStartingWith, message, "") {
@@ -82,6 +84,7 @@
chromeos::ErrorPtr error;
EXPECT_CALL(*config, GenerateConfigFile(_, _)).WillOnce(
DoAll(SetArgPointee<1>(config_str), Return(true)));
+ EXPECT_CALL(*config, ClaimDevice()).WillOnce(Return(true));
EXPECT_TRUE(service_.Start(&error));
EXPECT_EQ(nullptr, error);
}
@@ -96,7 +99,10 @@
TEST_F(ServiceTest, StopSuccess) {
StartDummyProcess();
+ MockConfig* config = new MockConfig();
+ SetConfig(config);
chromeos::ErrorPtr error;
+ EXPECT_CALL(*config, ReleaseDevice()).Times(1);
EXPECT_TRUE(service_.Stop(&error));
}