diff options
author | Ryuta Kamizono <kamipo@gmail.com> | 2019-06-15 22:17:20 +0900 |
---|---|---|
committer | Ryuta Kamizono <kamipo@gmail.com> | 2019-06-15 23:20:16 +0900 |
commit | fc38ff6e4417295c870f419f7c164ab5a7dbc4a5 (patch) | |
tree | b3c24f763b5f59412c07c2360680e0eb3f00f71c /activerecord/test/cases/arel/nodes/node_test.rb | |
parent | b388061947c087d0635619d7f0b3a2b1cc91a806 (diff) | |
download | rails-fc38ff6e4417295c870f419f7c164ab5a7dbc4a5.tar.gz rails-fc38ff6e4417295c870f419f7c164ab5a7dbc4a5.tar.bz2 rails-fc38ff6e4417295c870f419f7c164ab5a7dbc4a5.zip |
Remove unused `DepthFirst` visitor
We only use `ToSQL` visitors in the our codebase, do not use
`DepthFirst` and `Dot` visitors.
The `DepthFirst` visitor (which was introduced at c86c37e5f) is used to
traverse an Arel (partial) ast with depth first.
Is there any worth to keep that undocumented feature with much code and
test cases.
This removes that unused `DepthFirst` code and test cases.
Diffstat (limited to 'activerecord/test/cases/arel/nodes/node_test.rb')
-rw-r--r-- | activerecord/test/cases/arel/nodes/node_test.rb | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/activerecord/test/cases/arel/nodes/node_test.rb b/activerecord/test/cases/arel/nodes/node_test.rb index f4f07ef2c5..f1e0ce1ea9 100644 --- a/activerecord/test/cases/arel/nodes/node_test.rb +++ b/activerecord/test/cases/arel/nodes/node_test.rb @@ -18,24 +18,5 @@ module Arel assert klass.ancestors.include?(Nodes::Node), klass.name end end - - def test_each - list = [] - node = Nodes::Node.new - node.each { |n| list << n } - assert_equal [node], list - end - - def test_generator - list = [] - node = Nodes::Node.new - node.each.each { |n| list << n } - assert_equal [node], list - end - - def test_enumerable - node = Nodes::Node.new - assert_kind_of Enumerable, node - end end end |