From 6e638bba594b6164190d2a6fb96ffa07a20b11f3 Mon Sep 17 00:00:00 2001 From: Ernie Miller Date: Sat, 18 Aug 2012 22:33:25 -0400 Subject: Add equality to ALL THE THINGS (that matter) People are often trying to use ARel nodes inside ActiveRecord, and when they do so, lots of things can break, because ActiveRecord relies on Array#uniq and sometimes hash key equality to handle values that end up in wheres, havings, etc. By implementing equality for all the nodes, we should hopefully be able to prevent any nodes (even nodes containing other nodes) from failing an equality check they should otherwise pass, and alleviate many of these errors. Fixes #130 --- test/nodes/test_and.rb | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) create mode 100644 test/nodes/test_and.rb (limited to 'test/nodes/test_and.rb') diff --git a/test/nodes/test_and.rb b/test/nodes/test_and.rb new file mode 100644 index 0000000000..88d6d61531 --- /dev/null +++ b/test/nodes/test_and.rb @@ -0,0 +1,20 @@ +require 'helper' + +module Arel + module Nodes + describe 'And' do + describe 'equality' do + it 'is equal with equal ivars' do + array = [And.new(['foo', 'bar']), And.new(['foo', 'bar'])] + assert_equal 1, array.uniq.size + end + + it 'is not equal with different ivars' do + array = [And.new(['foo', 'bar']), And.new(['foo', 'baz'])] + assert_equal 2, array.uniq.size + end + end + end + end +end + -- cgit v1.2.3