Merge pull request #53 from iamcharankumar/fb_small_refactor

Used Required Args constructors.

Author: iamcharankumar

src/main/java/io/swaglabs/portal/qa/browsermanager/BrowserFactory.java

@@ -6,10 +6,10 @@ public class BrowserFactory {
 
     public IBrowser createBrowser() {
         return switch (BrowserName.fromConfigValue(WebPortalConstants.BROWSER)) {
+            case CHROME -> new ChromeBrowser();
+            case MS_EDGE -> new MsEdgeBrowser();
             case FIREFOX -> new FirefoxBrowser();
             case WEBKIT -> new WebkitBrowser();
-            case MS_EDGE -> new MsEdgeBrowser();
-            default -> new ChromeBrowser();
         };
     }
 }

src/main/java/io/swaglabs/portal/qa/browsermanager/BrowserName.java

@@ -1,19 +1,19 @@
 package io.swaglabs.portal.qa.browsermanager;
 
 import io.swaglabs.portal.qa.exceptions.WebUtilsException;
-import lombok.AllArgsConstructor;
 import lombok.Getter;
+import lombok.RequiredArgsConstructor;
 
 import java.util.Arrays;
 import java.util.Objects;
 
 @Getter
-@AllArgsConstructor
+@RequiredArgsConstructor
 public enum BrowserName {
 
     CHROME("chrome"),
-    FIREFOX("firefox"),
     MS_EDGE("msedge"),
+    FIREFOX("firefox"),
     WEBKIT("webkit");
 
     private final String browserType;

src/main/java/io/swaglabs/portal/qa/browsermanager/ChromeBrowser.java

@@ -13,7 +13,8 @@ public class ChromeBrowser implements IBrowser {
     @Override
     public BrowserContext createSession(Playwright playwright, boolean isHeadless) {
         BrowserType.LaunchOptions chromeLaunchOptions = WebPortalConstants.BROWSER_LAUNCH_OPTIONS.setHeadless(isHeadless)
-                .setArgs(List.of(WebPortalConstants.WINDOW_POSITION));
+                .setArgs(List.of(WebPortalConstants.WINDOW_POSITION))
+                .setChannel(BrowserName.CHROME.getBrowserType());
         return playwright.chromium().launch(chromeLaunchOptions).newContext(new Browser.NewContextOptions()
                 .setViewportSize(WebPortalConstants.SCREEN_WIDTH, WebPortalConstants.SCREEN_HEIGHT)
                 .setTimezoneId(WebPortalConstants.TIME_ZONE));

src/main/java/io/swaglabs/portal/qa/cdp/CdpCommands.java

@@ -1,10 +1,10 @@
 package io.swaglabs.portal.qa.cdp;
 
-import lombok.AllArgsConstructor;
 import lombok.Getter;
+import lombok.RequiredArgsConstructor;
 
 @Getter
-@AllArgsConstructor
+@RequiredArgsConstructor
 public enum CdpCommands {
 
     NETWORK_ENABLE("Network.enable"),

src/main/java/io/swaglabs/portal/qa/cdp/CdpEvents.java

@@ -1,10 +1,10 @@
 package io.swaglabs.portal.qa.cdp;
 
-import lombok.AllArgsConstructor;
 import lombok.Getter;
+import lombok.RequiredArgsConstructor;
 
 @Getter
-@AllArgsConstructor
+@RequiredArgsConstructor
 public enum CdpEvents {
 
     NETWORK_REQUEST_WILL_BE_SENT("Network.requestWillBeSent"),

src/main/java/io/swaglabs/portal/qa/constants/KeyboardEvents.java

@@ -1,10 +1,10 @@
 package io.swaglabs.portal.qa.constants;
 
-import lombok.AllArgsConstructor;
 import lombok.Getter;
+import lombok.RequiredArgsConstructor;
 
 @Getter
-@AllArgsConstructor
+@RequiredArgsConstructor
 public enum KeyboardEvents {
 
     ENTER("Enter"),