Skip to content

Commit 3f781b0

Browse files
committed
Corrections.
1 parent 1912991 commit 3f781b0

2 files changed

Lines changed: 4 additions & 8 deletions

File tree

exercises/concept/moviegoer/moviegoer_test.rb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,7 @@ def test_members_get_free_popcorn
2323
end
2424

2525
def test_regular_moviegoers_dont_get_free_popcorn
26-
expected_exception = Moviegoer.include?(NotMovieClubMemberError) ? Moviegoer::NotMovieClubMemberError : NotMovieClubMemberError
27-
assert_raises expected_exception do
26+
assert_raises NotMovieClubMemberError do
2827
Moviegoer.new(25, member: false).claim_free_popcorn!
2928
end
3029
end

exercises/concept/simple-calculator/simple_calculator_test.rb

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,17 +27,14 @@ def test_no_number_second_operand_raises_exception
2727
end
2828

2929
def test_raises_exception_for_non_valid_operations
30-
expected_exception = SimpleCalculator.include?(UnsupportedOperation) ? SimpleCalculator::UnsupportedOperation : UnsupportedOperation
31-
assert_raises(expected_exception) { SimpleCalculator.calculate(1, 2, '**') }
30+
assert_raises(SimpleCalculator::UnsupportedOperation) { SimpleCalculator.calculate(1, 2, '**') }
3231
end
3332

3433
def test_raises_exception_when_operation_is_nil
35-
expected_exception = SimpleCalculator.include?(UnsupportedOperation) ? SimpleCalculator::UnsupportedOperation : UnsupportedOperation
36-
assert_raises(expected_exception) { SimpleCalculator.calculate(1, 2, nil) }
34+
assert_raises(SimpleCalculator::UnsupportedOperation) { SimpleCalculator.calculate(1, 2, nil) }
3735
end
3836

3937
def test_raises_exception_when_operation_is_an_empty_string
40-
expected_exception = SimpleCalculator.include?(UnsupportedOperation) ? SimpleCalculator::UnsupportedOperation : UnsupportedOperation
41-
assert_raises(expected_exception) { SimpleCalculator.calculate(1, 2, '') }
38+
assert_raises(SimpleCalculator::UnsupportedOperation) { SimpleCalculator.calculate(1, 2, '') }
4239
end
4340
end

0 commit comments

Comments
 (0)