Skip to content

Commit

Permalink
Fix more warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
byroot committed Oct 1, 2019
1 parent 5dc5c99 commit 0ae09af
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
10 changes: 5 additions & 5 deletions test/data_loss_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -33,17 +33,17 @@ def test_strict_mode_is_disabled
end

def test_decimal_silently_changes_out_of_bound_values
maximum = BigDecimal.new(10 ** (Unicorn.columns_hash['decimal'].precision - Unicorn.columns_hash['decimal'].scale))
delta = BigDecimal.new(10 ** -(Unicorn.columns_hash['decimal'].scale), Unicorn.columns_hash['decimal'].precision)
maximum = BigDecimal(10 ** (Unicorn.columns_hash['decimal'].precision - Unicorn.columns_hash['decimal'].scale))
delta = BigDecimal(10 ** -(Unicorn.columns_hash['decimal'].scale), Unicorn.columns_hash['decimal'].precision)

refute_data_loss Unicorn.new(decimal: maximum - delta)
assert_data_loss Unicorn.new(decimal: maximum)
refute_data_loss Unicorn.new(decimal: 0 - maximum + delta)
assert_data_loss Unicorn.new(decimal: 0 - maximum)


maximum = BigDecimal.new(10 ** (Unicorn.columns_hash['unsigned_decimal'].precision - Unicorn.columns_hash['unsigned_decimal'].scale))
delta = BigDecimal.new(10 ** -(Unicorn.columns_hash['unsigned_decimal'].scale), Unicorn.columns_hash['unsigned_decimal'].precision)
maximum = BigDecimal(10 ** (Unicorn.columns_hash['unsigned_decimal'].precision - Unicorn.columns_hash['unsigned_decimal'].scale))
delta = BigDecimal(10 ** -(Unicorn.columns_hash['unsigned_decimal'].scale), Unicorn.columns_hash['unsigned_decimal'].precision)

refute_data_loss Unicorn.new(unsigned_decimal: maximum - delta)
assert_data_loss Unicorn.new(unsigned_decimal: maximum)
Expand Down Expand Up @@ -99,7 +99,7 @@ def test_blob_field_silently_drops_bytes_when_when_over_bytesize_limit
end

def test_utf8mb3_field_sliently_truncates_strings_after_first_4byte_character
emoji = '💩'
emoji = "\u{1F4A9}"
assert_equal 1, emoji.length
assert_equal 4, emoji.bytesize
assert_data_loss Unicorn.new(string: emoji)
Expand Down
2 changes: 1 addition & 1 deletion test/database_constraints_validator_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ def unsigned_integer_range
end

def test_error_messages
foo = Foo.new(string: 'ü' * 41, checked: nil, not_null_text: '')
foo = Foo.new(string: 'ü' * 41, checked: nil, not_null_text: "\u{1F4A9}")
refute foo.save

assert_equal ["is too long (maximum is 40 characters)"], foo.errors[:string]
Expand Down

0 comments on commit 0ae09af

Please sign in to comment.