Skip to content

Commit f2d79b0

Browse files
committed
Some code re-use in tests.
1 parent d2cc1e0 commit f2d79b0

File tree

1 file changed

+17
-38
lines changed

1 file changed

+17
-38
lines changed

src/app/clusters/general-commissioning-server/tests/TestGeneralCommissioningCluster.cpp

Lines changed: 17 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -49,23 +49,25 @@ struct TestGeneralCommissioningCluster : public ::testing::Test
4949
static void TearDownTestSuite() { chip::Platform::MemoryShutdown(); }
5050
};
5151

52+
GeneralCommissioningCluster::Context CreateStandardContext() {
53+
return {
54+
.commissioningWindowManager = Server::GetInstance().GetCommissioningWindowManager(), //
55+
.configurationManager = DeviceLayer::ConfigurationMgr(), //
56+
.deviceControlServer = DeviceLayer::DeviceControlServer::DeviceControlSvr(), //
57+
.fabricTable = Server::GetInstance().GetFabricTable(), //
58+
.failsafeContext = Server::GetInstance().GetFailSafeContext(), //
59+
.platformManager = DeviceLayer::PlatformMgr(), //
60+
#if CHIP_CONFIG_TERMS_AND_CONDITIONS_REQUIRED
61+
.termsAndConditionsProvider = TermsAndConditionsManager::GetInstance(),
62+
#endif // CHIP_CONFIG_TERMS_AND_CONDITIONS_REQUIRED
63+
};
64+
}
65+
5266
TEST_F(TestGeneralCommissioningCluster, TestAttributes)
5367
{
5468
// test without optional attributes
5569
{
56-
GeneralCommissioningCluster cluster(
57-
GeneralCommissioningCluster::Context {
58-
.commissioningWindowManager = Server::GetInstance().GetCommissioningWindowManager(), //
59-
.configurationManager = DeviceLayer::ConfigurationMgr(), //
60-
.deviceControlServer = DeviceLayer::DeviceControlServer::DeviceControlSvr(), //
61-
.fabricTable = Server::GetInstance().GetFabricTable(), //
62-
.failsafeContext = Server::GetInstance().GetFailSafeContext(), //
63-
.platformManager = DeviceLayer::PlatformMgr(), //
64-
#if CHIP_CONFIG_TERMS_AND_CONDITIONS_REQUIRED
65-
.termsAndConditionsProvider = TermsAndConditionsManager::GetInstance(),
66-
#endif // CHIP_CONFIG_TERMS_AND_CONDITIONS_REQUIRED
67-
},
68-
GeneralCommissioningCluster::OptionalAttributes(0));
70+
GeneralCommissioningCluster cluster(CreateStandardContext(), GeneralCommissioningCluster::OptionalAttributes(0));
6971

7072
ReadOnlyBufferBuilder<AttributeEntry> builder;
7173
ASSERT_EQ(cluster.Attributes({ kRootEndpointId, GeneralCommissioning::Id }, builder), CHIP_NO_ERROR);
@@ -100,18 +102,7 @@ TEST_F(TestGeneralCommissioningCluster, TestAttributes)
100102
// test with optional attributes
101103
{
102104
GeneralCommissioningCluster cluster(
103-
GeneralCommissioningCluster::Context {
104-
.commissioningWindowManager = Server::GetInstance().GetCommissioningWindowManager(), //
105-
.configurationManager = DeviceLayer::ConfigurationMgr(), //
106-
.deviceControlServer = DeviceLayer::DeviceControlServer::DeviceControlSvr(), //
107-
.fabricTable = Server::GetInstance().GetFabricTable(), //
108-
.failsafeContext = Server::GetInstance().GetFailSafeContext(), //
109-
.platformManager = DeviceLayer::PlatformMgr(), //
110-
#if CHIP_CONFIG_TERMS_AND_CONDITIONS_REQUIRED
111-
.termsAndConditionsProvider = TermsAndConditionsManager::GetInstance(),
112-
#endif // CHIP_CONFIG_TERMS_AND_CONDITIONS_REQUIRED
113-
},
114-
GeneralCommissioningCluster::OptionalAttributes().Set<IsCommissioningWithoutPower::Id>());
105+
CreateStandardContext(), GeneralCommissioningCluster::OptionalAttributes().Set<IsCommissioningWithoutPower::Id>());
115106

116107
ReadOnlyBufferBuilder<AttributeEntry> builder;
117108
ASSERT_EQ(cluster.Attributes({ kRootEndpointId, GeneralCommissioning::Id }, builder), CHIP_NO_ERROR);
@@ -152,19 +143,7 @@ TEST_F(TestGeneralCommissioningCluster, TestAttributes)
152143

153144
TEST_F(TestGeneralCommissioningCluster, TestAcceptedCommands)
154145
{
155-
GeneralCommissioningCluster cluster(
156-
GeneralCommissioningCluster::Context {
157-
.commissioningWindowManager = Server::GetInstance().GetCommissioningWindowManager(), //
158-
.configurationManager = DeviceLayer::ConfigurationMgr(), //
159-
.deviceControlServer = DeviceLayer::DeviceControlServer::DeviceControlSvr(), //
160-
.fabricTable = Server::GetInstance().GetFabricTable(), //
161-
.failsafeContext = Server::GetInstance().GetFailSafeContext(), //
162-
.platformManager = DeviceLayer::PlatformMgr(), //
163-
#if CHIP_CONFIG_TERMS_AND_CONDITIONS_REQUIRED
164-
.termsAndConditionsProvider = TermsAndConditionsManager::GetInstance(),
165-
#endif // CHIP_CONFIG_TERMS_AND_CONDITIONS_REQUIRED
166-
},
167-
GeneralCommissioningCluster::OptionalAttributes());
146+
GeneralCommissioningCluster cluster(CreateStandardContext(), GeneralCommissioningCluster::OptionalAttributes());
168147

169148
{
170149
ReadOnlyBufferBuilder<AcceptedCommandEntry> builder;

0 commit comments

Comments
 (0)