Skip to content

Commit 7e8f814

Browse files
committed
Add tests to increase test coverage
Signed-off-by: nxhafa <nxhafa11@gmail.com>
1 parent b534078 commit 7e8f814

2 files changed

Lines changed: 39 additions & 4 deletions

File tree

caching-service/src/main/java/org/zowe/apiml/caching/service/infinispan/config/InfinispanConfig.java

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -138,10 +138,13 @@ static String getRootFolder() {
138138
}
139139
}
140140

141+
public String getInfinispanConfigFile() {
142+
return isServerAttlsEnabled ? "infinispan-attls.xml" : "infinispan.xml";
143+
}
144+
141145
private ConfigurationBuilderHolder getCacheManagerConfig(ResourceLoader resourceLoader) {
142146
ConfigurationBuilderHolder holder;
143-
var infinispanConfigFile = isServerAttlsEnabled ? "infinispan-attls.xml" : "infinispan.xml";
144-
try (InputStream configurationStream = resourceLoader.getResource("classpath:" + infinispanConfigFile).getInputStream()) {
147+
try (InputStream configurationStream = resourceLoader.getResource("classpath:" + getInfinispanConfigFile()).getInputStream()) {
145148
holder = new ParserRegistry().parse(configurationStream, MediaType.APPLICATION_XML);
146149
} catch (IOException e) {
147150
throw new InfinispanConfigException("Can't read configuration file", e);

caching-service/src/test/java/org/zowe/apiml/caching/service/infinispan/config/InfinispanConfigTest.java

Lines changed: 34 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,35 @@ void whenKeystore_thenDontUpdate() {
5757
assertEquals("pass", ReflectionTestUtils.getField(infinispanConfig, "keyStorePass"));
5858
}
5959

60+
@Test
61+
void whenTruststoreKeyringUrlIsInvalidAndMissingPassword_thenFixTrustoreKeyringUrlAndSetPassword() {
62+
InfinispanConfig infinispanConfig = new InfinispanConfig();
63+
ReflectionTestUtils.setField(infinispanConfig, "trustStore", "safkeyringpce:///userId/ringId");
64+
infinispanConfig.updateKeyring();
65+
assertEquals("safkeyringpce://userId/ringId", ReflectionTestUtils.getField(infinispanConfig, "trustStore"));
66+
assertEquals("password", ReflectionTestUtils.getField(infinispanConfig, "trustStorePass"));
67+
}
68+
69+
@Test
70+
void whenTruststoreKeyringUrlIsInvalidAndSetPassword_thenFixTruststoreKeyringUrl() {
71+
InfinispanConfig infinispanConfig = new InfinispanConfig();
72+
ReflectionTestUtils.setField(infinispanConfig, "trustStore", "safkeyring:///userId/ringId");
73+
ReflectionTestUtils.setField(infinispanConfig, "trustStorePass", "pswd");
74+
infinispanConfig.updateKeyring();
75+
assertEquals("safkeyring://userId/ringId", ReflectionTestUtils.getField(infinispanConfig, "trustStore"));
76+
assertEquals("pswd", ReflectionTestUtils.getField(infinispanConfig, "trustStorePass"));
77+
}
78+
79+
@Test
80+
void whenTruststore_thenDontUpdate() {
81+
InfinispanConfig infinispanConfig = new InfinispanConfig();
82+
ReflectionTestUtils.setField(infinispanConfig, "trustStore", "/path");
83+
ReflectionTestUtils.setField(infinispanConfig, "trustStorePass", "pass");
84+
infinispanConfig.updateKeyring();
85+
assertEquals("/path", ReflectionTestUtils.getField(infinispanConfig, "trustStore"));
86+
assertEquals("pass", ReflectionTestUtils.getField(infinispanConfig, "trustStorePass"));
87+
}
88+
6089
}
6190

6291
@Nested
@@ -93,15 +122,13 @@ void givenNoEnvironmentValues_whenEvaluatingRootFolder_thenUseLocalhost() {
93122
void givenOnlyInstanceIdValues_whenEvaluatingRootFolder_thenUseRelativePath() {
94123
getEnvMap().put(INSTANCE, "myInstance");
95124
assertEquals("caching-service" + File.separator + "myInstance", InfinispanConfig.getRootFolder());
96-
97125
}
98126

99127
@Test
100128
void givenAllEnvironmentValues_whenEvaluatingRootFolder_thenUseExactLocation() {
101129
getEnvMap().put(INSTANCE, "lpar1");
102130
getEnvMap().put(WORKSPACE, "/some/path");
103131
assertEquals(File.separator + "some" + File.separator + "path" + File.separator + "caching-service" + File.separator + "lpar1", InfinispanConfig.getRootFolder());
104-
105132
}
106133

107134
@Test
@@ -111,6 +138,11 @@ void givenOnlyWorkspaceValues_whenEvaluatingRootFolder_thenUseExactLocationWithL
111138

112139
}
113140

141+
@Test
142+
void givenNoAttlsSetup_whenEvaluatingInfinispanConfigFile_thenReturnSslConfig() {
143+
assertEquals("infinispan.xml", new InfinispanConfig().getInfinispanConfigFile());
144+
}
145+
114146
}
115147

116148
}

0 commit comments

Comments
 (0)