Skip to content

Commit 39d1960

Browse files
committed
Add null checking for properties configuration
1 parent b87b854 commit 39d1960

5 files changed

Lines changed: 12 additions & 5 deletions

File tree

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
<groupId>com.applitools.imagetester</groupId>
66
<artifactId>ImageTester</artifactId>
7-
<version>3.6.1</version>
7+
<version>3.6.2</version>
88
<packaging>jar</packaging>
99
<properties>
1010
<maven.compiler.source>1.8</maven.compiler.source>

src/main/java/META-INF/MANIFEST.MF

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
Manifest-Version: 1.0
22
Main-Class: com.applitools.imagetester.ImageTester
3-
Implementation-Version: 3.6.1
3+
Implementation-Version: 3.6.2
44

src/main/java/com/applitools/imagetester/ImageTester.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
import com.applitools.imagetester.lib.Utils;
2727

2828
public class ImageTester {
29-
private static final String cur_ver = "3.6.1";
29+
private static final String cur_ver = "3.6.2";
3030

3131
public static void main(String[] args) {
3232

src/main/java/com/applitools/imagetester/TestObjects/TestBase.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,10 @@ public String name() {
5959

6060
public TestResults runSafe(Eyes eyes) {
6161
try {
62-
for(int i = 0; i < conf_.properties.length; i++) {
63-
eyes.addProperty(conf_.properties[i][0], conf_.properties[i][1]);
62+
if (conf_.properties != null && conf_.properties.length > 0) {
63+
for(int i = 0; i < conf_.properties.length; i++) {
64+
eyes.addProperty(conf_.properties[i][0], conf_.properties[i][1]);
65+
}
6466
}
6567
eyes.addProperty(FILE_NAME_PROP, file_.getName());
6668
TestResults res = run(eyes);

src/main/java/com/applitools/imagetester/lib/Config.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -244,6 +244,11 @@ public void setAccessibilityGraphicsRegions(String accessibilityGraphicsRegions)
244244
}
245245

246246
public void setProperties(String propArgument) {
247+
248+
if (propArgument == null || propArgument.isEmpty()) {
249+
return;
250+
}
251+
247252
boolean isValidFormat = Arrays.stream(propArgument.split("\\|"))
248253
.allMatch(s -> s.matches("[^:]+:[^:]+"));
249254

0 commit comments

Comments
 (0)