Fix some code issues.

python012 · python012 · commit 7486fe3f2b0c · 2017-10-24T22:22:28.000+08:00
diff --git a/src/com/semet/SetMetWebDriverSession.java b/src/com/semet/SetMetWebDriverSession.java
@@ -18,4 +18,13 @@ public static RemoteWebDriver get() {
     public static void set(RemoteWebDriver instance) {
         rwd.set(instance);
     }
+
+    public static void pause(long i) {
+        try {
+            Thread.sleep(i);
+        } catch (InterruptedException e) {
+            log.error("InterruptedException in pause()");
+            log.error(e.getMessage());
+        }
+    }
 }
diff --git a/src/com/solvent/util/Configurator.java b/src/com/solvent/util/Configurator.java
@@ -29,7 +29,8 @@ public class Configurator {
         try {
             defaults.load(Configurator.class.getResourceAsStream(DEFAULT_PROPERTIES_RESOURCE));
         } catch (IOException e) {
-            throw new ConfigurationError(e.getMessage(), e);
+//            throw new ConfigurationError(e.getMessage(), e);
+            log.error("ConfigurationError: " + e.getMessage());
         }
         if (null != propFileName) {
             File propFile = new File(propFileName);
@@ -39,8 +40,10 @@ public class Configurator {
                 props.load(new FileInputStream(propFile));
                 initialize(props);
             } catch (IOException e) {
-                throw new ConfigurationError("Error while loading configuration file: " + propFile.getAbsolutePath(),
-                        e);
+//                throw new ConfigurationError("Error while loading configuration file: " + propFile.getAbsolutePath(),
+//                        e);
+                log.error("Error while loading configuration file: " + propFile.getAbsolutePath());
+                log.error("ConfigurationError: " + e.getMessage());
             }
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -18,4 +18,13 @@ public static RemoteWebDriver get() {`
`18`	`18`	`public static void set(RemoteWebDriver instance) {`
`19`	`19`	`rwd.set(instance);`
`20`	`20`	`}`
	`21`	`+`
	`22`	`+ public static void pause(long i) {`
	`23`	`+ try {`
	`24`	`+ Thread.sleep(i);`
	`25`	`+ } catch (InterruptedException e) {`
	`26`	`+ log.error("InterruptedException in pause()");`
	`27`	`+ log.error(e.getMessage());`
	`28`	`+ }`
	`29`	`+ }`
`21`	`30`	`}`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,8 @@ public class Configurator {`
`29`	`29`	`try {`
`30`	`30`	`defaults.load(Configurator.class.getResourceAsStream(DEFAULT_PROPERTIES_RESOURCE));`
`31`	`31`	`} catch (IOException e) {`
`32`		`- throw new ConfigurationError(e.getMessage(), e);`
	`32`	`+// throw new ConfigurationError(e.getMessage(), e);`
	`33`	`+ log.error("ConfigurationError: " + e.getMessage());`
`33`	`34`	`}`
`34`	`35`	`if (null != propFileName) {`
`35`	`36`	`File propFile = new File(propFileName);`
`@@ -39,8 +40,10 @@ public class Configurator {`
`39`	`40`	`props.load(new FileInputStream(propFile));`
`40`	`41`	`initialize(props);`
`41`	`42`	`} catch (IOException e) {`
`42`		`- throw new ConfigurationError("Error while loading configuration file: " + propFile.getAbsolutePath(),`
`43`		`- e);`
	`43`	`+// throw new ConfigurationError("Error while loading configuration file: " + propFile.getAbsolutePath(),`
	`44`	`+// e);`
	`45`	`+ log.error("Error while loading configuration file: " + propFile.getAbsolutePath());`
	`46`	`+ log.error("ConfigurationError: " + e.getMessage());`
`44`	`47`	`}`
`45`	`48`	`}`
`46`	`49`	`}`