Mailinglist Archive: yast-commit (1914 mails)

< Previous Next >
[yast-commit] <rest-service> basesystem : Implemented (and tested) Route.find(:all).
  • From: Martin Vidner <mvidner@xxxxxxx>
  • Date: Fri, 4 Sep 2009 17:44:30 +0200
  • Message-id: <E1Mlf01-0004DS-B1@xxxxxxxxxxxxxxxx>
ref: refs/heads/basesystem
commit 5036a1eded45eae1a2480fa057eb9bf69f309d7a
Author: Martin Vidner <mvidner@xxxxxxx>
Date: Fri Sep 4 16:07:24 2009 +0200

Implemented (and tested) Route.find(:all).

@id is optionally settable separately
---
plugins/network/app/models/route.rb | 15 ++++++++++++---
plugins/network/test/unit/route_test.rb | 8 ++++++++
2 files changed, 20 insertions(+), 3 deletions(-)

diff --git a/plugins/network/app/models/route.rb
b/plugins/network/app/models/route.rb
index b3c34ac..af1dab6 100644
--- a/plugins/network/app/models/route.rb
+++ b/plugins/network/app/models/route.rb
@@ -12,8 +12,9 @@ class Route

public

- def initialize(kwargs)
+ def initialize(kwargs, id = nil)
@via = kwargs["via"]
+ @id = kwargs["id"] || id
end

# fills route instance with data from YaPI.
@@ -21,8 +22,15 @@ class Route
# +warn+: YaPI implements default only.
def Route.find( which )
response = YastService.Call("YaPI::NETWORK::Read")
- ret = Route.new(response["routes"][which])
- ret.id = which
+ routes_h = response["routes"]
+ if which == :all
+ ret = Hash.new
+ routes_h.each do |id, route_h|
+ ret[id] = Route.new(route_h, id)
+ end
+ else
+ ret = Route.new(routes_h[which], which)
+ end
return ret
end

@@ -41,6 +49,7 @@ class Route
xml.instruct! unless options[:skip_instruct]

xml.route do
+ xml.id @id
xml.via @via
end
end
diff --git a/plugins/network/test/unit/route_test.rb
b/plugins/network/test/unit/route_test.rb
index 0a9cd7d..b52d11d 100644
--- a/plugins/network/test/unit/route_test.rb
+++ b/plugins/network/test/unit/route_test.rb
@@ -18,6 +18,14 @@ class RouteTest < ActiveSupport::TestCase
def test_getter
route=Route.find('default')
assert_equal '10.20.7.254', route.via
+ assert_equal 'default', route.id
+ end
+
+ def test_index
+ routes = Route.find(:all)
+ assert_instance_of Hash, routes
+ route = routes["default"]
+ assert_equal 'default', route.id
end

end
--
To unsubscribe, e-mail: yast-commit+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: yast-commit+help@xxxxxxxxxxxx

< Previous Next >
This Thread
  • No further messages