Skip to content

Commit 33e1634

Browse files
jwhuijwhui
authored
[nexus] migrate srp_client_save_server_info to nexus (openthread#12947)
This commit migrates the srp_client_save_server_info test from the thread-cert Python-based test framework to the Nexus C++ simulation framework. The new Nexus test (test_srp_client_save_server_info.cpp) verifies: - SRP client selects an SRP server when auto-start is enabled. - SRP client sticks to the current server even if other SRP servers become available. - SRP client saves and reuses the selected server info across SRP client stops and restarts. - SRP client selects a new server if the current one becomes unavailable. - SRP client sticks to the new server even if the old one returns. The original Python test script is removed as its functionality is now fully covered by the new Nexus test.
1 parent a51677d commit 33e1634

3 files changed

Lines changed: 188 additions & 162 deletions

File tree

‎tests/nexus/CMakeLists.txt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -408,6 +408,7 @@ ot_nexus_test(router_downgrade_on_sec_policy_change "core;nexus")
408408
ot_nexus_test(srp_auto_start "core;nexus")
409409
ot_nexus_test(srp_client_change_lease "core;nexus")
410410
ot_nexus_test(srp_client_remove_host "core;nexus")
411+
ot_nexus_test(srp_client_save_server_info "core;nexus")
411412
ot_nexus_test(srp_lease "core;nexus")
412413
ot_nexus_test(srp_many_services_mtu_check "core;nexus")
413414
ot_nexus_test(srp_scale "core;nexus")

‎tests/nexus/test_srp_client_save_server_info.cpp‎

Lines changed: 187 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,187 @@
1+
/*
2+
* Copyright (c) 2026, The OpenThread Authors.
3+
* All rights reserved.
4+
*
5+
* Redistribution and use in source and binary forms, with or without
6+
* modification, are permitted provided that the following conditions are met:
7+
* 1. Redistributions of source code must retain the above copyright
8+
* notice, this list of conditions and the following disclaimer.
9+
* 2. Redistributions in binary form must reproduce the above copyright
10+
* notice, this list of conditions and the following disclaimer in the
11+
* documentation and/or other materials provided with the distribution.
12+
* 3. Neither the name of the copyright holder nor the
13+
* names of its contributors may be used to endorse or promote products
14+
* derived from this software without specific prior written permission.
15+
*
16+
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
17+
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
18+
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
19+
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
20+
* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
21+
* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
22+
* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
23+
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
24+
* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
25+
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
26+
* POSSIBILITY OF SUCH DAMAGE.
27+
*/
28+
29+
#include <stdio.h>
30+
31+
#include "platform/nexus_core.hpp"
32+
#include "platform/nexus_node.hpp"
33+
34+
namespace ot {
35+
namespace Nexus {
36+
37+
/**
38+
* Time to advance for a node to form a network and become leader, in milliseconds.
39+
*/
40+
static constexpr uint32_t kFormNetworkTime = 13 * 1000;
41+
42+
/**
43+
* Time to advance for the network to stabilize.
44+
*/
45+
static constexpr uint32_t kWaitTime = 16 * 1000;
46+
47+
/**
48+
* SRP service port.
49+
*/
50+
static constexpr uint16_t kSrpServicePort = 12345;
51+
52+
void TestSrpClientSaveServerInfo(const char *aJsonFileName)
53+
{
54+
Core nexus;
55+
56+
Node &client = nexus.CreateNode();
57+
Node &server1 = nexus.CreateNode();
58+
Node &server2 = nexus.CreateNode();
59+
Node &server3 = nexus.CreateNode();
60+
61+
client.SetName("CLIENT");
62+
server1.SetName("SERVER1");
63+
server2.SetName("SERVER2");
64+
server3.SetName("SERVER3");
65+
66+
SuccessOrQuit(Instance::SetGlobalLogLevel(kLogLevelNote));
67+
68+
Log("---------------------------------------------------------------------------------------");
69+
Log("Step 1: Start the server & client devices.");
70+
71+
client.Form();
72+
nexus.AdvanceTime(kFormNetworkTime);
73+
VerifyOrQuit(client.Get<Mle::Mle>().IsLeader());
74+
75+
AllowLinkBetween(client, server1);
76+
AllowLinkBetween(client, server2);
77+
AllowLinkBetween(client, server3);
78+
79+
server1.Join(client);
80+
server2.Join(client);
81+
server3.Join(client);
82+
nexus.AdvanceTime(Time::SecToMsec(200));
83+
84+
VerifyOrQuit(server1.Get<Mle::Mle>().IsRouterOrLeader());
85+
VerifyOrQuit(server2.Get<Mle::Mle>().IsRouterOrLeader());
86+
VerifyOrQuit(server3.Get<Mle::Mle>().IsRouterOrLeader());
87+
88+
Log("---------------------------------------------------------------------------------------");
89+
Log("Step 2: Enable auto start mode on client and check that server1 is used.");
90+
91+
SuccessOrQuit(server1.Get<Srp::Server>().SetAddressMode(Srp::Server::kAddressModeUnicast));
92+
server1.Get<Srp::Server>().SetEnabled(true);
93+
nexus.AdvanceTime(kWaitTime);
94+
95+
SuccessOrQuit(client.Get<Srp::Client>().SetHostName("host"));
96+
SuccessOrQuit(client.Get<Srp::Client>().EnableAutoHostAddress());
97+
{
98+
Srp::Client::Service service;
99+
ClearAllBytes(service);
100+
service.mName = "_ipps._tcp";
101+
service.mInstanceName = "my-service";
102+
service.mPort = kSrpServicePort;
103+
SuccessOrQuit(client.Get<Srp::Client>().AddService(service));
104+
}
105+
106+
client.Get<Srp::Client>().EnableAutoStartMode(nullptr, nullptr);
107+
VerifyOrQuit(client.Get<Srp::Client>().IsAutoStartModeEnabled());
108+
109+
nexus.AdvanceTime(kWaitTime);
110+
111+
VerifyOrQuit(client.Get<Srp::Client>().IsRunning());
112+
VerifyOrQuit(
113+
server1.Get<ThreadNetif>().HasUnicastAddress(client.Get<Srp::Client>().GetServerAddress().GetAddress()));
114+
VerifyOrQuit(client.Get<Srp::Client>().GetHostInfo().GetState() == Srp::Client::kRegistered);
115+
116+
Log("---------------------------------------------------------------------------------------");
117+
Log("Step 3: Enable server2 and server3 and check that server1 is still used.");
118+
119+
SuccessOrQuit(server2.Get<Srp::Server>().SetAddressMode(Srp::Server::kAddressModeUnicast));
120+
server2.Get<Srp::Server>().SetEnabled(true);
121+
SuccessOrQuit(server3.Get<Srp::Server>().SetAddressMode(Srp::Server::kAddressModeUnicast));
122+
server3.Get<Srp::Server>().SetEnabled(true);
123+
124+
nexus.AdvanceTime(kWaitTime);
125+
VerifyOrQuit(
126+
server1.Get<ThreadNetif>().HasUnicastAddress(client.Get<Srp::Client>().GetServerAddress().GetAddress()));
127+
128+
Log("---------------------------------------------------------------------------------------");
129+
Log("Step 4: Stop and restart the client (multiple times) and verify that server1 is always picked.");
130+
131+
for (int iter = 0; iter < 5; iter++)
132+
{
133+
client.Get<Srp::Client>().Stop();
134+
client.Get<Srp::Client>().EnableAutoStartMode(nullptr, nullptr);
135+
nexus.AdvanceTime(kWaitTime);
136+
VerifyOrQuit(
137+
server1.Get<ThreadNetif>().HasUnicastAddress(client.Get<Srp::Client>().GetServerAddress().GetAddress()));
138+
VerifyOrQuit(client.Get<Srp::Client>().GetHostInfo().GetState() == Srp::Client::kRegistered);
139+
}
140+
141+
Log("---------------------------------------------------------------------------------------");
142+
Log("Step 5: Stop the client, then stop server1 and restart client and verify that server1 is no longer used.");
143+
144+
client.Get<Srp::Client>().Stop();
145+
server1.Get<Srp::Server>().SetEnabled(false);
146+
nexus.AdvanceTime(kWaitTime);
147+
148+
client.Get<Srp::Client>().EnableAutoStartMode(nullptr, nullptr);
149+
nexus.AdvanceTime(kWaitTime);
150+
151+
Ip6::Address serverAddress = client.Get<Srp::Client>().GetServerAddress().GetAddress();
152+
VerifyOrQuit(!server1.Get<ThreadNetif>().HasUnicastAddress(serverAddress));
153+
VerifyOrQuit(server2.Get<ThreadNetif>().HasUnicastAddress(serverAddress) ||
154+
server3.Get<ThreadNetif>().HasUnicastAddress(serverAddress));
155+
VerifyOrQuit(client.Get<Srp::Client>().GetHostInfo().GetState() == Srp::Client::kRegistered);
156+
157+
Log("---------------------------------------------------------------------------------------");
158+
Log("Step 6: Start back server1, then restart client and verify that now we remain with the new saved server "
159+
"info.");
160+
161+
server1.Get<Srp::Server>().SetEnabled(true);
162+
nexus.AdvanceTime(kWaitTime);
163+
164+
for (int iter = 0; iter < 5; iter++)
165+
{
166+
client.Get<Srp::Client>().Stop();
167+
nexus.AdvanceTime(kWaitTime);
168+
client.Get<Srp::Client>().EnableAutoStartMode(nullptr, nullptr);
169+
nexus.AdvanceTime(kWaitTime);
170+
VerifyOrQuit(client.Get<Srp::Client>().GetServerAddress().GetAddress() == serverAddress);
171+
VerifyOrQuit(client.Get<Srp::Client>().GetHostInfo().GetState() == Srp::Client::kRegistered);
172+
}
173+
174+
Log("---------------------------------------------------------------------------------------");
175+
176+
nexus.SaveTestInfo(aJsonFileName);
177+
}
178+
179+
} // namespace Nexus
180+
} // namespace ot
181+
182+
int main(int argc, char *argv[])
183+
{
184+
ot::Nexus::TestSrpClientSaveServerInfo((argc > 2) ? argv[2] : "test_srp_client_save_server_info.json");
185+
printf("All tests passed\n");
186+
return 0;
187+
}

‎tests/scripts/thread-cert/test_srp_client_save_server_info.py‎

Lines changed: 0 additions & 162 deletions
This file was deleted.

0 commit comments

Comments
 (0)