Comment on revision r17520 in selenium

[sele...@googlecode.com]

Comment by a.u.savc...@gmail.com:

Score: Neutral

General Comment:
Why was it done?
It leads to filures at starting hub/nodes like:

INFO: Launching a selenium grid node
org.openqa.grid.common.exception.GridConfigurationException: Error with the
JSON of the config : defaults/DefaultNode.json is not a valid resource.
at
org.openqa.grid.common.RegistrationRequest.loadFromJSON(RegistrationRequest.java:585)
at
org.openqa.grid.common.RegistrationRequest.build(RegistrationRequest.java:379)
at org.openqa.grid.selenium.GridLauncher.main(GridLauncher.java:70)
Caused by: java.lang.RuntimeException: defaults/DefaultNode.json is not a
valid resource.
at
org.openqa.grid.common.JSONConfigurationUtils.loadJSON(JSONConfigurationUtils.java:53)
at
org.openqa.grid.common.RegistrationRequest.loadFromJSON(RegistrationRequest.java:550)
... 2 more
Exception in thread "main" java.lang.ExceptionInInitializerError
at org.openqa.grid.selenium.GridLauncher.main(GridLauncher.java:76)
Caused by: java.lang.RuntimeException:
bug.defaults/GridParameters.properties cannot be loaded.
at org.openqa.grid.common.GridDocHelper.load(GridDocHelper.java:106)
at org.openqa.grid.common.GridDocHelper.<clinit>(GridDocHelper.java:26)
... 1 more


Please fix it by reverting changes or by fix another point of .desc :)

For more information:
https://code.google.com/p/selenium/source/detail?r=17520

[sele...@googlecode.com]

Comment by dawa...@gmail.com:

General Comment:
Reverted as revision 17556, thanks for the report