Skip to content

Commit

Permalink
Client version update
Browse files Browse the repository at this point in the history
  • Loading branch information
HardNorth committed Jan 23, 2024
1 parent 5fb641a commit 838ff48
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 7 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
# Changelog

## [Unreleased]
### Changed
- Client version updated on [5.2.1](https://github.com/reportportal/client-java/releases/tag/5.2.1), by @HardNorth

## [5.1.3]
### Changed
Expand Down
7 changes: 4 additions & 3 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ repositories {
}

dependencies {
api 'com.epam.reportportal:client-java:5.1.22'
api 'com.epam.reportportal:client-java:5.2.1'
api 'com.epam.reportportal:commons-model:5.0.0'
api 'com.nordstrom.tools:junit-foundation:17.0.3'
api 'com.google.code.findbugs:jsr305:3.0.2'
Expand All @@ -40,8 +40,8 @@ dependencies {
testImplementation 'org.hamcrest:hamcrest-core:2.2'
testImplementation "org.mockito:mockito-core:${project.mockito_version}"
testImplementation "org.mockito:mockito-junit-jupiter:${project.mockito_version}"
testImplementation 'ch.qos.logback:logback-classic:1.3.8'
testImplementation 'com.epam.reportportal:logger-java-logback:5.1.6'
testImplementation 'ch.qos.logback:logback-classic:1.3.12'
testImplementation 'com.epam.reportportal:logger-java-logback:5.2.0'
testImplementation ("org.junit.platform:junit-platform-runner:${project.junit5_runner_version}") {
exclude module: 'junit'
}
Expand All @@ -52,6 +52,7 @@ dependencies {
testImplementation 'pl.pragmatists:JUnitParams:1.1.1'
testImplementation "org.powermock:powermock-module-junit4:${project.powermock_version}"
testImplementation "org.powermock:powermock-api-mockito2:${project.powermock_version}"
testImplementation 'com.squareup.okhttp3:okhttp:4.12.0'
}

test {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -958,17 +958,20 @@ protected List<ParameterResource> getMethodParameters(@Nonnull final FrameworkMe
try {
Optional<Constructor<?>> constructor = Arrays.stream(method.getDeclaringClass().getConstructors()).findFirst();
if (constructor.isPresent()) {
result.addAll(ParameterUtils.getParameters(constructor.get(),
Arrays.asList((Object[]) Accessible.on(runner).field("parameters").getValue())
));
Object[] params = (Object[]) Accessible.on(runner).field("parameters").getValue();
if(params != null) {
result.addAll(ParameterUtils.getParameters(constructor.get(), Arrays.asList(params)));
}
}
} catch (NoSuchFieldException e) {
LOGGER.warn("Unable to get parameters for parameterized runner", e);
}
} else if (callable != null) {
try {
Object[] params = (Object[]) Accessible.on(callable).field("val$params").getValue();
result.addAll(ParameterUtils.getParameters(method.getMethod(), Arrays.asList(params)));
if(params != null) {
result.addAll(ParameterUtils.getParameters(method.getMethod(), Arrays.asList(params)));
}
} catch (NoSuchFieldException e) {
LOGGER.warn("Unable to get parameters for parameterized runner", e);
}
Expand Down

0 comments on commit 838ff48

Please sign in to comment.