added support for YCP::Term in size() builtin

src/ruby/ycp/builtins.rb

@@ -45,7 +45,7 @@ def self.size object
       return nil if object.nil?
 
       case object
-      when String, Array, Hash then return object.size
+      when String, Array, Hash, YCP::Term then return object.size
       else
         raise "Invalid object for size() builtin"
       end

src/ruby/ycp/term.rb

@@ -21,6 +21,9 @@ def != second
     def [] index
       params[index]
     end
-
+
+    def size
+      params.size
+    end
   end
 end

tests/ruby/builtins_test.rb

@@ -5,6 +5,7 @@
 
 require "ycp/builtins"
 require "ycp/path"
+require "ycp/term"
 
 class BuiltinsPathTest < YCP::TestCase
   def test_add_list
@@ -77,5 +78,10 @@ def test_size
     assert_equal 0, YCP::Builtins.size([])
     assert_equal 0, YCP::Builtins.size({})
     assert_equal 0, YCP::Builtins.size("")
+
+    assert_equal 0, YCP::Builtins.size(YCP::Term.new(:HBox))
+    assert_equal 1, YCP::Builtins.size(YCP::Term.new(:HBox, "test"))
+    assert_equal 2, YCP::Builtins.size(YCP::Term.new(:HBox, "test", "test"))
+    assert_equal 1, YCP::Builtins.size(YCP::Term.new(:HBox, YCP::Term.new(:VBox, "test", "test")))
   end
 end

tests/ruby/term_test.rb

@@ -25,4 +25,11 @@ def test_equal
     assert_not_equal YCP::Term.new(:HBox), YCP::Term.new(:HBox, "test")
   end
 
+  def test_size
+    assert_equal 0, YCP::Term.new(:HBox).size
+    assert_equal 1, YCP::Term.new(:HBox, "test").size
+    assert_equal 1, YCP::Term.new(:HBox, "test").size
+    assert_equal 1, YCP::Term.new(:HBox, YCP::Term.new(:VBox, "test", "test")).size
+  end
+
 end