Fix build

reportportal · Jan 21, 2025 · ef046f7 · ef046f7
1 parent 66a0569
commit ef046f7
Show file tree

Hide file tree

Showing 11 changed files with 20 additions and 18 deletions.
diff --git a/src/main/java/com/epam/reportportal/junit/ReportPortalListener.java b/src/main/java/com/epam/reportportal/junit/ReportPortalListener.java
@@ -953,7 +953,7 @@ protected List<ParameterResource> getMethodParameters(@Nonnull final FrameworkMe
 			Object target = getTargetFor(runner, method);
 			if (target instanceof ArtifactParams) {
 				//noinspection Guava
-				com.google.common.base.Optional<Map<String, Object>> params = ((ArtifactParams) target).getParameters();
+				Optional<Map<String, Object>> params = ((ArtifactParams) target).getParameters();
 				if (params.isPresent()) {
 					for (Map.Entry<String, Object> param : params.get().entrySet()) {
 						ParameterResource parameter = new ParameterResource();

diff --git a/.../java/com/epam/reportportal/junit/features/callback/RetrieveByDescriptionFeatureTest.java b/.../java/com/epam/reportportal/junit/features/callback/RetrieveByDescriptionFeatureTest.java
@@ -23,7 +23,6 @@
 import com.epam.reportportal.service.tree.TestItemTree;
 import com.epam.ta.reportportal.ws.model.FinishTestItemRQ;
 import com.epam.ta.reportportal.ws.model.attribute.ItemAttributesRQ;
-import com.google.common.collect.Sets;
 import org.junit.AfterClass;
 import org.junit.Rule;
 import org.junit.Test;
@@ -36,6 +35,7 @@
 import javax.annotation.Nonnull;
 import java.util.ArrayList;
 import java.util.Calendar;
+import java.util.Collections;
 import java.util.List;
 
 import static java.util.Optional.ofNullable;
@@ -51,7 +51,8 @@ public class RetrieveByDescriptionFeatureTest {
 	public TestRule rule = new TestWatcher() {
 		@Override
 		protected void finished(@Nonnull Description description) {
-			ofNullable(ItemTreeUtils.retrieveLeaf(description,
+			ofNullable(ItemTreeUtils.retrieveLeaf(
+					description,
 					ParallelRunningContext.getCurrent().getItemTree()
 			)).ifPresent(testItemLeaves::add);
 		}
@@ -71,8 +72,9 @@ private static void addSaucelabsAttribute() {
 		FinishTestItemRQ request = new FinishTestItemRQ();
 		request.setEndTime(Calendar.getInstance().getTime());
 		request.setStatus("PASSED");
-		request.setAttributes(Sets.newHashSet(new ItemAttributesRQ(SLID, SLID_VALUE)));
-		ItemTreeReporter.finishItem(ReportPortalListener.getReportPortal().getClient(),
+		request.setAttributes(Collections.singleton(new ItemAttributesRQ(SLID, SLID_VALUE)));
+		ItemTreeReporter.finishItem(
+				ReportPortalListener.getReportPortal().getClient(),
 				request,
 				ParallelRunningContext.getCurrent().getItemTree().getLaunchId(),
 				testItemLeaves.get(0)

diff --git a/src/test/java/com/epam/reportportal/junit/features/ignore/ParameterizedIgnoredTest.java b/src/test/java/com/epam/reportportal/junit/features/ignore/ParameterizedIgnoredTest.java
@@ -17,7 +17,6 @@
 package com.epam.reportportal.junit.features.ignore;
 
 import com.epam.reportportal.annotations.ParameterKey;
-import com.google.common.base.Optional;
 import com.nordstrom.automation.junit.ArtifactParams;
 import com.nordstrom.automation.junit.AtomIdentity;
 import org.junit.Ignore;
@@ -30,6 +29,7 @@
 
 import java.util.Collections;
 import java.util.Map;
+import java.util.Optional;
 
 @RunWith(Parameterized.class)
 public class ParameterizedIgnoredTest implements ArtifactParams {

diff --git a/src/test/java/com/epam/reportportal/junit/features/parameters/JUnitParamsNullValueTest.java b/src/test/java/com/epam/reportportal/junit/features/parameters/JUnitParamsNullValueTest.java
@@ -16,7 +16,6 @@
 
 package com.epam.reportportal.junit.features.parameters;
 
-import com.google.common.base.Optional;
 import com.nordstrom.automation.junit.ArtifactParams;
 import com.nordstrom.automation.junit.AtomIdentity;
 import com.nordstrom.automation.junit.AtomicTest;
@@ -30,6 +29,7 @@
 import org.junit.runner.RunWith;
 
 import java.util.Map;
+import java.util.Optional;
 
 @RunWith(JUnitParamsRunner.class)
 public class JUnitParamsNullValueTest implements ArtifactParams {
@@ -66,7 +66,7 @@ public Optional<Map<String, Object>> getParameters() {
 			String param = (String) params[0];
 			return Param.mapOf(Param.param("param", param));
 		} catch (IllegalAccessException | NoSuchFieldException e) {
-			return Optional.absent();
+			return Optional.empty();
 		}
 	}
 }
diff --git a/src/test/java/com/epam/reportportal/junit/features/parameters/JUnitParamsSimpleTest.java b/src/test/java/com/epam/reportportal/junit/features/parameters/JUnitParamsSimpleTest.java
@@ -16,7 +16,6 @@
 
 package com.epam.reportportal.junit.features.parameters;
 
-import com.google.common.base.Optional;
 import com.nordstrom.automation.junit.ArtifactParams;
 import com.nordstrom.automation.junit.AtomIdentity;
 import com.nordstrom.automation.junit.AtomicTest;
@@ -30,6 +29,7 @@
 import org.junit.runner.RunWith;
 
 import java.util.Map;
+import java.util.Optional;
 
 @RunWith(JUnitParamsRunner.class)
 public class JUnitParamsSimpleTest implements ArtifactParams {
@@ -62,7 +62,7 @@ public Optional<Map<String, Object>> getParameters() {
 			String param = (String) params[0];
 			return Param.mapOf(Param.param("param", param));
 		} catch (IllegalAccessException | NoSuchFieldException e) {
-			return Optional.absent();
+			return Optional.empty();
 		}
 	}
 }
diff --git a/src/test/java/com/epam/reportportal/junit/features/parameters/JUnitParamsTwoParamsTest.java b/src/test/java/com/epam/reportportal/junit/features/parameters/JUnitParamsTwoParamsTest.java
@@ -16,7 +16,6 @@
 
 package com.epam.reportportal.junit.features.parameters;
 
-import com.google.common.base.Optional;
 import com.nordstrom.automation.junit.ArtifactParams;
 import com.nordstrom.automation.junit.AtomIdentity;
 import com.nordstrom.automation.junit.AtomicTest;
@@ -30,6 +29,7 @@
 import org.junit.runner.RunWith;
 
 import java.util.Map;
+import java.util.Optional;
 
 @RunWith(JUnitParamsRunner.class)
 public class JUnitParamsTwoParamsTest implements ArtifactParams {
@@ -61,7 +61,7 @@ public Optional<Map<String, Object>> getParameters() {
 			Object[] params = LifecycleHooks.getFieldValue(callable, "val$params");
 			return Param.mapOf(Param.param("param1", params[0]), Param.param("param2", params[1]));
 		} catch (IllegalAccessException | NoSuchFieldException e) {
-			return Optional.absent();
+			return Optional.empty();
 		}
 	}
 }
diff --git a/...java/com/epam/reportportal/junit/features/parameters/StandardParametersNullValueTest.java b/...java/com/epam/reportportal/junit/features/parameters/StandardParametersNullValueTest.java
@@ -16,7 +16,6 @@
 
 package com.epam.reportportal.junit.features.parameters;
 
-import com.google.common.base.Optional;
 import com.nordstrom.automation.junit.ArtifactParams;
 import com.nordstrom.automation.junit.AtomIdentity;
 import org.junit.Rule;
@@ -28,6 +27,7 @@
 
 import java.util.Collections;
 import java.util.Map;
+import java.util.Optional;
 
 @RunWith(Parameterized.class)
 public class StandardParametersNullValueTest implements ArtifactParams {

diff --git a/...st/java/com/epam/reportportal/junit/features/parameters/StandardParametersSimpleTest.java b/...st/java/com/epam/reportportal/junit/features/parameters/StandardParametersSimpleTest.java
@@ -16,7 +16,6 @@
 
 package com.epam.reportportal.junit.features.parameters;
 
-import com.google.common.base.Optional;
 import com.nordstrom.automation.junit.ArtifactParams;
 import com.nordstrom.automation.junit.AtomIdentity;
 import org.junit.Rule;
@@ -28,6 +27,7 @@
 
 import java.util.Collections;
 import java.util.Map;
+import java.util.Optional;
 
 @RunWith(Parameterized.class)
 public class StandardParametersSimpleTest implements ArtifactParams {

diff --git a/...java/com/epam/reportportal/junit/features/parameters/StandardParametersTwoParamsTest.java b/...java/com/epam/reportportal/junit/features/parameters/StandardParametersTwoParamsTest.java
@@ -16,7 +16,6 @@
 
 package com.epam.reportportal.junit.features.parameters;
 
-import com.google.common.base.Optional;
 import com.nordstrom.automation.junit.ArtifactParams;
 import com.nordstrom.automation.junit.AtomIdentity;
 import org.junit.Rule;
@@ -28,6 +27,7 @@
 
 import java.util.HashMap;
 import java.util.Map;
+import java.util.Optional;
 
 @RunWith(Parameterized.class)
 public class StandardParametersTwoParamsTest implements ArtifactParams {

diff --git a/src/test/java/com/epam/reportportal/junit/features/retry/StandardParametersRetryTest.java b/src/test/java/com/epam/reportportal/junit/features/retry/StandardParametersRetryTest.java
@@ -16,7 +16,6 @@
 
 package com.epam.reportportal.junit.features.retry;
 
-import com.google.common.base.Optional;
 import com.nordstrom.automation.junit.ArtifactParams;
 import com.nordstrom.automation.junit.AtomIdentity;
 import org.junit.Rule;
@@ -28,6 +27,7 @@
 
 import java.util.Collections;
 import java.util.Map;
+import java.util.Optional;
 import java.util.concurrent.atomic.AtomicInteger;
 
 @RunWith(Parameterized.class)

diff --git a/src/test/java/com/epam/reportportal/junit/utils/SystemAttributesFetcherTest.java b/src/test/java/com/epam/reportportal/junit/utils/SystemAttributesFetcherTest.java
@@ -17,11 +17,11 @@
 package com.epam.reportportal.junit.utils;
 
 import com.epam.ta.reportportal.ws.model.attribute.ItemAttributesRQ;
-import com.google.common.collect.Lists;
 import org.apache.commons.collections.CollectionUtils;
 import org.junit.Assert;
 import org.junit.Test;
 
+import java.util.Arrays;
 import java.util.List;
 import java.util.Objects;
 import java.util.Set;
@@ -34,7 +34,7 @@
  */
 public class SystemAttributesFetcherTest {
 
-	private static List<String> expectedKeys = Lists.newArrayList("jvm", "os", "agent", "skippedIssue");
+	private static final List<String> expectedKeys = Arrays.asList("jvm", "os", "agent", "skippedIssue");
 
 	@Test
 	public void systemAttributesFetchingTest() {