-
-
Notifications
You must be signed in to change notification settings - Fork 99
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
Identify packages with tests executed without logger implementation and add logger implementation #1606
Comments
Affected modules are:
|
In PR #1631 we added a stopgap measure to fill the logging gap fot the build system. To cleanly solve this issue, all relevant deegree modules have to a have a correct test dependency to a logging implementation like After this is done, the following block can be removed: Lines 150 to 156 in 6a51fcd
And if Lines 157 to 159 in 6a51fcd
|
Additionally, the following modules should also be using a test-reference only for logging (and the log4j*.properties removed).
|
Also, this stack trace occurs:
|
In addition, following modules were identified when testing the draft pull request:
|
Additionally, in the test scope had to be removed in tests modules: lat-lon@6900d0e |
Test with
confirms that logging error does not occur anymore. Draft pull request was converted to final pull request. |
Fix the broken test dependencies for missing logger implementation,
Relates to #1587
The text was updated successfully, but these errors were encountered: