Skip to content

Commit

Permalink
Fixed a problem with running in multiprocessor node - incompatible re…
Browse files Browse the repository at this point in the history
…sults due to using lambdas
  • Loading branch information
ericsson49 committed Jun 11, 2024
1 parent e07e058 commit d88ded5
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 5 deletions.
5 changes: 2 additions & 3 deletions tests/generators/fork_choice_generated/instance_generator.py
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,8 @@ def prepare_fn() -> None:
def make_cases_fn() -> Iterable[TestCase]:
for seed in seeds:
mutation_generator = MutatorsGenerator(
spec, seed, number_of_mutations,
lambda: test_fn(fork_name, preset_name, seed, solution),
debug=debug)
fork_name, preset_name, spec, solution,
seed, number_of_mutations, test_fn, debug=debug)
for j in range(1 + number_of_mutations):
yield TestCase(fork_name=fork_name,
preset_name=preset_name,
Expand Down
9 changes: 7 additions & 2 deletions tests/generators/fork_choice_generated/mutation_operators.py
Original file line number Diff line number Diff line change
Expand Up @@ -240,8 +240,11 @@ def mk_mutations(spec, seed, num, test_fn, debug=False):


class MutatorsGenerator:
def __init__(self, spec, seed, num, test_fn, debug=False):
def __init__(self, fork, preset, spec, solution, seed, num, test_fn, debug=False):
self.fork = fork
self.preset = preset
self.spec = spec
self.solution = solution
self.seed = seed
self.num = num
self.test_fn = test_fn
Expand All @@ -250,6 +253,8 @@ def __init__(self, spec, seed, num, test_fn, debug=False):

def next_test_case(self):
if self.iterator is None:
self.iterator = iter(mk_mutations(self.spec, self.seed, self.num, self.test_fn, self.debug))
def test_inst_fn():
return self.test_fn(self.fork, self.preset, self.seed, self.solution)
self.iterator = iter(mk_mutations(self.spec, self.seed, self.num, test_inst_fn, self.debug))
_, test_case = next(self.iterator)
return test_case

0 comments on commit d88ded5

Please sign in to comment.