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.

Sign up for GitHub

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

Jump to bottom

Tests refactor #25

Merged

cote3804 merged 4 commits into main from surface_flows

Dec 5, 2024

Owner

cote3804 commented Dec 5, 2024

Summary

Refactored tests and made minor sets/jobs fixes.

Removed mock_cwd fixture as it was creating temporary files in the code repo when tests would run.
Refactored file paths across tests to be consistent with changed os.getcwd().
Added values to input files to be compliant with automatically calculated tags from InputSetGenerator.
commented out **jdftx_calculation_kwargs and added TODO statement.
Removed line that was forcing coords-type Cartesian regardless of user inputs.

cote3804 added 4 commits

December 5, 2024 12:31


          TODO add jdftx_calculation kwargs to Calculation schema

dd6ee03


          removed forced Cartesian coords-type

acbd1c9


          small typo fix

735a02f


          reafactored tests

8843a49

cote3804 merged commit af343cb into main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet