Merge pull request #32 from AutomateThePlanet/smpInternal

SMP Release

Author: n1xan

bellatrix.android/src/main/java/solutions/bellatrix/android/infrastructure/DriverService.java

@@ -19,7 +19,6 @@
 import io.appium.java_client.remote.MobileCapabilityType;
 import lombok.SneakyThrows;
 import org.openqa.selenium.MutableCapabilities;
-import org.openqa.selenium.Platform;
 import org.openqa.selenium.remote.DesiredCapabilities;
 import solutions.bellatrix.android.configuration.AndroidSettings;
 import solutions.bellatrix.android.configuration.GridSettings;
@@ -30,11 +29,10 @@
 
 import java.io.IOException;
 import java.io.InputStream;
-import java.net.MalformedURLException;
 import java.net.URL;
+import java.time.Duration;
 import java.util.HashMap;
 import java.util.Properties;
-import java.util.concurrent.TimeUnit;
 
 public class DriverService {
     private static final ThreadLocal<Boolean> DISPOSED;
@@ -84,7 +82,7 @@ public static AndroidDriver start(AppConfiguration configuration) {
             driver = initializeDriverGridMode(gridSettings.get(), testName);
         }
 
-        driver.manage().timeouts().implicitlyWait(androidSettings.getTimeoutSettings().getImplicitWaitTimeout(), TimeUnit.SECONDS);
+        driver.manage().timeouts().implicitlyWait(Duration.ofSeconds(androidSettings.getTimeoutSettings().getImplicitWaitTimeout()));
         WRAPPED_ANDROID_DRIVER.set(driver);
         solutions.bellatrix.web.infrastructure.DriverService.setWrappedDriver(driver);
         return driver;

bellatrix.core/pom.xml

@@ -24,7 +24,7 @@
     <version>1.0</version>
 
     <properties>
-        <selenium.version>4.8.1</selenium.version>
+        <selenium.version>4.11.0</selenium.version>
         <appium.version>8.3.0</appium.version>
         <testng.version>7.7.0</testng.version>
         <junit.version>5.9.2</junit.version>
@@ -62,19 +62,16 @@
             <groupId>software.amazon.awssdk</groupId>
             <artifactId>textract</artifactId>
         </dependency>
-
         <dependency>
             <groupId>com.opencsv</groupId>
             <artifactId>opencsv</artifactId>
             <version>5.7.1</version>
         </dependency>
-
         <dependency>
             <groupId>com.mailslurp</groupId>
             <artifactId>mailslurp-client-java</artifactId>
             <version>15.17.1</version>
         </dependency>
-
         <dependency>
             <groupId>commons-io</groupId>
             <artifactId>commons-io</artifactId>
@@ -83,7 +80,7 @@
         <dependency>
             <groupId>com.google.guava</groupId>
             <artifactId>guava</artifactId>
-            <version>31.1-jre</version>
+            <version>33.0.0-jre</version>
         </dependency>
         <dependency>
             <groupId>com.google.code.gson</groupId>
@@ -156,15 +153,5 @@
             <version>${rest.assured.version}</version>
             <scope>compile</scope>
         </dependency>
-        <dependency>
-            <groupId>com.azure</groupId>
-            <artifactId>azure-security-keyvault-secrets</artifactId>
-            <version>4.2.3</version>
-        </dependency>
-        <dependency>
-            <groupId>com.azure</groupId>
-            <artifactId>azure-identity</artifactId>
-            <version>1.2.0</version>
-        </dependency>
     </dependencies>
 </project>

bellatrix.core/src/main/java/solutions/bellatrix/core/assertions/EntitiesAsserter.java

@@ -61,25 +61,32 @@ private static <TEntity> List<Exception> assertAreEqualsInternal(TEntity expecte
                 try {
                     currentExpectedProperty = expectedObject.getClass().getMethod(currentRealProperty.getName());
                 } catch (NoSuchMethodException e) {
-                    failedAssertions.add(e);
+                    System.out.println(String.format("Property %s not found.", currentRealProperty));
+//                    failedAssertions.add(e);
                 }
 
                 var exceptionMessage = "The property " + currentRealProperty.getName() + " of class " + realObject.getClass().getSimpleName() + " was not as expected.";
 
                 try {
                     if (currentRealProperty.getReturnType() == LocalDateTime.class) {
                         LocalDateTimeAssert.areEqual(
-                                (LocalDateTime) currentExpectedProperty.invoke(expectedObject),
-                                (LocalDateTime) currentRealProperty.invoke(realObject),
+                                (LocalDateTime)currentExpectedProperty.invoke(expectedObject),
+                                (LocalDateTime)currentRealProperty.invoke(realObject),
                                 deltaType, deltaQuantity, exceptionMessage);
                     } else {
                         Assertions.assertEquals(
                                 currentExpectedProperty.invoke(expectedObject),
                                 currentRealProperty.invoke(realObject),
                                 exceptionMessage);
                     }
-
-                } catch (Exception ex) {
+                }
+                catch (NoSuchMethodException nsm){
+                    //ignore this case
+                }
+                catch (NullPointerException nsmex){
+                    //ignore this case
+                }
+                catch (Exception ex) {
                     failedAssertions.add(ex);
                 }
             }

bellatrix.core/src/main/java/solutions/bellatrix/core/configuration/ConfigurationService.java

@@ -30,6 +30,10 @@
 public final class ConfigurationService {
     private static String environment;
 
+    public static String getEnvironment() {
+        return environment;
+    }
+
     public static <T> T get(Class<T> configSection) {
         T mappedObject = (T)new Object();
         if (environment == null) {

bellatrix.core/src/main/java/solutions/bellatrix/core/integrations/azure/KeyVault.java

@@ -10,17 +10,10 @@ public static String getSecret(Supplier<String> getConfigValue) {
             return System.getenv(configValue.replace("env_", ""));
         }
 
-        if (configValue.startsWith("vault_")) {
-            return KeyVault.getSecret(configValue.replace("vault_", ""));
-        }
-
         return configValue;
     }
 
     public static String getSecret(String name) {
-        if (KeyVault.isAvailable) {
-            return KeyVault.getSecret(name);
-        }
 
         String environmentalVariable = System.getenv(name);
 

bellatrix.core/src/main/java/solutions/bellatrix/core/integrations/azure/SecretsResolver.java

@@ -14,12 +14,10 @@
 package solutions.bellatrix.core.plugins;
 
 import java.lang.reflect.Method;
-import java.util.HashSet;
-import java.util.LinkedHashSet;
-import java.util.Set;
+import java.util.*;
 
 public final class PluginExecutionEngine {
-    private final static Set<Plugin> PLUGINS;
+    private final static LinkedHashSet<Plugin> PLUGINS;
 
     static {
         PLUGINS = new LinkedHashSet<>();

bellatrix.core/src/main/java/solutions/bellatrix/core/plugins/PluginExecutionEngine.java

@@ -13,10 +13,7 @@
 
 package solutions.bellatrix.core.plugins.junit;
 
-import org.junit.jupiter.api.AfterAll;
-import org.junit.jupiter.api.AfterEach;
-import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.TestInfo;
+import org.junit.jupiter.api.*;
 import org.junit.jupiter.api.extension.ExtendWith;
 import solutions.bellatrix.core.plugins.PluginExecutionEngine;
 import solutions.bellatrix.core.plugins.TestResult;
@@ -28,6 +25,7 @@
 import java.util.List;
 
 @ExtendWith(TestResultWatcher.class)
+@TestInstance(TestInstance.Lifecycle.PER_CLASS)
 public class BaseTest extends UsesPlugins {
     static final ThreadLocal<TestResult> CURRENT_TEST_RESULT = new ThreadLocal<>();
     private static final ThreadLocal<Boolean> CONFIGURATION_EXECUTED = new ThreadLocal<>();
@@ -92,7 +90,7 @@ public void afterMethodCore(TestInfo testInfo) {
         try {
             var testClass = this.getClass();
             assert testInfo.getTestMethod().isPresent();
-            var methodInfo = testClass.getMethod(testInfo.getTestMethod().get().getName());
+            var methodInfo = testClass.getMethod(testInfo.getTestMethod().get().getName(), testInfo.getTestMethod().get().getParameterTypes());
             PluginExecutionEngine.preAfterTest(CURRENT_TEST_RESULT.get(), methodInfo);
             afterEach();
            // PluginExecutionEngine.postAfterTest(CURRENT_TEST_RESULT.get(), methodInfo);
@@ -103,11 +101,12 @@ public void afterMethodCore(TestInfo testInfo) {
     }
 
     @AfterAll
-    public static void afterClassCore(TestInfo testInfo) {
+    public void afterClassCore(TestInfo testInfo) {
         try {
             var testClass = testInfo.getTestClass();
             if (testClass.isPresent()) {
                 PluginExecutionEngine.preAfterClass(testClass.get());
+                afterClass();
                 PluginExecutionEngine.postAfterClass(testClass.get());
             }
         } catch (Exception e) {
@@ -134,4 +133,7 @@ protected void onBeforeEachFailure() {
 
     protected void afterEach() {
     }
+
+    protected void afterClass() {
+    }
 }

bellatrix.core/src/main/java/solutions/bellatrix/core/plugins/junit/BaseTest.java

@@ -0,0 +1,26 @@
+package solutions.bellatrix.core.utilities;
+
+import java.awt.*;
+import java.awt.datatransfer.*;
+import java.io.IOException;
+
+public class ClipboardManager {
+    private static final Clipboard systemClipboard = Toolkit.getDefaultToolkit().getSystemClipboard();
+
+    public static String getLastEntity() {
+        var lastCopiedEntity = "";
+
+        try {
+            lastCopiedEntity = (String)systemClipboard.getData(DataFlavor.stringFlavor);
+        } catch (IOException | UnsupportedFlavorException e) {
+            throw new RuntimeException(e);
+        }
+
+        return lastCopiedEntity;
+    }
+
+    public static void copyTextToClipboard(String text) {
+        Transferable transferable = new StringSelection(text);
+        systemClipboard.setContents(transferable, null);
+    }
+}

bellatrix.core/src/main/java/solutions/bellatrix/core/utilities/ClipboardManager.java

@@ -36,7 +36,7 @@ public static <T> T getInstance(Class<T> classOf, Object... initargs) {
 
             return (T)SINGLETON_FACTORY.mapHolder.get(classOf.getName());
         } catch (Exception e) {
-            // not the best practice to return null. But probably we will never end here so it is OK.
+            Log.error("Failed to create instance of the object. Exception was: " + e);
             return null;
         }
     }