Grid: Fixing timeout issue again... Why isn't it covered by tests? A rhetorical question...

barancev · barancev · commit bf66042eb2ae · 2016-01-29T00:06:09.000+03:00
diff --git a/java/server/src/org/openqa/grid/internal/utils/SelfRegisteringRemote.java b/java/server/src/org/openqa/grid/internal/utils/SelfRegisteringRemote.java
@@ -63,7 +63,7 @@ public SelfRegisteringRemote(RegistrationRequest config) {
     try {
       JsonObject hubParameters = getHubConfiguration();
       if (hubParameters.has(RegistrationRequest.TIME_OUT)){
-        int timeout = hubParameters.get(RegistrationRequest.TIME_OUT).getAsInt() / 1000;
+        int timeout = hubParameters.get(RegistrationRequest.TIME_OUT).getAsInt();
         nodeConfig.getConfiguration().put(RegistrationRequest.TIME_OUT, timeout);
       }
       if (hubParameters.has(RegistrationRequest.BROWSER_TIME_OUT)) {

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ public SelfRegisteringRemote(RegistrationRequest config) {`
`63`	`63`	`try {`
`64`	`64`	`JsonObject hubParameters = getHubConfiguration();`
`65`	`65`	`if (hubParameters.has(RegistrationRequest.TIME_OUT)){`
`66`		`- int timeout = hubParameters.get(RegistrationRequest.TIME_OUT).getAsInt() / 1000;`
	`66`	`+ int timeout = hubParameters.get(RegistrationRequest.TIME_OUT).getAsInt();`
`67`	`67`	`nodeConfig.getConfiguration().put(RegistrationRequest.TIME_OUT, timeout);`
`68`	`68`	`}`
`69`	`69`	`if (hubParameters.has(RegistrationRequest.BROWSER_TIME_OUT)) {`