Skip to content

Commit

Permalink
remove redundant assertion from test code
Browse files Browse the repository at this point in the history
As we use the equivalent pytest-based assertion, the old assertion is redundant
  • Loading branch information
schroeding committed Dec 10, 2024
1 parent bf98342 commit 65c25be
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions benchexec/test_core_assignment_new.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@ def class_decorator(c) -> callable:
for number_cores, number_threads in core_and_thread_tupels:

def decorator_test_invalid(self):
self.assertInvalid(number_cores, number_threads)
with pytest.raises(SystemExit):
get_cpu_distribution(
number_cores,
Expand Down Expand Up @@ -95,16 +94,6 @@ def assertValid(self, coreLimit, num_of_threads, expectedResult=None):
f"Incorrect result for {coreLimit} cores and {num_of_threads} threads.",
)

def assertInvalid(self, coreLimit, num_of_threads):
self.assertRaises(
SystemExit,
get_cpu_distribution,
coreLimit,
num_of_threads,
self.use_hyperthreading,
*self.machine(),
)

machine_definition = None

def test_assert_refactor_translation_identical(self):
Expand Down

0 comments on commit 65c25be

Please sign in to comment.