Skip to content

Commit

Permalink
mark slabs test xfail per pmg bug
Browse files Browse the repository at this point in the history
  • Loading branch information
lbluque committed Apr 26, 2024
1 parent 702c236 commit 228f4a4
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions tests/test_bulk.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,9 @@ def test_unique_slab_enumeration(self):
assert slab not in seen
seen.append(slab)

# pymatgen-2023.5.10 + ase 3.22.1
# pymatgen bug see https://github.com/materialsproject/pymatgen/issues/3747
if len(slabs) == 15:
pytest.xfail(f"Number of generated slabs {len(slabs)} is off due to pymatgen bug!")
assert len(slabs) == 14

with open(self.precomputed_path, "wb") as f:
Expand All @@ -86,7 +88,10 @@ def test_precomputed_slab(self):
precomputed_slabs = self.bulk.get_slabs(
precomputed_slabs_dir=precomputed_slabs_dir
)
# pymatgen-2023.5.10 + ase 3.22.1

if len(precomputed_slabs) == 15:
pytest.xfail(f"Number of generated slabs {len(precomputed_slabs)} is off due to pymatgen bug!")

assert len(precomputed_slabs) == 14

slabs = self.bulk.get_slabs()
Expand Down

0 comments on commit 228f4a4

Please sign in to comment.