Skip to content

Commit ace3bd4

Browse files
committed
test: improve test method naming in McpApplicationTest
1 parent 2c6a431 commit ace3bd4

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

src/test/java/com/github/thought2code/mcp/annotated/McpApplicationTest.java

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ private void startServer(String configFileName) {
7070
}
7171

7272
@Test
73-
void testStartStdioServer_shouldSucceed() {
73+
void startStdioServer_succeeds() {
7474
TestMcpStdioServer.main(new String[] {}); // just for jacoco coverage report
7575

7676
String classpath = System.getProperty("java.class.path");
@@ -89,7 +89,7 @@ void testStartStdioServer_shouldSucceed() {
8989
}
9090

9191
@Test
92-
void testStartSseServer_shouldSucceed() {
92+
void startSseServer_succeeds() {
9393
final int port = new Random().nextInt(8000, 9000);
9494

9595
McpServerConfiguration.Builder configuration =
@@ -113,7 +113,7 @@ void testStartSseServer_shouldSucceed() {
113113
}
114114

115115
@Test
116-
void testStartStreamableServer_shouldSucceed() {
116+
void startStreamableServer_succeeds() {
117117
final int port = new Random().nextInt(8000, 9000);
118118

119119
McpServerConfiguration.Builder configuration =
@@ -137,15 +137,15 @@ void testStartStreamableServer_shouldSucceed() {
137137
}
138138

139139
@Test
140-
void testStartServer_useDefaultConfigFileName_shouldSucceed() {
140+
void startServer_withDefaultConfig_succeeds() {
141141
String configFileName = "mcp-server.yml";
142142
McpConfigurationLoader configLoader = new McpConfigurationLoader(configFileName);
143143
McpServerConfiguration configuration = configLoader.loadConfig();
144144
assertEquals(ServerMode.STREAMABLE, configuration.mode());
145145
}
146146

147147
@Test
148-
void testStartServer_disabledMCP_shouldSucceed() {
148+
void startServer_whenDisabled_succeeds() {
149149
String configFileName = "test-mcp-server-disabled.yml";
150150
McpConfigurationLoader configLoader = new McpConfigurationLoader(configFileName);
151151
McpServerConfiguration configuration = configLoader.loadConfig();
@@ -154,7 +154,7 @@ void testStartServer_disabledMCP_shouldSucceed() {
154154
}
155155

156156
@Test
157-
void testStartServer_enableStdioMode_shouldSucceed() {
157+
void startServer_withStdioMode_succeeds() {
158158
String configFileName = "test-mcp-server-enable-stdio-mode.yml";
159159
McpConfigurationLoader configLoader = new McpConfigurationLoader(configFileName);
160160
McpServerConfiguration configuration = configLoader.loadConfig();
@@ -163,7 +163,7 @@ void testStartServer_enableStdioMode_shouldSucceed() {
163163
}
164164

165165
@Test
166-
void testStartServer_enableHttpSseMode_shouldSucceed() {
166+
void startServer_withSseMode_succeeds() {
167167
String configFileName = "test-mcp-server-enable-http-sse-mode.yml";
168168
McpConfigurationLoader configLoader = new McpConfigurationLoader(configFileName);
169169
McpServerConfiguration configuration = configLoader.loadConfig();
@@ -172,7 +172,7 @@ void testStartServer_enableHttpSseMode_shouldSucceed() {
172172
}
173173

174174
@Test
175-
void testStartServer_enableStreamableHttpMode_shouldSucceed() {
175+
void startServer_withStreamableMode_succeeds() {
176176
String configFileName = "test-mcp-server-enable-streamable-http-mode.yml";
177177
McpConfigurationLoader configLoader = new McpConfigurationLoader(configFileName);
178178
McpServerConfiguration configuration = configLoader.loadConfig();
@@ -181,7 +181,7 @@ void testStartServer_enableStreamableHttpMode_shouldSucceed() {
181181
}
182182

183183
@Test
184-
void testStartServer_enableUnknownMode_shouldThrowException() {
184+
void startServer_withUnknownMode_throwsException() {
185185
String configFileName = "test-mcp-server-enable-unknown-mode.yml";
186186
assertThrows(McpServerConfigurationException.class, () -> startServer(configFileName));
187187
}

0 commit comments

Comments
 (0)