refactor and improvements

Author: Tahanima

README.md

@@ -148,10 +148,10 @@ The project is structured as follows:
 - ### Test Data
   The project uses *csv* file to store test data and [*univocity-parsers*](https://github.com/uniVocity/univocity-parsers) to retrieve the data and map it to a Java bean.
 
-  To add configurations for new test data, add a new Java bean in the [*data*](./src/main/java/io/github/tahanima/data) package. For example, let's say I want to add test data for a `User` with the attributes `First Name` and `Last Name`. The code for this is as follows:
-     
+  To add configurations for new test data, add a new Java bean in the [*data*](./src/main/java/io/github/tahanima/dto) package. For example, let's say I want to add test data for a `User` with the attributes `First Name` and `Last Name`. The code for this is as follows:
+
    ```java
-   package io.github.tahanima.data;
+   package io.github.tahanima.dto;
 
    import com.univocity.parsers.annotations.Parsed;
 
@@ -176,7 +176,7 @@ The project is structured as follows:
    Test Case ID,First Name,Last Name
    TC-1,Tahanima,Chowdhury
    ```
-   For reference, check [this](./src/main/java/io/github/tahanima/data/LoginData.java), [this](./src/test/resources/testdata/login.csv) and [this](./src/test/java/io/github/tahanima/e2e/LoginE2ETest.java).
+   For reference, check [this](./src/main/java/io/github/tahanima/dto/LoginData.java), [this](./src/test/resources/testdata/login.csv) and [this](./src/test/java/io/github/tahanima/e2e/LoginE2ETest.java).
 
 - ### Page Objects and Page Component Objects
   The project uses [*Page Objects* and *Page Component Objects*](https://www.selenium.dev/documentation/test_practices/encouraged/page_object_models/) to capture the relevant behaviors of a web page. Check the [*ui*](./src/main/java/io/github/tahanima/ui) package for reference.

build.gradle

@@ -10,11 +10,6 @@ repositories {
     mavenCentral()
 }
 
-java {
-    sourceCompatibility = JavaVersion.VERSION_11
-    targetCompatibility = JavaVersion.VERSION_11
-}
-
 dependencies {
     implementation 'org.aeonbits.owner:owner:1.0.12'
     implementation 'com.univocity:univocity-parsers:2.9.1'
@@ -26,6 +21,8 @@ dependencies {
 
     compileOnly 'org.projectlombok:lombok:1.18.30'
     annotationProcessor 'org.projectlombok:lombok:1.18.30'
+    testCompileOnly 'org.projectlombok:lombok:1.18.30'
+    testAnnotationProcessor 'org.projectlombok:lombok:1.18.30'
 
     testImplementation platform('org.junit:junit-bom:5.10.0')
     testImplementation 'org.junit.jupiter:junit-jupiter'

src/main/java/io/github/tahanima/data/BaseData.java renamed to src/main/java/io/github/tahanima/dto/BaseDto.java

@@ -1,4 +1,4 @@
-package io.github.tahanima.data;
+package io.github.tahanima.dto;
 
 import com.univocity.parsers.annotations.Parsed;
 
@@ -10,7 +10,7 @@
  */
 @Getter
 @ToString
-public class BaseData {
+public class BaseDto {
 
     @Parsed(field = "Test Case ID", defaultNullRead = "")
     private String testCaseId;

src/main/java/io/github/tahanima/data/LoginData.java renamed to src/main/java/io/github/tahanima/dto/LoginDto.java

@@ -1,4 +1,4 @@
-package io.github.tahanima.data;
+package io.github.tahanima.dto;
 
 import com.univocity.parsers.annotations.Parsed;
 
@@ -10,7 +10,7 @@
  */
 @Getter
 @ToString(callSuper = true)
-public class LoginData extends BaseData {
+public class LoginDto extends BaseDto {
 
     @Parsed(field = "Username", defaultNullRead = "")
     private String username;

src/main/java/io/github/tahanima/data/ProductsData.java renamed to src/main/java/io/github/tahanima/dto/ProductsDto.java

@@ -1,4 +1,4 @@
-package io.github.tahanima.data;
+package io.github.tahanima.dto;
 
 import com.univocity.parsers.annotations.Parsed;
 
@@ -10,7 +10,7 @@
  */
 @Getter
 @ToString(callSuper = true)
-public final class ProductsData extends BaseData {
+public final class ProductsDto extends BaseDto {
 
     @Parsed(field = "Username", defaultNullRead = "")
     private String username;

src/main/java/io/github/tahanima/factory/BasePageFactory.java

@@ -4,9 +4,12 @@
 
 import io.github.tahanima.ui.page.BasePage;
 
+import lombok.extern.slf4j.Slf4j;
+
 /**
  * @author tahanima
  */
+@Slf4j
 public final class BasePageFactory {
 
     private BasePageFactory() {}
@@ -20,7 +23,7 @@ public static <T extends BasePage> T createInstance(final Page page, final Class
 
             return clazz.cast(instance);
         } catch (Exception e) {
-            e.printStackTrace();
+            log.error("BasePageFactory::createInstance", e);
         }
 
         throw new NullPointerException("Page class instantiation failed.");

src/main/java/io/github/tahanima/factory/BrowserFactory.java

@@ -11,7 +11,6 @@
  * @author tahanima
  */
 public enum BrowserFactory {
-
     CHROMIUM {
         @Override
         public Browser createInstance(final Playwright playwright) {

src/main/java/io/github/tahanima/util/BrowserManager.java

@@ -15,7 +15,6 @@ public final class BrowserManager {
     private BrowserManager() {}
 
     public static Browser getBrowser(final Playwright playwright) {
-        return BrowserFactory.valueOf(config().browser().toUpperCase())
-                .createInstance(playwright);
+        return BrowserFactory.valueOf(config().browser().toUpperCase()).createInstance(playwright);
     }
 }

src/test/java/io/github/tahanima/annotation/DataSource.java

@@ -1,6 +1,6 @@
 package io.github.tahanima.annotation;
 
-import io.github.tahanima.data.BaseData;
+import io.github.tahanima.dto.BaseDto;
 import io.github.tahanima.util.DataArgumentsProvider;
 
 import org.junit.jupiter.params.provider.ArgumentsSource;
@@ -20,5 +20,5 @@
 
     String fileName();
 
-    Class<? extends BaseData> clazz();
+    Class<? extends BaseDto> clazz();
 }

src/test/java/io/github/tahanima/e2e/BaseE2ETest.java renamed to src/test/java/io/github/tahanima/e2e/BaseTest.java

@@ -28,7 +28,7 @@
  * @author tahanima
  */
 @TestInstance(Lifecycle.PER_CLASS)
-public abstract class BaseE2ETest {
+public abstract class BaseTest {
 
     protected Playwright playwright;
     protected Browser browser;