Skip to content

pre-commit: fix flake8 errors #10

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 1 commit into from
Oct 7, 2025
Merged

pre-commit: fix flake8 errors #10

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
8 changes: 5 additions & 3 deletions tests/test_distanceprinter.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,15 @@ def test_distanceprinter(monkeypatch):
example_file_dir = os.path.join(current_module_dir, "data")
# must be cif, stru, or xyz file
strufile = os.path.join(example_file_dir, "Ni-9008476.cif")
# atoms typically pull from list of elements in stru output of PDFFitStructure. 'all' is always first option
# atoms typically pull from list of elements in output of PDFFitStructure.
# 'all' is always first option
atomi = "all"
atomj = "all"
# lb and ub are lower an upper bound of distance to list in angstroms
lb = 1
ub = 10
# 1 means keep duplicate atom pairs with same inter-atomic distance, 0 means not to
# 1 means keep duplicate atom pairs with same inter-atomic distance
# 0 means not to
comp = "1"
monkeypatch.setattr(
"sys.argv",
Expand Down Expand Up @@ -44,7 +46,7 @@ def test_distanceprinter(monkeypatch):
rv1 = "".join(rv1)
teststr = rv1

# Remove structure file path when comparing as it may differ between machines
# Remove stru file path when comparing as it may differ between machines
resultstr = resultstr.split("\n", 2)[2]
teststr = teststr.split("\n", 2)[2]
assert resultstr == teststr