Skip to content

Commit

Permalink
Responding to comments
Browse files Browse the repository at this point in the history
  • Loading branch information
john-science committed Dec 4, 2023
1 parent 68b144c commit 95d5b26
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions armi/reactor/tests/test_parameters.py
Original file line number Diff line number Diff line change
Expand Up @@ -542,8 +542,6 @@ def setUp(self):
a.add(makeComp("block{}-{}".format(ai, bi)))

self.comps = [self.r.core] + self.r.core.getChildren(deep=True)
for pd in MockSyncPC().paramDefs:
pd.assigned = parameters.NEVER

@unittest.skipIf(context.MPI_SIZE <= 1 or MPI_EXE is None, "Parallel test only")
def test_noConflicts(self):
Expand All @@ -553,7 +551,7 @@ def test_noConflicts(self):
:id: T_ARMI_CMP_MPI0
:tests: R_ARMI_CMP_MPI
"""
syncCount = self.r.syncMpiState()
_syncCount = self.r.syncMpiState()

for ci, comp in enumerate(self.comps):
if ci % context.MPI_SIZE == context.MPI_RANK:
Expand All @@ -569,7 +567,7 @@ def test_noConflicts(self):

@unittest.skipIf(context.MPI_SIZE <= 1 or MPI_EXE is None, "Parallel test only")
def test_withConflicts(self):
"""Test conflicts arrise correctly if we force a conflict.
"""Test conflicts arise correctly if we force a conflict.
.. test:: Raise errors when there are conflicts across processes.
:id: T_ARMI_CMP_MPI1
Expand Down

0 comments on commit 95d5b26

Please sign in to comment.