Updated external parameter/property fetch function.

ansonliao · ansonliao · commit 6a7ddf601e77 · 2017-11-30T15:34:33.000+08:00
diff --git a/README.md b/README.md
@@ -19,7 +19,7 @@ Add the below dependencies in your `pom.xml` (Master)
 <dependency>
     <groupId>com.github.ansonliao</groupId>
     <artifactId>Selenium-Extensions</artifactId>
-    <version>2.3.0-SNAPSHOT</version>
+    <version>2.3.0</version>
 </dependency>
 ```
 
diff --git a/pom.xml b/pom.xml
@@ -4,7 +4,7 @@
 
   <groupId>com.github.ansonliao</groupId>
   <artifactId>Selenium-Extensions</artifactId>
-  <version>2.3.0-SNAPSHOT</version>
+  <version>2.3.0</version>
   <packaging>jar</packaging>
 
   <name>Selenium-Extensions</name>
diff --git a/src/main/java/com/github/ansonliao/selenium/factory/ChromeFactory.java b/src/main/java/com/github/ansonliao/selenium/factory/ChromeFactory.java
@@ -1,5 +1,6 @@
 package com.github.ansonliao.selenium.factory;
 
+import com.github.ansonliao.selenium.utils.SEFilterUtils;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.chrome.ChromeDriver;
 import org.openqa.selenium.chrome.ChromeOptions;
@@ -36,7 +37,7 @@ public WebDriver getDriver() {
 
     @Override
     public String getExportParameterKey() {
-        return "webdriver.chrome.driver";
+        return SEFilterUtils.getChromeDriverExportKey();
     }
 
     @Override
diff --git a/src/main/java/com/github/ansonliao/selenium/factory/EdgeFactory.java b/src/main/java/com/github/ansonliao/selenium/factory/EdgeFactory.java
@@ -1,5 +1,6 @@
 package com.github.ansonliao.selenium.factory;
 
+import com.github.ansonliao.selenium.utils.SEFilterUtils;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.edge.EdgeDriver;
 import org.slf4j.Logger;
@@ -26,7 +27,7 @@ public WebDriver getDriver() {
 
     @Override
     public String getExportParameterKey() {
-        return "webdriver.edge.driver";
+        return SEFilterUtils.getEdgeDriverExportKey();
     }
 
     @Override
diff --git a/src/main/java/com/github/ansonliao/selenium/factory/FirefoxFactory.java b/src/main/java/com/github/ansonliao/selenium/factory/FirefoxFactory.java
@@ -1,5 +1,6 @@
 package com.github.ansonliao.selenium.factory;
 
+import com.github.ansonliao.selenium.utils.SEFilterUtils;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.firefox.FirefoxBinary;
 import org.openqa.selenium.firefox.FirefoxDriver;
@@ -38,7 +39,7 @@ public WebDriver getDriver() {
 
     @Override
     public String getExportParameterKey() {
-        return "webdriver.gecko.driver";
+        return SEFilterUtils.getFirefoxDriverExportKey();
     }
 
     @Override
diff --git a/src/main/java/com/github/ansonliao/selenium/factory/InternetExplorerFactory.java b/src/main/java/com/github/ansonliao/selenium/factory/InternetExplorerFactory.java
@@ -1,5 +1,6 @@
 package com.github.ansonliao.selenium.factory;
 
+import com.github.ansonliao.selenium.utils.SEFilterUtils;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.ie.InternetExplorerDriver;
 import org.slf4j.Logger;
@@ -26,7 +27,7 @@ public WebDriver getDriver() {
 
     @Override
     public String getExportParameterKey() {
-        return "webdriver.ie.driver";
+        return SEFilterUtils.getIEDriverExportKey();
     }
 
     @Override
diff --git a/src/main/java/com/github/ansonliao/selenium/factory/OperaFactory.java b/src/main/java/com/github/ansonliao/selenium/factory/OperaFactory.java
@@ -1,5 +1,6 @@
 package com.github.ansonliao.selenium.factory;
 
+import com.github.ansonliao.selenium.utils.SEFilterUtils;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.opera.OperaDriver;
 import org.slf4j.Logger;
@@ -26,7 +27,7 @@ public WebDriver getDriver() {
 
     @Override
     public String getExportParameterKey() {
-        return "webdriver.opera.driver";
+        return SEFilterUtils.getOperaDriverExportKey();
     }
 
     @Override
diff --git a/src/main/java/com/github/ansonliao/selenium/factory/PhantomJsFactory.java b/src/main/java/com/github/ansonliao/selenium/factory/PhantomJsFactory.java
@@ -1,5 +1,6 @@
 package com.github.ansonliao.selenium.factory;
 
+import com.github.ansonliao.selenium.utils.SEFilterUtils;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.phantomjs.PhantomJSDriver;
 import org.slf4j.Logger;
@@ -26,7 +27,7 @@ public WebDriver getDriver() {
 
     @Override
     public String getExportParameterKey() {
-        return "phantomjs.binary.path";
+        return SEFilterUtils.getPhantomJsDriverExportKey();
     }
 
     @Override
diff --git a/src/main/java/com/github/ansonliao/selenium/parallel/SeleniumParallelTestListener.java b/src/main/java/com/github/ansonliao/selenium/parallel/SeleniumParallelTestListener.java
@@ -2,12 +2,11 @@
 
 import com.github.ansonliao.selenium.annotations.URL;
 import com.github.ansonliao.selenium.factory.DriverManagerFactory;
-import com.github.ansonliao.selenium.factory.WDManager;
 import com.github.ansonliao.selenium.internal.Constants;
 import com.github.ansonliao.selenium.report.factory.ExtentTestManager;
 import com.github.ansonliao.selenium.utils.AuthorUtils;
 import com.github.ansonliao.selenium.utils.MyFileUtils;
-import com.github.ansonliao.selenium.utils.SEConfig;
+import com.github.ansonliao.selenium.utils.SEFilterUtils;
 import com.github.ansonliao.selenium.utils.TestGroupUtils;
 import com.github.ansonliao.selenium.utils.WDMHelper;
 import com.google.common.base.Strings;
@@ -44,7 +43,7 @@ public void beforeInvocation(IInvokedMethod iInvokedMethod, ITestResult iTestRes
                 iInvokedMethod.getTestMethod().getConstructorOrMethod().getMethod();
 
         List<String> groups = TestGroupUtils.getMethodTestGroups(method);
-        if (SEConfig.getBoolean("addBrowserGroupToReport")) {
+        if (SEFilterUtils.addBrowserGroupToReport()) {
             groups.add(browserName);
         }
 
diff --git a/src/main/java/com/github/ansonliao/selenium/report/factory/ExtentTestManager.java b/src/main/java/com/github/ansonliao/selenium/report/factory/ExtentTestManager.java
@@ -6,13 +6,10 @@
 import com.github.ansonliao.selenium.annotations.Description;
 import com.google.common.collect.Maps;
 import org.testng.Reporter;
-import org.testng.annotations.Test;
 
 import java.lang.reflect.Method;
-import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.Optional;
 
 public class ExtentTestManager {
     public static ThreadLocal<ExtentTest> extentTests = new ThreadLocal<>();
diff --git a/src/main/java/com/github/ansonliao/selenium/testng/TestNGFilter.java b/src/main/java/com/github/ansonliao/selenium/testng/TestNGFilter.java
@@ -1,11 +1,9 @@
 package com.github.ansonliao.selenium.testng;
 
-import com.github.ansonliao.selenium.internal.Variables;
 import com.github.ansonliao.selenium.parallel.ClassFinder;
 import com.github.ansonliao.selenium.parallel.MethodFinder;
 import com.github.ansonliao.selenium.utils.BrowserUtils;
-import com.github.ansonliao.selenium.utils.SEConfig;
-import com.google.common.base.Strings;
+import com.github.ansonliao.selenium.utils.SEFilterUtils;
 import com.google.common.collect.HashMultimap;
 import com.google.common.collect.Multimap;
 import com.google.common.collect.Sets;
@@ -16,7 +14,6 @@
 
 import java.lang.reflect.Method;
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.Collection;
 import java.util.HashMap;
 import java.util.List;
@@ -28,11 +25,10 @@ public class TestNGFilter {
     private static final Logger logger = LoggerFactory.getLogger(TestNGFilter.class);
     private static List<Class<?>> testngClasses = Lists.newArrayList();
     private static Multimap<Class<?>, Method> testNGClass2MethodMap = HashMultimap.create();
-    private static final String EXTERNAL_RUN_BROWSER_KEY = "runByBrowsers";
 
     static {
-        String[] testingPackageNames = Variables.TESTING_PACKAGE_NAMES.toArray(
-                new String[Variables.TESTING_PACKAGE_NAMES.size()]);
+        String[] testingPackageNames = SEFilterUtils.testingPackageNames().toArray(
+                new String[SEFilterUtils.testingPackageNames().size()]);
         testngClasses = ClassFinder.findAllTestNGTestClasses(testingPackageNames);
     }
 
@@ -45,10 +41,10 @@ public static Map<String, Map<Class<?>, List<Method>>> run() {
 
     // filter testng classes by given testng class list
     public static List<Class<?>> filterClassesByTestingClassList() {
-        if (!Variables.TESTING_TESTNG_CLASSES.isEmpty()) {
+        if (!SEFilterUtils.testingTestNGClasses().isEmpty()) {
             return testngClasses.parallelStream()
                     .filter(aClass ->
-                            Variables.TESTING_TESTNG_CLASSES
+                            SEFilterUtils.testingTestNGClasses()
                                     .parallelStream()
                                     .map(String::toUpperCase)
                                     .collect(Collectors.toList())
@@ -66,7 +62,7 @@ public static List<Class<?>> filterClassesByTestingClassList() {
      */
     public static Multimap<Class<?>, Method> filterTestNGMethodByTestGroups() {
         testNGClass2MethodMap = HashMultimap.create();
-        if (Strings.isNullOrEmpty(SEConfig.getString("testingTestGroups"))) {
+        if (SEFilterUtils.testingTestGroups().isEmpty()) {
             testngClasses.stream()
                     .forEach(aClass ->
                             MethodFinder.findAllAnnotatedTestMethodInClass(aClass).stream()
@@ -76,12 +72,8 @@ public static Multimap<Class<?>, Method> filterTestNGMethodByTestGroups() {
             return testNGClass2MethodMap;
         }
 
-        Variables.TESTING_TEST_GROUPS =
-                Arrays.stream(SEConfig.getString("testingTestGroups").split(","))
-                .map(String::trim).collect(Collectors.toList());
-
         testngClasses.stream().forEach(aClass -> {
-            List<Method> methodList = Variables.TESTING_TEST_GROUPS
+            List<Method> methodList = SEFilterUtils.testingTestGroups()
                     .parallelStream()
                     .map(group -> MethodFinder.findTestNGMethodInClassByGroup(aClass, group))
                     .flatMap(Collection::stream)
@@ -98,12 +90,12 @@ public static Multimap<Class<?>, Method> filterTestNGMethodByTestGroups() {
 
     // filter testng method by given test browser list
     public static Multimap<Class<?>, Method> filterTestNGMethodByBrowsers() {
-        if (!Variables.TESTING_BROWSER_NAMES.isEmpty()) {
+        if (!SEFilterUtils.testingBrowserNames().isEmpty()) {
             testNGClass2MethodMap.keySet().parallelStream().forEach(aClass ->
                     testNGClass2MethodMap.get(aClass).stream().forEach(method -> {
                         Sets.SetView result = Sets.intersection(
                                 BrowserUtils.getMethodSupportedBrowsers(method),
-                                Sets.newHashSet(Variables.TESTING_BROWSER_NAMES));
+                                Sets.newHashSet(SEFilterUtils.testingBrowserNames()));
                         if (result.size() == 0) {
                             if (testNGClass2MethodMap.get(aClass).contains(method)) {
                                 testNGClass2MethodMap.get(aClass).remove(method);
@@ -119,12 +111,11 @@ public static Map<String, Map<Class<?>, List<Method>>> getBrowser2TestNGClass2Te
         Map<String, Map<Class<?>, List<Method>>> browserTestingMap = new HashMap<>();
         Multimap<String, Method> browserTestngMethodMap = HashMultimap.create();
 
-        if (!Strings.isNullOrEmpty(SEConfig.getString(EXTERNAL_RUN_BROWSER_KEY))) {
+        if (!SEFilterUtils.runByBrowsers().isEmpty()) {
             // only fetch support browsers
             List<String> browsers =
                     Sets.intersection(
-                            Arrays.stream(SEConfig.getString(EXTERNAL_RUN_BROWSER_KEY).split(","))
-                                    .map(String::trim).collect(Collectors.toSet()),
+                            SEFilterUtils.runByBrowsers().stream().collect(Collectors.toSet()),
                             Sets.newHashSet(BrowserUtils.getSupportedBrowsers()))
                     .parallelStream().collect(Collectors.toList());
 
@@ -185,7 +176,7 @@ public static Map<String, Map<Class<?>, List<Method>>> getBrowser2TestNGClass2Te
     }
 
     private static Set<String> addIncludedDefaultBrowser(Method method) {
-        String DEFAULT_BROWSER_TYPE_NAME = Variables.DEFAULT_BROWSER_TYPE_NAME.trim().toUpperCase();
+        String DEFAULT_BROWSER_TYPE_NAME = SEFilterUtils.defaultBrowser().toUpperCase();
         Set<String> methodSupportedBrowsers = BrowserUtils.getMethodSupportedBrowsers(method);
         Set<String> methodBrowsers = BrowserUtils.getMethodBrowsers(method);
         Set<String> ignoreBrowsers = BrowserUtils.getMethodIgnoredBrowsers(method);
diff --git a/src/main/java/com/github/ansonliao/selenium/testng/XmlTestBuilder.java b/src/main/java/com/github/ansonliao/selenium/testng/XmlTestBuilder.java
@@ -1,7 +1,7 @@
 package com.github.ansonliao.selenium.testng;
 
 import com.github.ansonliao.selenium.internal.Constants;
-import com.github.ansonliao.selenium.internal.Variables;
+import com.github.ansonliao.selenium.utils.SEFilterUtils;
 import com.google.common.collect.Lists;
 import com.google.common.collect.Multimap;
 import com.google.common.collect.Sets;
@@ -20,13 +20,13 @@ public class XmlTestBuilder {
     public static List<XmlTest> build() {
         Set<XmlTest> xmlTestList = Sets.newHashSet();
         Multimap<String, XmlClass> browserXmlclassMap = XmlClassBuilder.build();
-        final int DEFAULT_TEST_CLASS_SIZE = Variables.DEFAULT_TEST_CLASS_SIZE_OF_TESTNG_XML;
+        final int DEFAULT_TEST_CLASS_SIZE = SEFilterUtils.defaultTestClassesSizeOfTestNGXML();
 
         browserXmlclassMap.keySet().forEach(browserName -> {
             List<XmlClass> xmlClassList = browserXmlclassMap.get(browserName)
                     .stream().distinct().collect(Collectors.toList());
             ArrayList<XmlClass> tempXmlClass = Lists.newArrayList(xmlClassList);
-            int xmlClassGroupSize = xmlClassList.size() / Variables.DEFAULT_TEST_CLASS_SIZE_OF_TESTNG_XML + 1;
+            int xmlClassGroupSize = xmlClassList.size() / SEFilterUtils.defaultTestClassesSizeOfTestNGXML() + 1;
             int counter = xmlClassGroupSize;
 
             int startIndex = 0;
diff --git a/src/main/java/com/github/ansonliao/selenium/utils/AuthorUtils.java b/src/main/java/com/github/ansonliao/selenium/utils/AuthorUtils.java
@@ -20,10 +20,9 @@
 
 public class AuthorUtils {
     private static final Logger logger = LoggerFactory.getLogger(AuthorUtils.class);
-    private static JavaProjectBuilder javaProjectBuilder;
+    private static JavaProjectBuilder javaProjectBuilder = new JavaProjectBuilder();
 
     static {
-        javaProjectBuilder = new JavaProjectBuilder();
         javaProjectBuilder.addSourceTree(new File(
                 Constants.PROJECT_ROOT_DIR
                         .concat(Constants.FILE_SEPARATOR)
@@ -39,7 +38,7 @@ public synchronized static List<String> getMethodAuthors(Method method) {
 
         // get class authors
         getClassAuthors(clazz)
-                .ifPresent(authorList -> classAuthors.addAll(authorList));
+                .ifPresent(classAuthors::addAll);
 
         // get method authors
         javaProjectBuilder.getClassByName(className).getMethods().stream()
diff --git a/src/main/java/com/github/ansonliao/selenium/utils/BrowserUtils.java b/src/main/java/com/github/ansonliao/selenium/utils/BrowserUtils.java
@@ -23,7 +23,8 @@ public static List<String> getSupportedBrowsers() {
                 .filter(className ->
                         className.startsWith(Constants.BROWSER_ANNOTATION_PACKAGE))
                 .map(ClassFinder::createClass)
-                .filter(aClass -> aClass.isAnnotation())
+                //.filter(aClass -> aClass.isAnnotation())
+                .filter(Class::isAnnotation)
                 .map(aClass -> aClass.getSimpleName().toUpperCase())
                 .filter(className ->
                         !className.startsWith(Constants.BROWSER_IGNORE_ANNOTATION_PREFIX))
@@ -37,7 +38,8 @@ public static List<String> getSupportedIgnoreBrowsers() {
                 .filter(className ->
                         className.startsWith(Constants.BROWSER_ANNOTATION_PACKAGE))
                 .map(ClassFinder::createClass)
-                .filter(aClass -> aClass.isAnnotation())
+                //.filter(aClass -> aClass.isAnnotation())
+                .filter(Class::isAnnotation)
                 .map(aClass -> aClass.getSimpleName().toUpperCase())
                 .filter(className ->
                         className.startsWith(Constants.BROWSER_IGNORE_ANNOTATION_PREFIX))
@@ -82,7 +84,9 @@ public static Set<String> getMethodSupportedBrowsers(Class<?> clazz, Method meth
         enabledBrowsers = Sets.union(classSupportedBrowsers, enabledBrowsers);
         ignoredBrowsers = Sets.union(classIgnoredBrowsers, ignoredBrowsers);
 
-        return Sets.difference(enabledBrowsers, Sets.intersection(enabledBrowsers, ignoredBrowsers))
+        return Sets.difference(
+                enabledBrowsers,
+                Sets.intersection(enabledBrowsers, ignoredBrowsers))
                 .parallelStream().collect(Collectors.toSet());
     }
 
diff --git a/src/main/java/com/github/ansonliao/selenium/utils/MyFileUtils.java b/src/main/java/com/github/ansonliao/selenium/utils/MyFileUtils.java
@@ -2,10 +2,13 @@
 
 import com.github.ansonliao.selenium.internal.Constants;
 import org.apache.commons.io.FileUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.io.File;
 
 public class MyFileUtils extends FileUtils {
+    private static final Logger logger = LoggerFactory.getLogger(MyFileUtils.class);
 
     public static synchronized File createScreenshotFolderForBrowser(
             Class clazz, String browserName) {
@@ -19,6 +22,7 @@ public static synchronized File createScreenshotFolderForBrowser(
             return new File(destDir);
         }
 
+        logger.info("Create screenshot directory: {}", destDir);
         new File(destDir).mkdirs();
         return new File(destDir);
     }
diff --git a/src/main/java/com/github/ansonliao/selenium/utils/SEConfig.java b/src/main/java/com/github/ansonliao/selenium/utils/SEConfig.java
@@ -31,7 +31,9 @@ public static String getString(String key) {
             }
 
         }
-        return value;
+        return value.contains("\"") || value.contains("'")
+                ? value.replace("\"", "").replace("'", "")
+                : value;
     }
 
     public static int getInt(String key) {
diff --git a/src/main/java/com/github/ansonliao/selenium/utils/SEFilterUtils.java b/src/main/java/com/github/ansonliao/selenium/utils/SEFilterUtils.java
diff --git a/src/main/java/com/github/ansonliao/selenium/utils/TestGroupUtils.java b/src/main/java/com/github/ansonliao/selenium/utils/TestGroupUtils.java
diff --git a/src/main/java/com/github/ansonliao/selenium/utils/WDMHelper.java b/src/main/java/com/github/ansonliao/selenium/utils/WDMHelper.java
