gvdutra opened a new pull request, #8246:
URL: https://github.com/apache/hop/pull/8246
Fixes #8187.
Project variables were applied after environment configuration files were
loaded, so paths such as ${CONFIG_HOME}/environment.json could not be resolved.
Apply project variables before resolving those paths, then reapply them
afterward to preserve the existing variable precedence.
Added a regression test using CONFIG_HOME=${PROJECT_HOME}/config and a real
environment configuration file. Before the fix, the expected environment value
was null. With the fix, the file is loaded and all 13 ProjectTest tests pass.
Validation:
- ./mvnw -pl plugins/misc/projects -am -Dtest=ProjectTest
-Dsurefire.failIfNoSpecifiedTests=false test
- Spotless formatting and validation
- git diff --check
- [x] I hereby declare this contribution to be licensed under the [Apache
License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
