helidon-io / helidon

Java libraries for writing microservices
https://helidon.io
Apache License 2.0
3.44k stars 562 forks source link

4.x: add helidon-logging-jul as a test dependency to some modules #779 #8810

Closed jbescos closed 3 weeks ago

jbescos commented 1 month ago

Description

Fixes #7799

Documentation

N/A

jbescos commented 1 month ago

These are the steps I did to find the places to modify:

$ grep -Rl --include '*.properties' 'io.helidon.logging.jul.HelidonConsoleHandler' | grep 'src/test/resources'
config/encryption/src/test/resources/logging-test.properties
webserver/tests/grpc/src/test/resources/logging.properties
webserver/tests/upgrade/src/test/resources/logging-test.properties
webserver/tests/resource-limits/src/test/resources/logging-test.properties
webserver/testing/junit5/websocket/src/test/resources/logging.properties
fault-tolerance/fault-tolerance/src/test/resources/logging.properties
logging/jul/src/test/resources/logging.properties
webclient/tests/webclient/src/test/resources/logging.properties
security/providers/oidc/src/test/resources/logging.properties
security/providers/oidc-common/src/test/resources/logging.properties
integrations/micrometer/cdi/src/test/resources/logging.properties
tracing/tests/it-tracing-client-zipkin/src/test/resources/logging.properties
microprofile/health/src/test/resources/logging.properties
microprofile/metrics/src/test/resources/logging.properties
microprofile/security/src/test/resources/logging.properties
microprofile/tracing/src/test/resources/logging-test.properties
microprofile/lra/jax-rs/src/test/resources/logging.properties
microprofile/tests/tck/tck-telemetry/src/test/resources/logging.properties
microprofile/tests/tck/tck-config/src/test/resources/logging.properties
tests/functional/request-scope-cdi/src/test/resources/logging.properties
tests/functional/context-propagation/src/test/resources/logging.properties
tests/functional/mp-compression/src/test/resources/logging.properties
tests/functional/request-scope-injection/src/test/resources/logging.properties
tests/functional/jax-rs-multiple-apps/src/test/resources/logging.properties
tests/functional/jax-rs-subresource/src/test/resources/logging.properties
tests/functional/request-scope/src/test/resources/logging.properties
tests/integration/dbclient/app/src/test/resources/logging.properties
tests/integration/vault/src/test/resources/logging.properties
tests/integration/kafka/src/test/resources/logging.properties
tests/integration/jpa/simple/src/test/resources/logging.properties
tests/integration/jpa/appl/src/test/resources/logging.properties
tests/integration/jms/src/test/resources/logging.properties
tests/integration/tls-revocation-config/src/test/resources/logging.properties
examples/microprofile/cors/src/test/resources/logging.properties
examples/microprofile/static-content/src/test/resources/logging.properties
http/tests/encoding/gzip/src/test/resources/logging-test.properties
http/tests/encoding/deflate/src/test/resources/logging-test.properties
http/http2/src/test/resources/logging-test.properties

Make it to point to pom.xml:

config/encryption/pom.xml
webserver/tests/grpc/pom.xml
webserver/tests/upgrade/pom.xml
webserver/tests/resource-limits/pom.xml
webserver/testing/junit5/websocket/pom.xml
fault-tolerance/fault-tolerance/pom.xml
logging/jul/pom.xml
webclient/tests/webclient/pom.xml
security/providers/oidc/pom.xml
security/providers/oidc-common/pom.xml
integrations/micrometer/cdi/pom.xml
tracing/tests/it-tracing-client-zipkin/pom.xml
microprofile/health/pom.xml
microprofile/metrics/pom.xml
microprofile/security/pom.xml
microprofile/tracing/pom.xml
microprofile/lra/jax-rs/pom.xml
microprofile/tests/tck/tck-telemetry/pom.xml
microprofile/tests/tck/tck-config/pom.xml
tests/functional/request-scope-cdi/pom.xml
tests/functional/context-propagation/pom.xml
tests/functional/mp-compression/pom.xml
tests/functional/request-scope-injection/pom.xml
tests/functional/jax-rs-multiple-apps/pom.xml
tests/functional/jax-rs-subresource/pom.xml
tests/functional/request-scope/pom.xml
tests/integration/dbclient/app/pom.xml
tests/integration/vault/pom.xml
tests/integration/kafka/pom.xml
tests/integration/jpa/simple/pom.xml
tests/integration/jpa/appl/pom.xml
tests/integration/jms/pom.xml
tests/integration/tls-revocation-config/pom.xml
examples/microprofile/cors/pom.xml
examples/microprofile/static-content/pom.xml
http/tests/encoding/gzip/pom.xml
http/tests/encoding/deflate/pom.xml
http/http2/pom.xml

Find pom.xml that does not contain the dependency helidon-logging-jul:

while IFS= read -r file; do
  if ! grep -q "helidon-logging-jul" "$file"; then
    echo "$file"
  fi
done < /home/jbescos/Documents/filesWithLogging.txt
webserver/tests/grpc/pom.xml
webserver/tests/resource-limits/pom.xml
webserver/testing/junit5/websocket/pom.xml
webclient/tests/webclient/pom.xml
security/providers/oidc/pom.xml
security/providers/oidc-common/pom.xml
integrations/micrometer/cdi/pom.xml
tracing/tests/it-tracing-client-zipkin/pom.xml
microprofile/health/pom.xml
microprofile/metrics/pom.xml
microprofile/security/pom.xml
microprofile/tracing/pom.xml
microprofile/lra/jax-rs/pom.xml
microprofile/tests/tck/tck-telemetry/pom.xml
microprofile/tests/tck/tck-config/pom.xml
tests/functional/request-scope-cdi/pom.xml
tests/functional/context-propagation/pom.xml
tests/functional/mp-compression/pom.xml
tests/functional/request-scope-injection/pom.xml
tests/functional/jax-rs-multiple-apps/pom.xml
tests/functional/jax-rs-subresource/pom.xml
tests/functional/request-scope/pom.xml
tests/integration/vault/pom.xml
tests/integration/kafka/pom.xml
tests/integration/jpa/simple/pom.xml
tests/integration/jpa/appl/pom.xml
tests/integration/jms/pom.xml
examples/microprofile/cors/pom.xml
examples/microprofile/static-content/pom.xml
http/tests/encoding/gzip/pom.xml
http/tests/encoding/deflate/pom.xml