Skip to content

Commit f87b60d

Browse files
committed
Fixed #2577
1 parent 317130c commit f87b60d

File tree

6 files changed

+16
-16
lines changed

6 files changed

+16
-16
lines changed

source/src/main/java/org/cerberus/core/engine/execution/impl/ExecutionRunService.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1610,9 +1610,9 @@ private TestCaseExecution stopExecutionRobotAndProxy(TestCaseExecution execution
16101610
// Stopping remote proxy.
16111611
try {
16121612
this.executorService.stopRemoteProxy(execution);
1613-
LOG.debug("Stop Cerberus Executor Proxy for execution {}", execution.getId());
1613+
LOG.debug("Stop Cerberus Robot Proxy for execution {}", execution.getId());
16141614
} catch (Exception exception) {
1615-
LOG.warn("Exception on Cerberus Executor Proxy stop for execution {}", execution.getId(), exception);
1615+
LOG.warn("Exception on Cerberus Robot Proxy stop for execution {}", execution.getId(), exception);
16161616
}
16171617

16181618
// Websocket --> we refresh the corresponding Detail Execution pages attached to this execution.

source/src/main/java/org/cerberus/core/engine/execution/impl/ExecutionStartService.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -621,7 +621,7 @@ public TestCaseExecution startExecution(TestCaseExecution execution) throws Cerb
621621
execution.setExecutor(execution.getTestCaseObj().getExecutor());
622622
}
623623

624-
//Stop the Cerberus Executor Proxy
624+
//Define websocket parameter
625625
execution.setCerberus_featureflipping_activatewebsocketpush(parameterService.getParameterBooleanByKey("cerberus_featureflipping_activatewebsocketpush", "", false));
626626
execution.setCerberus_featureflipping_websocketpushperiod(parameterService.getParameterLongByKey("cerberus_featureflipping_websocketpushperiod", "", 5000));
627627

source/src/main/java/org/cerberus/core/engine/execution/impl/RobotServerService.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -245,13 +245,13 @@ public void startServer(TestCaseExecution execution) throws CerberusException {
245245
LOG.debug("Session is set.");
246246

247247
/*
248-
* Starting Cerberus Executor Proxy if it has been activated at
248+
* Starting Cerberus Robot Proxy if it has been activated at
249249
* robot level.
250250
*/
251251
if (execution.getRobotExecutorObj() != null && RobotExecutor.PROXY_TYPE_NETWORKTRAFFIC.equals(execution.getRobotExecutorObj().getExecutorProxyType())) {
252-
LOG.debug("Start Remote Proxy");
252+
LOG.debug("Start Remote Cerberus Proxy");
253253
executorService.startRemoteProxy(execution);
254-
LOG.debug("Started Remote Proxy on port: {}", execution.getRemoteProxyPort());
254+
LOG.debug("Started Remote Cerberus Robot Proxy on port: {}", execution.getRemoteProxyPort());
255255
}
256256

257257
// SetUp Capabilities
@@ -1142,7 +1142,7 @@ public boolean stopServer(TestCaseExecution tce) {
11421142
default:
11431143
}
11441144
try {
1145-
// Get Har File when Cerberus Executor is activated.
1145+
// Get Har File when Cerberus Robot Proxy is activated.
11461146
// If proxy started and parameter verbose >= 1 activated
11471147
if (RobotExecutor.PROXY_TYPE_NETWORKTRAFFIC.equals(tce.getRobotExecutorObj().getExecutorProxyType())
11481148
&& tce.getVerbose() >= 1 && (parameterService.getParameterBooleanByKey("cerberus_networkstatsave_active", tce.getSystem(), false))) {
@@ -1176,7 +1176,7 @@ public boolean stopServer(TestCaseExecution tce) {
11761176
}
11771177
}
11781178
} catch (Exception ex) {
1179-
LOG.error("Exception Getting Har File from Cerberus Executor {}", tce.getId(), ex);
1179+
LOG.error("Exception Getting Har File from Cerberus Robot Proxy for execution id {}", tce.getId(), ex);
11801180
}
11811181

11821182
// We Stop the Robot Session (Selenium or Appium).

source/src/main/java/org/cerberus/core/enums/MessageGeneralEnum.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ public enum MessageGeneralEnum {
6666
VALIDATION_FAILED_TEST_NOT_FOUND(87, "FA", "Test '%TEST%' does not exist."),
6767
VALIDATION_FAILED_TEST_NOTACTIVE(88, "FA", "The test '%TEST%' isn't active."),
6868
VALIDATION_FAILED_USERAGENTDIFFERENT(90, "FA", "User Agent has been specified at robot and TestCase level with different values. TestCase : '%UATESTCASE%' Robot : '%UAROBOT%'"),
69-
VALIDATION_FAILED_CERBERUSEXECUTORNOTAVAILABLE(90, "FA", "Could not connect to Cerberus Executor host %HOST% on port %PORT%. Check the Cerberus Executor is up and running, or unselect the Proxy flag at robot executor level (robot : %ROBOT%, executor : %ROBOTEXE%)."),
69+
VALIDATION_FAILED_CERBERUSEXECUTORNOTAVAILABLE(90, "FA", "Could not connect to Cerberus Robot Proxy host '%HOST%' on port %PORT%. Check the Cerberus Robot Proxy is up and running, or unselect the Proxy flag at robot executor level (robot : %ROBOT%, executor : %ROBOTEXE%)."),
7070
VALIDATION_FAILED_KAFKACONSUMERSEEK(90, "FA", "Could not start execution because failed when trying to retreive Kafka topics latest offsets on Service %SERVICE%. %DETAIL%"),
7171
VALIDATION_FAILED_INSTANCE_INACTIVE(91, "FA", "This Cerberus Instance has been temporary disabled. Please try again later."),
7272
VALIDATION_SUCCEEDED(89, "PE", "The validation succeeded"),

source/src/main/java/org/cerberus/core/service/robotproxy/impl/RobotProxyService.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ public void startRemoteProxy(TestCaseExecution tce) {
7777
url += "&bsKey=" + tce.getRobotExecutorObj().getHostPassword();
7878
url += "&bsLocalIdentifier=" + tce.getExecutionUUID();
7979
}
80-
LOG.debug("Starting Proxy on Cerberus Executor calling : " + url);
80+
LOG.debug("Starting Cerberus Robot Proxy calling : '{}'", url);
8181

8282
try (InputStream is = new URL(url).openStream()) {
8383
BufferedReader rd = new BufferedReader(new InputStreamReader(is, Charset.forName("UTF-8")));
@@ -93,7 +93,7 @@ public void startRemoteProxy(TestCaseExecution tce) {
9393
tce.setRemoteProxyUUID(json.getString("uuid"));
9494
tce.setRemoteProxyStarted(true);
9595

96-
LOG.debug("Cerberus Executor Proxy extention started on port : " + tce.getRemoteProxyPort() + " (uuid : " + tce.getRemoteProxyUUID() + ")");
96+
LOG.debug("Cerberus Robot Proxy started on port : " + tce.getRemoteProxyPort() + " (uuid : " + tce.getRemoteProxyUUID() + ")");
9797

9898
} catch (Exception ex) {
9999
logEventService.createForPrivateCalls("", "EXEC", LogEvent.STATUS_ERROR, "Error when trying to open a remote proxy on Cerberus Robot Proxy. " + ex.toString());
@@ -108,25 +108,25 @@ public void stopRemoteProxy(TestCaseExecution tce) {
108108
if (tce.isRemoteProxyStarted()) {
109109
tce.setRemoteProxyStarted(false);
110110
/**
111-
* We Stop the Proxy on Cerberus Executor.
111+
* We Stop the Cerberus Robot Proxy.
112112
*/
113113
try {
114114
// Ask the Proxy to stop.
115115
if (tce.getRobotExecutorObj() != null && RobotExecutor.PROXY_TYPE_NETWORKTRAFFIC.equals(tce.getRobotExecutorObj().getExecutorProxyType())) {
116116

117117
String urlStop = "http://" + tce.getRobotExecutorObj().getExecutorExtensionHost() + ":" + tce.getRobotExecutorObj().getExecutorExtensionPort() + "/stopProxy?uuid=" + tce.getRemoteProxyUUID();
118118

119-
LOG.debug("Shutting down of Proxy on Cerberus Executor calling : " + urlStop);
119+
LOG.debug("Shutting down of Cerberus Robot Proxy calling : '{}'", urlStop);
120120

121121
InputStream is = new URL(urlStop).openStream();
122122
is.close();
123123

124-
LOG.debug("Cerberus Executor Proxy extention shutdown done (uuid : " + tce.getRemoteProxyUUID() + ").");
124+
LOG.debug("Cerberus Robot Proxy shutdown done (uuid : " + tce.getRemoteProxyUUID() + ").");
125125

126126
}
127127

128128
} catch (Exception ex) {
129-
LOG.error("Exception when asking Cerberus Executor proxy to stop " + tce.getId(), ex);
129+
LOG.error("Exception when asking Cerberus Robot proxy to stop " + tce.getId(), ex);
130130
}
131131
}
132132

source/src/main/resources/documentation/D1/include/en/testmanagement_tcactions_en.adoc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -811,7 +811,7 @@ Will allow you to calculate a *[red]#Property#* defined in the property section
811811
| *[green]#GUI#* | *[red]#SRV#* | *[green]#APK#* | *[green]#IPA#* | *[green]#FAT#* | *[red]#BAT#*
812812

813813
|===
814-
Retrieve the network traffic from Cerberus Executor (when activated at robot level) and calculate a json message that can be used for direct controls on all attached controls. If Robot Executor is not activated, Action will end in NE Status and all attached controls ignored. Once all controls of that action are finished, current content are set back to normal value (could be an html or app page or another service).
814+
Retrieve the network traffic from Cerberus Robot Proxy (when activated at robot level) and calculate a json message that can be used for direct controls on all attached controls. If Robot Executor is not activated, Action will end in NE Status and all attached controls ignored. Once all controls of that action are finished, current content are set back to normal value (could be an html or app page or another service).
815815

816816
*[blue]#Value1#* : URL to filter. Network requests will be filtered based on that value. Any url that contain the value1 will be included in the network traffic.
817817

0 commit comments

Comments
 (0)