Skip to content

Refactor test_transfor.py: pass variables to pytest parametrize, use conftest.py for common error messages #236

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

Merged
merged 8 commits into from
Dec 16, 2024
Merged

Refactor test_transfor.py: pass variables to pytest parametrize, use conftest.py for common error messages #236

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
5d88cf6
Remove 2 wavelength warnings with test_tth_to_q func
bobleesj Dec 15, 2024
73d8ada
Refactor transform tests, pass variables to parameters
bobleesj Dec 15, 2024
5e39538
Remove the use of extra variable for params to prevent global scope i…
bobleesj Dec 15, 2024
cdf96df
Apply pre-commit
bobleesj Dec 15, 2024
ef75197
improve UC comment for test_q_to_tth func
bobleesj Dec 15, 2024
8ebaede
[pre-commit.ci] auto fixes from pre-commit hooks
pre-commit-ci[bot] Dec 15, 2024
9343e86
Do not call UCs for test function
bobleesj Dec 16, 2024
d962c1b
[pre-commit.ci] auto fixes from pre-commit hooks
pre-commit-ci[bot] Dec 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove the use of extra variable for params to prevent global scope i… 
…n the file
  • Loading branch information
@bobleesj
bobleesj committed Dec 15, 2024
commit 5e395388634b268b0d9f81b8d3dc7e4d2a7cd43f
8 changes: 8 additions & 0 deletions tests/conftest.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,3 +46,11 @@ def wavelength_warning_msg():
"To specify a wavelength, if you have do = DiffractionObject(xarray, yarray, 'tth'), "
"you may set do.wavelength = 1.54 for a wavelength of 1.54 angstroms."
)


@pytest.fixture
def invalid_q_or_d_or_wavelength_error_msg():
return (
"The supplied input array and wavelength will result in an impossible two-theta. "
"Please check these values and re-instantiate the DiffractionObject with correct values."
)
Loading