Skip to content

Navigation Menu

Sign in

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit f45233c

Browse filesBrowse files
sayemssayems
committed
Merge pull request #2 from georgekankava/staging/throwable-print-stack-trace-should-not-be-called-fix-1
squid:S1148 Throwable.printStackTrace(...) should not be called
2 parents c93722f + 0b9b5e7 commit f45233c
Copy full SHA for f45233c

File tree

5 files changed

+18
-8
lines changed
Filter options

5 files changed

+18
-8
lines changed

‎src/main/java/org/sayem/webdriver/algorithm/Retry.java

Copy file name to clipboardExpand all lines: src/main/java/org/sayem/webdriver/algorithm/Retry.java
+7-2
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,17 @@
22

33
import java.util.concurrent.TimeUnit;
44

5+
import org.slf4j.Logger;
6+
7+
import static org.slf4j.LoggerFactory.getLogger;
8+
59
/**
610
* Created by sayem on 10/05/15.
711
*/
812
@SuppressWarnings("Duplicates")
913
public class Retry {
1014

15+
private static final Logger log = getLogger(Retry.class);
1116
private final long interval;
1217
private final TimeUnit unit;
1318
private boolean on;
@@ -76,7 +81,7 @@ private <T> T perform(Attemptable<T> task) throws Exception {
7681
return attempt;
7782
} catch (Exception e) {
7883
on();
79-
e.printStackTrace();
84+
log.error("Exception while execution", e);
8085
if (count == 0) {
8186
throw e;
8287
}
@@ -92,7 +97,7 @@ private void rest() {
9297
try {
9398
unit.sleep(interval);
9499
} catch (InterruptedException e) {
95-
e.printStackTrace();
100+
log.error("Thread has been interrupted", e);
96101
}
97102
}
98103

‎src/main/java/org/sayem/webdriver/browsers/config/BrowserThreads.java

Copy file name to clipboardExpand all lines: src/main/java/org/sayem/webdriver/browsers/config/BrowserThreads.java
+1-1
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ private void instantiateWebDriver(DesiredCapabilities desiredCapabilities) {
119119
try {
120120
seleniumGridURL = new URL(System.getProperty("gridURL"));
121121
} catch (MalformedURLException e) {
122-
e.printStackTrace();
122+
log.error("Either no legal protocol could be found or the string could not be parsed.", e);
123123
}
124124
String desiredBrowserVersion = System.getProperty("desiredBrowserVersion");
125125
String desiredPlatform = System.getProperty("desiredPlatform");

‎src/main/java/org/sayem/webdriver/listeners/ScreenshotListener.java

Copy file name to clipboardExpand all lines: src/main/java/org/sayem/webdriver/listeners/ScreenshotListener.java
+1-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ private boolean createFile(File screenshot) {
3333
try {
3434
fileCreated = screenshot.createNewFile();
3535
} catch (IOException errorCreatingScreenshot) {
36-
errorCreatingScreenshot.printStackTrace();
36+
log.error("failed or interrupted I/O operations", errorCreatingScreenshot);
3737
}
3838
}
3939
}

‎src/main/java/org/sayem/webdriver/properties/PropertiesUtil.java

Copy file name to clipboardExpand all lines: src/main/java/org/sayem/webdriver/properties/PropertiesUtil.java
+8-3
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,19 @@
11
package org.sayem.webdriver.properties;
22

3+
import static org.slf4j.LoggerFactory.getLogger;
4+
35
import java.io.FileInputStream;
46
import java.io.IOException;
57
import java.lang.reflect.InvocationTargetException;
68
import java.util.Properties;
79

10+
import org.slf4j.Logger;
11+
812
/**
913
* Created by sayem on 12/28/15.
1014
*/
1115
public class PropertiesUtil {
16+
private static final Logger log = getLogger(PropertiesUtil.class);
1217
private String dataLocation;
1318
private Properties properties;
1419

@@ -44,7 +49,7 @@ private void loadProperties() {
4449
properties = new Properties();
4550
properties.load(fileInputStream);
4651
} catch (IOException e) {
47-
e.printStackTrace();
52+
log.error("failed or interrupted I/O operations", e);;
4853
}
4954
}
5055

@@ -55,7 +60,7 @@ public String getProperties(String name) {
5560
prop = properties.getProperty(name);
5661

5762
} catch (Exception e) {
58-
e.printStackTrace();
63+
log.error("Exception while execution", e);
5964
}
6065
return prop;
6166
}
@@ -68,7 +73,7 @@ public <T> T getProperties(String name, Class<T> type) {
6873
prop = valueOf(type, temp);
6974

7075
} catch (Exception e) {
71-
e.printStackTrace();
76+
log.error("Exception while execution", e);
7277
}
7378
return prop;
7479
}

‎src/main/java/org/sayem/webdriver/selenium/Browser.java

Copy file name to clipboardExpand all lines: src/main/java/org/sayem/webdriver/selenium/Browser.java
+1-1
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ public void delayFor(int millis) {
121121
try {
122122
Thread.sleep(millis);
123123
} catch (InterruptedException e) {
124-
e.printStackTrace();
124+
log.error("Thread has been interrupted", e);
125125
}
126126
}
127127

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.