Skip to content

Commit ab58e1f

Browse files
committed
fix
1 parent 8963ee5 commit ab58e1f

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

tests/test_phonons_cli.py

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ def test_phonons():
3636
"""Test calculating force constants and band structure."""
3737
results_dir = Path("./janus_results")
3838
phonopy_path = results_dir / "NaCl-phonopy.yml"
39-
bands_path = results_dir / "NaCl-auto_bands.yml.xz"
39+
bands_path = results_dir / "NaCl-auto_bands.yml"
4040
log_path = results_dir / "NaCl-phonons-log.yml"
4141
summary_path = results_dir / "NaCl-phonons-summary.yml"
4242

@@ -68,7 +68,7 @@ def test_phonons():
6868

6969
has_eigenvectors = False
7070
has_velocity = False
71-
with lzma.open(bands_path, mode="rt") as file:
71+
with open(bands_path) as file:
7272
for line in file:
7373
if "eigenvector" in line:
7474
has_eigenvectors = True
@@ -113,7 +113,7 @@ def test_phonons():
113113
def test_bands_simple(tmp_path):
114114
"""Test calculating force constants and reduced bands information."""
115115
file_prefix = tmp_path / "NaCl"
116-
autoband_results = tmp_path / "NaCl-auto_bands.yml.xz"
116+
autoband_results = tmp_path / "NaCl-auto_bands.yml"
117117
summary_path = tmp_path / "NaCl-phonons-summary.yml"
118118

119119
result = runner.invoke(
@@ -157,6 +157,7 @@ def test_hdf5(tmp_path):
157157
file_prefix = tmp_path / "test" / "NaCl"
158158
phonon_results = tmp_path / "test" / "NaCl-phonopy.yml"
159159
hdf5_results = tmp_path / "test" / "NaCl-force_constants.hdf5"
160+
bands_results = tmp_path / "test" / "NaCl-auto_bands.hdf5"
160161
summary_path = tmp_path / "test" / "NaCl-phonons-summary.yml"
161162
log_path = tmp_path / "test" / "NaCl-phonons-log.yml"
162163

@@ -168,6 +169,7 @@ def test_hdf5(tmp_path):
168169
DATA_PATH / "NaCl.cif",
169170
"--arch",
170171
"mace_mp",
172+
"--bands",
171173
"--file-prefix",
172174
file_prefix,
173175
"--hdf5",
@@ -176,6 +178,7 @@ def test_hdf5(tmp_path):
176178
assert result.exit_code == 0
177179
assert phonon_results.exists()
178180
assert hdf5_results.exists()
181+
assert bands_results.exists()
179182

180183
# Read phonons summary file
181184
with open(summary_path, encoding="utf8") as file:
@@ -184,7 +187,7 @@ def test_hdf5(tmp_path):
184187
output_files = {
185188
"params": phonon_results,
186189
"force_constants": hdf5_results,
187-
"bands": None,
190+
"bands": bands_results,
188191
"bands_plot": None,
189192
"dos": None,
190193
"dos_plot": None,
@@ -293,10 +296,10 @@ def test_plot(tmp_path):
293296
"""Test for plotting routines."""
294297
file_prefix = tmp_path / "NaCl"
295298
phonon_results = tmp_path / "NaCl-phonopy.yml"
296-
bands_path = tmp_path / "NaCl-auto_bands.yml.xz"
299+
bands_path = tmp_path / "NaCl-auto_bands.yml"
297300
pdos_results = tmp_path / "NaCl-pdos.dat"
298301
dos_results = tmp_path / "NaCl-dos.dat"
299-
autoband_results = tmp_path / "NaCl-auto_bands.yml.xz"
302+
autoband_results = tmp_path / "NaCl-auto_bands.yml"
300303
summary_path = tmp_path / "NaCl-phonons-summary.yml"
301304
log_path = tmp_path / "NaCl-phonons-log.yml"
302305
svgs = [
@@ -634,7 +637,7 @@ def test_paths(tmp_path):
634637
"""Test displacement_kwargs can be set."""
635638
file_prefix = tmp_path / "NaCl"
636639
qpoint_file = DATA_PATH / "paths.yml"
637-
band_results = tmp_path / "NaCl-bands.yml.xz"
640+
band_results = tmp_path / "NaCl-bands.yml"
638641

639642
result = runner.invoke(
640643
app,

0 commit comments

Comments
 (0)