add missing test case in IcohplistTest.test_values() (thanks @JaGeo)

fixes flake8 F841 unused variable
materialsproject · Apr 16, 2022 · 576ea88 · 576ea88
1 parent 74f7eee
commit 576ea88
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pymatgen/io/lobster/tests/test_lobster.py b/pymatgen/io/lobster/tests/test_lobster.py
@@ -533,7 +533,6 @@ def test_values(self):
                 "translation": [0, 0, 0],
             },
         }
-        # TODO: why is this unused?
         icooplist_bise = {
             "1": {
                 "length": 2.88231,
@@ -619,6 +618,7 @@ def test_values(self):
 
         self.assertEqual(icohplist_bise, self.icohp_bise.icohplist)
         self.assertEqual(icooplist_fe, self.icoop_fe.icohplist)
+        self.assertEqual(icooplist_bise, self.icoop_bise.icohplist)
         self.assertAlmostEqual(self.icobi.icohplist["1"]["icohp"][Spin.up], 0.58649)
         self.assertAlmostEqual(self.icobi_orbitalwise.icohplist["2"]["icohp"][Spin.up], 0.58649)
         self.assertAlmostEqual(self.icobi_orbitalwise.icohplist["1"]["icohp"][Spin.up], 0.58649)