Skip to content

Commit ab18315

Browse files
davidfarinajrmazeau
authored andcommitted
commented out check_surface_thermo_groups_have_surface_attributes db test
The surface thermo groups should no longer have metal attrs since the adsorption groups are used for any type of metal/facet and this can mess with descending the trees if the thermo group entry has a metal, and the structure does not. Instead, the metal and facet are saved as thermo group database attrs
1 parent acf1fc2 commit ab18315

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

testing/databaseTest.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -242,12 +242,12 @@ def test_thermo(self):
242242
yield test, group_name
243243

244244
# tests for adsorption groups
245-
if 'adsorption' in group_name.lower():
246-
test = lambda x: self.check_surface_thermo_groups_have_surface_attributes(group_name, group)
247-
test_name = "Thermo surface groups {0}: Entry has metal attributes?".format(group_name)
248-
test.description = test_name
249-
self.compat_func_name = test_name
250-
yield test, group_name
245+
# if 'adsorption' in group_name.lower():
246+
# test = lambda x: self.check_surface_thermo_groups_have_surface_attributes(group_name, group)
247+
# test_name = "Thermo surface groups {0}: Entry has metal attributes?".format(group_name)
248+
# test.description = test_name
249+
# self.compat_func_name = test_name
250+
# yield test, group_name
251251

252252
for library_name, library in self.database.thermo.libraries.items():
253253
if 'surface' in library_name.lower():

0 commit comments

Comments
 (0)