Skip to content

Commit 4222d4a

Browse files
committed
[MINOR] Remove unnecesary string splits in test code
### What changes were proposed in this pull request? This PR aims to remove unnenesary string splits in the test code. ### Why are the changes needed? There are three instances which looks like mistakes. ```java - "ConfigMap name" + " should include secondary resource prefix"); + "ConfigMap name should include secondary resource prefix"); ``` ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#338 from dongjoon-hyun/minor_string. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent b4ad042 commit 4222d4a

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

spark-operator-api/src/test/java/org/apache/spark/k8s/operator/spec/ApplicationSpecTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ void testSpecWithDriverServiceIngressList() throws IOException {
5858
InputStream inputStream =
5959
getClass()
6060
.getClassLoader()
61-
.getResourceAsStream("spark-job-with-driver-service" + "-ingress.json");
61+
.getResourceAsStream("spark-job-with-driver-service-ingress.json");
6262
ApplicationSpec spec = objectMapper.readValue(inputStream, ApplicationSpec.class);
6363
assertNotNull(spec.getDriverServiceIngressList());
6464
assertEquals(1, spec.getDriverServiceIngressList().size());

spark-submission-worker/src/test/java/org/apache/spark/k8s/operator/SparkAppDriverConfTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,10 @@ void testResourceNamePrefix() {
5252
+ "but different values are detected");
5353
assertTrue(
5454
sparkAppDriverConf.configMapNameDriver().contains(resourcePrefix),
55-
"ConfigMap name" + " should include secondary resource prefix");
55+
"ConfigMap name should include secondary resource prefix");
5656
assertTrue(
5757
sparkAppDriverConf.driverServiceName().contains(resourcePrefix),
58-
"Driver service " + "name should include secondary resource prefix");
58+
"Driver service name should include secondary resource prefix");
5959
}
6060

6161
@Test

0 commit comments

Comments
 (0)