Skip to content

Commit 6893772

Browse files
authored
Merge pull request #1929 from arunvenmany-ibm/liberty_debug_port_alias_fix
adding alias for debug parameters
2 parents 95ccb1c + 4ce98b7 commit 6893772

File tree

5 files changed

+10
-2
lines changed

5 files changed

+10
-2
lines changed

liberty-maven-plugin/src/it/dev-it/resources/basic-dev-project-umbrella-deps/pom.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,8 @@
123123
<default.https.port>${testServerHttpsPort}</default.https.port>
124124
<com.ibm.ws.logging.message.format>json</com.ibm.ws.logging.message.format>
125125
</bootstrapProperties>
126+
<libertyDebug>true</libertyDebug>
127+
<libertyDebugPort>8077</libertyDebugPort>
126128
</configuration>
127129
</plugin>
128130
<!-- Plugin to run functional tests -->

liberty-maven-plugin/src/it/dev-it/resources/basic-dev-project/pom.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -202,6 +202,8 @@
202202
<param>-Xms512m</param>
203203
</jvmOptions>
204204
<!-- ADDITIONAL_CONFIGURATION -->
205+
<debug>true</debug>
206+
<debugPort>7077</debugPort>
205207
</configuration>
206208
</plugin>
207209
<!-- Plugin to run functional tests -->

liberty-maven-plugin/src/it/dev-it/src/test/java/net/wasdev/wlp/test/dev/it/DevGenerateFeaturesDependenciesTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,8 @@ public static void cleanUpAfterClass() throws Exception {
4141

4242
@Test
4343
public void updateDependencyTest() throws Exception {
44+
//debugPort set as 8077
45+
assertTrue(verifyLogMessageExists("Listening for transport dt_socket at address: 8077", 20000) || verifyLogMessageExists("The debug port 8077 is not available.",20000));
4446
assertTrue(verifyLogMessageExists("Liberty is running in dev mode.", 10000));
4547

4648
File generatedFeaturesFile = getGeneratedFeaturesFile();

liberty-maven-plugin/src/it/dev-it/src/test/java/net/wasdev/wlp/test/dev/it/DevHotTestingTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,8 @@ public static void cleanUpAfterClass() throws Exception {
3434

3535
@Test
3636
public void autoTestsInvocationTest() throws Exception {
37+
//debugPort set as 7077
38+
assertTrue(verifyLogMessageExists("Listening for transport dt_socket at address: 7077", 20000) || verifyLogMessageExists("The debug port 7077 is not available.",20000));
3739
assertTrue(verifyLogMessageExists("Recompile skipped for dev-sample-proj since earlier compilation is successful", 20000));
3840
assertTrue(verifyLogMessageExists("Tests will run automatically", 20000));
3941

liberty-maven-plugin/src/main/java/io/openliberty/tools/maven/server/DevMojo.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -126,10 +126,10 @@ public class DevMojo extends LooseAppSupport {
126126
@Parameter(property = "skipInstallFeature", defaultValue = "false")
127127
protected boolean skipInstallFeature;
128128

129-
@Parameter(property = "debug", defaultValue = "true")
129+
@Parameter(property = "debug", defaultValue = "true", alias = "debug")
130130
private boolean libertyDebug;
131131

132-
@Parameter(property = "debugPort", defaultValue = "7777")
132+
@Parameter(property = "debugPort", defaultValue = "7777", alias = "debugPort")
133133
private int libertyDebugPort;
134134

135135
@Parameter(property = "container", defaultValue = "false")

0 commit comments

Comments
 (0)