Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Gradle test dependencies loading #174

Merged
merged 6 commits into from
Nov 7, 2023
Merged

Conversation

rob93c
Copy link
Member

@rob93c rob93c commented Nov 6, 2023

This pull request aims to fix the following warning Gradle shows when launching unit tests:

The automatic loading of test framework implementation dependencies has been deprecated. This is scheduled to be removed in Gradle 9.0. Declare the desired test framework directly on the test suite or explicitly declare the test framework implementation dependencies on the test's runtime classpath. Consult the upgrading guide for further information: https://docs.gradle.org/8.4/userguide/upgrading_version_8.html#test_framework_implementation_dependencies

In addition to this, the code has been cleaned up in some classes.

@rob93c rob93c added enhancement New feature or request dependencies Pull requests that update a dependency file labels Nov 6, 2023
@rob93c rob93c requested a review from MartelliEnrico November 6, 2023 12:58
@rob93c rob93c self-assigned this Nov 6, 2023
Copy link
Contributor

@MartelliEnrico MartelliEnrico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MartelliEnrico MartelliEnrico merged commit d4939ae into main Nov 7, 2023
7 checks passed
@MartelliEnrico MartelliEnrico deleted the fix-gradle-test-warning branch November 7, 2023 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants