Skip to content

Commit

Permalink
Explicitely use set() instead of {}
Browse files Browse the repository at this point in the history
Some Python version may interpret the latter as an (empty) dict instead
of a set
  • Loading branch information
Flamefire committed Oct 25, 2023
1 parent 510cb33 commit 8c9b09a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/framework/easyconfig.py
Original file line number Diff line number Diff line change
Expand Up @@ -387,7 +387,7 @@ def test_false_dep_version(self):
eb = EasyConfig(self.eb_file)
deps = eb.dependencies()
self.assertEqual(deps, [])
self.assertEqual(eb.dependency_names(), {})
self.assertEqual(eb.dependency_names(), set())

def test_extra_options(self):
""" extra_options should allow other variables to be stored """
Expand Down Expand Up @@ -1655,7 +1655,7 @@ def test_filter_deps(self):
init_config(build_options=build_options)
ec = EasyConfig(ec_file, validate=False)
self.assertEqual(ec.dependencies(), [])
self.assertEqual(ec.dependency_names(), {})
self.assertEqual(ec.dependency_names(), set())

def test_replaced_easyconfig_parameters(self):
"""Test handling of replaced easyconfig parameters."""
Expand Down

0 comments on commit 8c9b09a

Please sign in to comment.