This PR removes the dependency of the installed tkltest command on relative paths to library jars and other resources. With this update, the CLI command can be run from anywhere on the file system. The PR also fixes the related docker issue that required library jars to be available on the host machine for CLI commands to run in the container.
The PR contains the following updates:
Install resource dependencies (jar, jinja, gif files) via manifest file
Create packages for directories that contain resource files
Reference resource files using package names via importlib.resources module
Jinja template loading using package loader instead of file-system loader
Please check the types of changes your PR introduces.
[x] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Refactoring (non-breaking code restructuring that preserves functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Build-related update (CI workflow, test cases)
[x] Documentation update
[ ] Other (please describe):
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so that it can be replicated.
Please also list any relevant details for your test configuration.
[ ] Test A
[ ] Test B
Checklist
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
Description
This PR removes the dependency of the installed
tkltestcommand on relative paths to library jars and other resources. With this update, the CLI command can be run from anywhere on the file system. The PR also fixes the related docker issue that required library jars to be available on the host machine for CLI commands to run in the container.The PR contains the following updates:
importlib.resourcesmodulelibtotkltest-lib, removelib/downloadfolder)Related to #117
Type of Change
Please check the types of changes your PR introduces.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so that it can be replicated. Please also list any relevant details for your test configuration.
Checklist