Skip to content

Commit b4bca94

Browse files
authored
Merge pull request rails#39023 from kamipo/remove_unused_operand1_operand2
Remove unused `operand1`/`operand2` aliases
2 parents 24c1429 + 9b8087a commit b4bca94

File tree

2 files changed

+0
-18
lines changed

2 files changed

+0
-18
lines changed

activerecord/lib/arel/nodes/equality.rb

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@ module Arel # :nodoc: all
44
module Nodes
55
class Equality < Arel::Nodes::Binary
66
def operator; :== end
7-
alias :operand1 :left
8-
alias :operand2 :right
97

108
def invert
119
Arel::Nodes::NotEqual.new(left, right)

activerecord/test/cases/arel/nodes/equality_test.rb

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -15,22 +15,6 @@ module Nodes
1515
end
1616
end
1717

18-
describe "operand1" do
19-
it "should equal left" do
20-
attr = Table.new(:users)[:id]
21-
left = attr.eq(10)
22-
_(left.left).must_equal left.operand1
23-
end
24-
end
25-
26-
describe "operand2" do
27-
it "should equal right" do
28-
attr = Table.new(:users)[:id]
29-
left = attr.eq(10)
30-
_(left.right).must_equal left.operand2
31-
end
32-
end
33-
3418
describe "to_sql" do
3519
it "takes an engine" do
3620
engine = FakeRecord::Base.new

0 commit comments

Comments
 (0)