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

Make parameter loading work in CI #13

Open
jkleiber opened this issue Dec 21, 2022 · 0 comments
Open

Make parameter loading work in CI #13

jkleiber opened this issue Dec 21, 2022 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@jkleiber
Copy link
Contributor

Summary
In #8 we added a way to load parameters from a file. It would be good if unit tests could depend on the parameter management system by loading the parameters for specific tests.

As of right now, parameter loading doesn't work in CI because the unit tests create specific folders in the build directory without copying the parameters file into the same folder. The objective of this issue is to fix the build scripts such that the parameters.toml gets copied to the deploy directory when running CI or unit tests locally.

Note: CI runs Linux, so a linux computer may need to be used to avoid pushing to CI repeatedly to debug.

Work To Do

  • Figure out how to copy files in CI with the build scripts
  • Update build.gradle to copy the parameters.toml to the correct directories
  • Add unit tests that use the parameters.toml

Verification

  • CI passes with the new unit tests
@jkleiber jkleiber added the bug Something isn't working label Dec 21, 2022
@jkleiber jkleiber self-assigned this Dec 21, 2022
@danielbrownmsm danielbrownmsm added this to the Enhanced Functionality milestone Dec 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants