YaST Commits September 2009

yast-commit@lists.opensuse.org

35 participants
1625 discussions

[yast-commit] <rest-service> network : Moved the other controllers, adapted route ctrl test.

by Martin Vidner

ref: refs/heads/network commit 038b2cadbf4208635f7c23e764bd4164d8cc12d9 Author: Martin Vidner <martin(a)vidner.net> Date: Tue Sep 1 16:12:25 2009 +0200 Moved the other controllers, adapted route ctrl test. --- plugins/network/app/controllers/dns_controller.rb | 45 ------------------ .../app/controllers/network/dns_controller.rb | 45 ++++++++++++++++++ .../app/controllers/network/routes_controller.rb | 49 ++++++++++++++++++++ .../network/app/controllers/routes_controller.rb | 49 -------------------- .../test/functional/routes_controller_test.rb | 26 +++++----- 5 files changed, 107 insertions(+), 107 deletions(-) diff --git a/plugins/network/app/controllers/dns_controller.rb b/plugins/network/app/controllers/dns_controller.rb deleted file mode 100644 index 6e92ee0..0000000 --- a/plugins/network/app/controllers/dns_controller.rb +++ /dev/null @@ -1,45 +0,0 @@ -# = Hostname controller -# Provides access to hostname settings for authenticated users. -# Main goal is checking permissions. -class DnsController < ApplicationController - - before_filter :login_required - - # Sets hostname settings. Requires write permissions for network YaPI. - def update - unless permission_check( "org.opensuse.yast.modules.yapi.network.write") - render ErrorResult.error(403, 1, "no permission") and return - end - - root = params[:hostname] - if root == nil - render ErrorResult.error(404, 2, "format or internal error") and return - end - - @dns = DNS.new(root) - @dns.save - render :show - end - - # See update - def create - update - end - - # Shows hostname settings. Requires read permission for network YaPI. - def show - - unless permission_check( "org.opensuse.yast.modules.yapi.network.read") - render ErrorResult.error( 403, 1, "no permission" ) and return - end - - @dns = DNS.find - - respond_to do |format| - format.html { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } - format.xml { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } - format.json { render :json => @dns.to_json } - end - end - -end diff --git a/plugins/network/app/controllers/network/dns_controller.rb b/plugins/network/app/controllers/network/dns_controller.rb new file mode 100644 index 0000000..fa13f91 --- /dev/null +++ b/plugins/network/app/controllers/network/dns_controller.rb @@ -0,0 +1,45 @@ +# = Hostname controller +# Provides access to hostname settings for authenticated users. +# Main goal is checking permissions. +class Network::DnsController < ApplicationController + + before_filter :login_required + + # Sets hostname settings. Requires write permissions for network YaPI. + def update + unless permission_check( "org.opensuse.yast.modules.yapi.network.write") + render ErrorResult.error(403, 1, "no permission") and return + end + + root = params[:hostname] + if root == nil + render ErrorResult.error(404, 2, "format or internal error") and return + end + + @dns = DNS.new(root) + @dns.save + render :show + end + + # See update + def create + update + end + + # Shows hostname settings. Requires read permission for network YaPI. + def show + + unless permission_check( "org.opensuse.yast.modules.yapi.network.read") + render ErrorResult.error( 403, 1, "no permission" ) and return + end + + @dns = DNS.find + + respond_to do |format| + format.html { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } + format.xml { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } + format.json { render :json => @dns.to_json } + end + end + +end diff --git a/plugins/network/app/controllers/network/routes_controller.rb b/plugins/network/app/controllers/network/routes_controller.rb new file mode 100644 index 0000000..dbf734e --- /dev/null +++ b/plugins/network/app/controllers/network/routes_controller.rb @@ -0,0 +1,49 @@ +# = Routing controller +# Provides access to hostname settings for authenticated users. +# Main goal is checking permissions. +class Network::RoutesController < ApplicationController + + before_filter :login_required + + # Sets hostname settings. Requires write permissions for network YaPI. + def update + unless permission_check( "org.opensuse.yast.modules.yapi.network.write") + render ErrorResult.error(403, 1, "no permission") and return + end + + root = params[:routes] + if root == nil + render ErrorResult.error(404, 2, "format or internal error") and return + end + + @routing = Route.new(root) + @routing.save + render :show + end + + # See update + def create + update + end + + # Shows hostname settings. Requires read permission for network YaPI. + def show + + unless permission_check( "org.opensuse.yast.modules.yapi.network.read") + render ErrorResult.error( 403, 1, "no permission" ) and return + end + + @route = Route.find(params[:id]) + + respond_to do |format| + format.html { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } + format.xml { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } + format.json { render :json => @route.to_json } + end + end + + def index + show + end + +end diff --git a/plugins/network/app/controllers/routes_controller.rb b/plugins/network/app/controllers/routes_controller.rb deleted file mode 100644 index dea97a8..0000000 --- a/plugins/network/app/controllers/routes_controller.rb +++ /dev/null @@ -1,49 +0,0 @@ -# = Routing controller -# Provides access to hostname settings for authenticated users. -# Main goal is checking permissions. -class RoutesController < ApplicationController - - before_filter :login_required - - # Sets hostname settings. Requires write permissions for network YaPI. - def update - unless permission_check( "org.opensuse.yast.modules.yapi.network.write") - render ErrorResult.error(403, 1, "no permission") and return - end - - root = params[:routes] - if root == nil - render ErrorResult.error(404, 2, "format or internal error") and return - end - - @routing = Route.new(root) - @routing.save - render :show - end - - # See update - def create - update - end - - # Shows hostname settings. Requires read permission for network YaPI. - def show - - unless permission_check( "org.opensuse.yast.modules.yapi.network.read") - render ErrorResult.error( 403, 1, "no permission" ) and return - end - - @route = Route.find(params[:id]) - - respond_to do |format| - format.html { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } - format.xml { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } - format.json { render :json => @route.to_json } - end - end - - def index - show - end - -end diff --git a/plugins/network/test/functional/routes_controller_test.rb b/plugins/network/test/functional/routes_controller_test.rb index 68e1366..0bd377a 100644 --- a/plugins/network/test/functional/routes_controller_test.rb +++ b/plugins/network/test/functional/routes_controller_test.rb @@ -1,23 +1,23 @@ require File.expand_path(File.dirname(__FILE__) + "/../test_helper") require 'test/unit' -require 'rubygems' require 'route' -require "scr" +require 'rubygems' require 'mocha' require File.expand_path( File.join("test","plugin_basic_tests"), RailsParent.parent ) -class RoutesControllerTest < Test::Unit::TestCase +class RoutesControllerTest < ActionController::TestCase + + def setup + @model_class = Route + Route.stubs(:find).returns(Route.new({"via" => "42.42.42.42"})) + @controller = Network::RoutesController.new + @request = ActionController::TestRequest.new + # http://railsforum.com/viewtopic.php?id=1719 + @request.session[:account_id] = 1 # defined in fixtures + end - # this is test only for mocked data - not very useful - # we need to extend it to test both model and controller - # to extract routes data from all YaPI map - # - def test_show - # if this code raises, test-unit 2.0.3 and mocha 0.9.5 fail - # with PASSTHROUGH_EXCEPTIONS. version mismatch?! - Route.expects(:find).returns({:routes=>{:default=>{'via'=>'10.20.30.40'}}}) - Route.find - end + # some cases fail because PluginBasicTests expects a singular controller + #include PluginBasicTests end -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

12 years, 11 months

[yast-commit] <rest-service> master : Moved the other controllers, adapted route ctrl test.

by Martin Vidner

ref: refs/heads/master commit 038b2cadbf4208635f7c23e764bd4164d8cc12d9 Author: Martin Vidner <martin(a)vidner.net> Date: Tue Sep 1 16:12:25 2009 +0200 Moved the other controllers, adapted route ctrl test. --- plugins/network/app/controllers/dns_controller.rb | 45 ------------------ .../app/controllers/network/dns_controller.rb | 45 ++++++++++++++++++ .../app/controllers/network/routes_controller.rb | 49 ++++++++++++++++++++ .../network/app/controllers/routes_controller.rb | 49 -------------------- .../test/functional/routes_controller_test.rb | 26 +++++----- 5 files changed, 107 insertions(+), 107 deletions(-) diff --git a/plugins/network/app/controllers/dns_controller.rb b/plugins/network/app/controllers/dns_controller.rb deleted file mode 100644 index 6e92ee0..0000000 --- a/plugins/network/app/controllers/dns_controller.rb +++ /dev/null @@ -1,45 +0,0 @@ -# = Hostname controller -# Provides access to hostname settings for authenticated users. -# Main goal is checking permissions. -class DnsController < ApplicationController - - before_filter :login_required - - # Sets hostname settings. Requires write permissions for network YaPI. - def update - unless permission_check( "org.opensuse.yast.modules.yapi.network.write") - render ErrorResult.error(403, 1, "no permission") and return - end - - root = params[:hostname] - if root == nil - render ErrorResult.error(404, 2, "format or internal error") and return - end - - @dns = DNS.new(root) - @dns.save - render :show - end - - # See update - def create - update - end - - # Shows hostname settings. Requires read permission for network YaPI. - def show - - unless permission_check( "org.opensuse.yast.modules.yapi.network.read") - render ErrorResult.error( 403, 1, "no permission" ) and return - end - - @dns = DNS.find - - respond_to do |format| - format.html { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } - format.xml { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } - format.json { render :json => @dns.to_json } - end - end - -end diff --git a/plugins/network/app/controllers/network/dns_controller.rb b/plugins/network/app/controllers/network/dns_controller.rb new file mode 100644 index 0000000..fa13f91 --- /dev/null +++ b/plugins/network/app/controllers/network/dns_controller.rb @@ -0,0 +1,45 @@ +# = Hostname controller +# Provides access to hostname settings for authenticated users. +# Main goal is checking permissions. +class Network::DnsController < ApplicationController + + before_filter :login_required + + # Sets hostname settings. Requires write permissions for network YaPI. + def update + unless permission_check( "org.opensuse.yast.modules.yapi.network.write") + render ErrorResult.error(403, 1, "no permission") and return + end + + root = params[:hostname] + if root == nil + render ErrorResult.error(404, 2, "format or internal error") and return + end + + @dns = DNS.new(root) + @dns.save + render :show + end + + # See update + def create + update + end + + # Shows hostname settings. Requires read permission for network YaPI. + def show + + unless permission_check( "org.opensuse.yast.modules.yapi.network.read") + render ErrorResult.error( 403, 1, "no permission" ) and return + end + + @dns = DNS.find + + respond_to do |format| + format.html { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } + format.xml { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } + format.json { render :json => @dns.to_json } + end + end + +end diff --git a/plugins/network/app/controllers/network/routes_controller.rb b/plugins/network/app/controllers/network/routes_controller.rb new file mode 100644 index 0000000..dbf734e --- /dev/null +++ b/plugins/network/app/controllers/network/routes_controller.rb @@ -0,0 +1,49 @@ +# = Routing controller +# Provides access to hostname settings for authenticated users. +# Main goal is checking permissions. +class Network::RoutesController < ApplicationController + + before_filter :login_required + + # Sets hostname settings. Requires write permissions for network YaPI. + def update + unless permission_check( "org.opensuse.yast.modules.yapi.network.write") + render ErrorResult.error(403, 1, "no permission") and return + end + + root = params[:routes] + if root == nil + render ErrorResult.error(404, 2, "format or internal error") and return + end + + @routing = Route.new(root) + @routing.save + render :show + end + + # See update + def create + update + end + + # Shows hostname settings. Requires read permission for network YaPI. + def show + + unless permission_check( "org.opensuse.yast.modules.yapi.network.read") + render ErrorResult.error( 403, 1, "no permission" ) and return + end + + @route = Route.find(params[:id]) + + respond_to do |format| + format.html { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } + format.xml { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } + format.json { render :json => @route.to_json } + end + end + + def index + show + end + +end diff --git a/plugins/network/app/controllers/routes_controller.rb b/plugins/network/app/controllers/routes_controller.rb deleted file mode 100644 index dea97a8..0000000 --- a/plugins/network/app/controllers/routes_controller.rb +++ /dev/null @@ -1,49 +0,0 @@ -# = Routing controller -# Provides access to hostname settings for authenticated users. -# Main goal is checking permissions. -class RoutesController < ApplicationController - - before_filter :login_required - - # Sets hostname settings. Requires write permissions for network YaPI. - def update - unless permission_check( "org.opensuse.yast.modules.yapi.network.write") - render ErrorResult.error(403, 1, "no permission") and return - end - - root = params[:routes] - if root == nil - render ErrorResult.error(404, 2, "format or internal error") and return - end - - @routing = Route.new(root) - @routing.save - render :show - end - - # See update - def create - update - end - - # Shows hostname settings. Requires read permission for network YaPI. - def show - - unless permission_check( "org.opensuse.yast.modules.yapi.network.read") - render ErrorResult.error( 403, 1, "no permission" ) and return - end - - @route = Route.find(params[:id]) - - respond_to do |format| - format.html { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } - format.xml { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } - format.json { render :json => @route.to_json } - end - end - - def index - show - end - -end diff --git a/plugins/network/test/functional/routes_controller_test.rb b/plugins/network/test/functional/routes_controller_test.rb index 68e1366..0bd377a 100644 --- a/plugins/network/test/functional/routes_controller_test.rb +++ b/plugins/network/test/functional/routes_controller_test.rb @@ -1,23 +1,23 @@ require File.expand_path(File.dirname(__FILE__) + "/../test_helper") require 'test/unit' -require 'rubygems' require 'route' -require "scr" +require 'rubygems' require 'mocha' require File.expand_path( File.join("test","plugin_basic_tests"), RailsParent.parent ) -class RoutesControllerTest < Test::Unit::TestCase +class RoutesControllerTest < ActionController::TestCase + + def setup + @model_class = Route + Route.stubs(:find).returns(Route.new({"via" => "42.42.42.42"})) + @controller = Network::RoutesController.new + @request = ActionController::TestRequest.new + # http://railsforum.com/viewtopic.php?id=1719 + @request.session[:account_id] = 1 # defined in fixtures + end - # this is test only for mocked data - not very useful - # we need to extend it to test both model and controller - # to extract routes data from all YaPI map - # - def test_show - # if this code raises, test-unit 2.0.3 and mocha 0.9.5 fail - # with PASSTHROUGH_EXCEPTIONS. version mismatch?! - Route.expects(:find).returns({:routes=>{:default=>{'via'=>'10.20.30.40'}}}) - Route.find - end + # some cases fail because PluginBasicTests expects a singular controller + #include PluginBasicTests end -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

12 years, 11 months

[yast-commit] <rest-service> dmacvicar_log_viewer : Moved the other controllers, adapted route ctrl test.

by Martin Vidner

ref: refs/heads/dmacvicar_log_viewer commit 038b2cadbf4208635f7c23e764bd4164d8cc12d9 Author: Martin Vidner <martin(a)vidner.net> Date: Tue Sep 1 16:12:25 2009 +0200 Moved the other controllers, adapted route ctrl test. --- plugins/network/app/controllers/dns_controller.rb | 45 ------------------ .../app/controllers/network/dns_controller.rb | 45 ++++++++++++++++++ .../app/controllers/network/routes_controller.rb | 49 ++++++++++++++++++++ .../network/app/controllers/routes_controller.rb | 49 -------------------- .../test/functional/routes_controller_test.rb | 26 +++++----- 5 files changed, 107 insertions(+), 107 deletions(-) diff --git a/plugins/network/app/controllers/dns_controller.rb b/plugins/network/app/controllers/dns_controller.rb deleted file mode 100644 index 6e92ee0..0000000 --- a/plugins/network/app/controllers/dns_controller.rb +++ /dev/null @@ -1,45 +0,0 @@ -# = Hostname controller -# Provides access to hostname settings for authenticated users. -# Main goal is checking permissions. -class DnsController < ApplicationController - - before_filter :login_required - - # Sets hostname settings. Requires write permissions for network YaPI. - def update - unless permission_check( "org.opensuse.yast.modules.yapi.network.write") - render ErrorResult.error(403, 1, "no permission") and return - end - - root = params[:hostname] - if root == nil - render ErrorResult.error(404, 2, "format or internal error") and return - end - - @dns = DNS.new(root) - @dns.save - render :show - end - - # See update - def create - update - end - - # Shows hostname settings. Requires read permission for network YaPI. - def show - - unless permission_check( "org.opensuse.yast.modules.yapi.network.read") - render ErrorResult.error( 403, 1, "no permission" ) and return - end - - @dns = DNS.find - - respond_to do |format| - format.html { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } - format.xml { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } - format.json { render :json => @dns.to_json } - end - end - -end diff --git a/plugins/network/app/controllers/network/dns_controller.rb b/plugins/network/app/controllers/network/dns_controller.rb new file mode 100644 index 0000000..fa13f91 --- /dev/null +++ b/plugins/network/app/controllers/network/dns_controller.rb @@ -0,0 +1,45 @@ +# = Hostname controller +# Provides access to hostname settings for authenticated users. +# Main goal is checking permissions. +class Network::DnsController < ApplicationController + + before_filter :login_required + + # Sets hostname settings. Requires write permissions for network YaPI. + def update + unless permission_check( "org.opensuse.yast.modules.yapi.network.write") + render ErrorResult.error(403, 1, "no permission") and return + end + + root = params[:hostname] + if root == nil + render ErrorResult.error(404, 2, "format or internal error") and return + end + + @dns = DNS.new(root) + @dns.save + render :show + end + + # See update + def create + update + end + + # Shows hostname settings. Requires read permission for network YaPI. + def show + + unless permission_check( "org.opensuse.yast.modules.yapi.network.read") + render ErrorResult.error( 403, 1, "no permission" ) and return + end + + @dns = DNS.find + + respond_to do |format| + format.html { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } + format.xml { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } + format.json { render :json => @dns.to_json } + end + end + +end diff --git a/plugins/network/app/controllers/network/routes_controller.rb b/plugins/network/app/controllers/network/routes_controller.rb new file mode 100644 index 0000000..dbf734e --- /dev/null +++ b/plugins/network/app/controllers/network/routes_controller.rb @@ -0,0 +1,49 @@ +# = Routing controller +# Provides access to hostname settings for authenticated users. +# Main goal is checking permissions. +class Network::RoutesController < ApplicationController + + before_filter :login_required + + # Sets hostname settings. Requires write permissions for network YaPI. + def update + unless permission_check( "org.opensuse.yast.modules.yapi.network.write") + render ErrorResult.error(403, 1, "no permission") and return + end + + root = params[:routes] + if root == nil + render ErrorResult.error(404, 2, "format or internal error") and return + end + + @routing = Route.new(root) + @routing.save + render :show + end + + # See update + def create + update + end + + # Shows hostname settings. Requires read permission for network YaPI. + def show + + unless permission_check( "org.opensuse.yast.modules.yapi.network.read") + render ErrorResult.error( 403, 1, "no permission" ) and return + end + + @route = Route.find(params[:id]) + + respond_to do |format| + format.html { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } + format.xml { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } + format.json { render :json => @route.to_json } + end + end + + def index + show + end + +end diff --git a/plugins/network/app/controllers/routes_controller.rb b/plugins/network/app/controllers/routes_controller.rb deleted file mode 100644 index dea97a8..0000000 --- a/plugins/network/app/controllers/routes_controller.rb +++ /dev/null @@ -1,49 +0,0 @@ -# = Routing controller -# Provides access to hostname settings for authenticated users. -# Main goal is checking permissions. -class RoutesController < ApplicationController - - before_filter :login_required - - # Sets hostname settings. Requires write permissions for network YaPI. - def update - unless permission_check( "org.opensuse.yast.modules.yapi.network.write") - render ErrorResult.error(403, 1, "no permission") and return - end - - root = params[:routes] - if root == nil - render ErrorResult.error(404, 2, "format or internal error") and return - end - - @routing = Route.new(root) - @routing.save - render :show - end - - # See update - def create - update - end - - # Shows hostname settings. Requires read permission for network YaPI. - def show - - unless permission_check( "org.opensuse.yast.modules.yapi.network.read") - render ErrorResult.error( 403, 1, "no permission" ) and return - end - - @route = Route.find(params[:id]) - - respond_to do |format| - format.html { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } - format.xml { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } - format.json { render :json => @route.to_json } - end - end - - def index - show - end - -end diff --git a/plugins/network/test/functional/routes_controller_test.rb b/plugins/network/test/functional/routes_controller_test.rb index 68e1366..0bd377a 100644 --- a/plugins/network/test/functional/routes_controller_test.rb +++ b/plugins/network/test/functional/routes_controller_test.rb @@ -1,23 +1,23 @@ require File.expand_path(File.dirname(__FILE__) + "/../test_helper") require 'test/unit' -require 'rubygems' require 'route' -require "scr" +require 'rubygems' require 'mocha' require File.expand_path( File.join("test","plugin_basic_tests"), RailsParent.parent ) -class RoutesControllerTest < Test::Unit::TestCase +class RoutesControllerTest < ActionController::TestCase + + def setup + @model_class = Route + Route.stubs(:find).returns(Route.new({"via" => "42.42.42.42"})) + @controller = Network::RoutesController.new + @request = ActionController::TestRequest.new + # http://railsforum.com/viewtopic.php?id=1719 + @request.session[:account_id] = 1 # defined in fixtures + end - # this is test only for mocked data - not very useful - # we need to extend it to test both model and controller - # to extract routes data from all YaPI map - # - def test_show - # if this code raises, test-unit 2.0.3 and mocha 0.9.5 fail - # with PASSTHROUGH_EXCEPTIONS. version mismatch?! - Route.expects(:find).returns({:routes=>{:default=>{'via'=>'10.20.30.40'}}}) - Route.find - end + # some cases fail because PluginBasicTests expects a singular controller + #include PluginBasicTests end -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

12 years, 11 months

[yast-commit] <rest-service> basesystem : Moved the other controllers, adapted route ctrl test.

by Martin Vidner

ref: refs/heads/basesystem commit 038b2cadbf4208635f7c23e764bd4164d8cc12d9 Author: Martin Vidner <martin(a)vidner.net> Date: Tue Sep 1 16:12:25 2009 +0200 Moved the other controllers, adapted route ctrl test. --- plugins/network/app/controllers/dns_controller.rb | 45 ------------------ .../app/controllers/network/dns_controller.rb | 45 ++++++++++++++++++ .../app/controllers/network/routes_controller.rb | 49 ++++++++++++++++++++ .../network/app/controllers/routes_controller.rb | 49 -------------------- .../test/functional/routes_controller_test.rb | 26 +++++----- 5 files changed, 107 insertions(+), 107 deletions(-) diff --git a/plugins/network/app/controllers/dns_controller.rb b/plugins/network/app/controllers/dns_controller.rb deleted file mode 100644 index 6e92ee0..0000000 --- a/plugins/network/app/controllers/dns_controller.rb +++ /dev/null @@ -1,45 +0,0 @@ -# = Hostname controller -# Provides access to hostname settings for authenticated users. -# Main goal is checking permissions. -class DnsController < ApplicationController - - before_filter :login_required - - # Sets hostname settings. Requires write permissions for network YaPI. - def update - unless permission_check( "org.opensuse.yast.modules.yapi.network.write") - render ErrorResult.error(403, 1, "no permission") and return - end - - root = params[:hostname] - if root == nil - render ErrorResult.error(404, 2, "format or internal error") and return - end - - @dns = DNS.new(root) - @dns.save - render :show - end - - # See update - def create - update - end - - # Shows hostname settings. Requires read permission for network YaPI. - def show - - unless permission_check( "org.opensuse.yast.modules.yapi.network.read") - render ErrorResult.error( 403, 1, "no permission" ) and return - end - - @dns = DNS.find - - respond_to do |format| - format.html { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } - format.xml { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } - format.json { render :json => @dns.to_json } - end - end - -end diff --git a/plugins/network/app/controllers/network/dns_controller.rb b/plugins/network/app/controllers/network/dns_controller.rb new file mode 100644 index 0000000..fa13f91 --- /dev/null +++ b/plugins/network/app/controllers/network/dns_controller.rb @@ -0,0 +1,45 @@ +# = Hostname controller +# Provides access to hostname settings for authenticated users. +# Main goal is checking permissions. +class Network::DnsController < ApplicationController + + before_filter :login_required + + # Sets hostname settings. Requires write permissions for network YaPI. + def update + unless permission_check( "org.opensuse.yast.modules.yapi.network.write") + render ErrorResult.error(403, 1, "no permission") and return + end + + root = params[:hostname] + if root == nil + render ErrorResult.error(404, 2, "format or internal error") and return + end + + @dns = DNS.new(root) + @dns.save + render :show + end + + # See update + def create + update + end + + # Shows hostname settings. Requires read permission for network YaPI. + def show + + unless permission_check( "org.opensuse.yast.modules.yapi.network.read") + render ErrorResult.error( 403, 1, "no permission" ) and return + end + + @dns = DNS.find + + respond_to do |format| + format.html { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } + format.xml { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } + format.json { render :json => @dns.to_json } + end + end + +end diff --git a/plugins/network/app/controllers/network/routes_controller.rb b/plugins/network/app/controllers/network/routes_controller.rb new file mode 100644 index 0000000..dbf734e --- /dev/null +++ b/plugins/network/app/controllers/network/routes_controller.rb @@ -0,0 +1,49 @@ +# = Routing controller +# Provides access to hostname settings for authenticated users. +# Main goal is checking permissions. +class Network::RoutesController < ApplicationController + + before_filter :login_required + + # Sets hostname settings. Requires write permissions for network YaPI. + def update + unless permission_check( "org.opensuse.yast.modules.yapi.network.write") + render ErrorResult.error(403, 1, "no permission") and return + end + + root = params[:routes] + if root == nil + render ErrorResult.error(404, 2, "format or internal error") and return + end + + @routing = Route.new(root) + @routing.save + render :show + end + + # See update + def create + update + end + + # Shows hostname settings. Requires read permission for network YaPI. + def show + + unless permission_check( "org.opensuse.yast.modules.yapi.network.read") + render ErrorResult.error( 403, 1, "no permission" ) and return + end + + @route = Route.find(params[:id]) + + respond_to do |format| + format.html { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } + format.xml { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } + format.json { render :json => @route.to_json } + end + end + + def index + show + end + +end diff --git a/plugins/network/app/controllers/routes_controller.rb b/plugins/network/app/controllers/routes_controller.rb deleted file mode 100644 index dea97a8..0000000 --- a/plugins/network/app/controllers/routes_controller.rb +++ /dev/null @@ -1,49 +0,0 @@ -# = Routing controller -# Provides access to hostname settings for authenticated users. -# Main goal is checking permissions. -class RoutesController < ApplicationController - - before_filter :login_required - - # Sets hostname settings. Requires write permissions for network YaPI. - def update - unless permission_check( "org.opensuse.yast.modules.yapi.network.write") - render ErrorResult.error(403, 1, "no permission") and return - end - - root = params[:routes] - if root == nil - render ErrorResult.error(404, 2, "format or internal error") and return - end - - @routing = Route.new(root) - @routing.save - render :show - end - - # See update - def create - update - end - - # Shows hostname settings. Requires read permission for network YaPI. - def show - - unless permission_check( "org.opensuse.yast.modules.yapi.network.read") - render ErrorResult.error( 403, 1, "no permission" ) and return - end - - @route = Route.find(params[:id]) - - respond_to do |format| - format.html { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } - format.xml { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } - format.json { render :json => @route.to_json } - end - end - - def index - show - end - -end diff --git a/plugins/network/test/functional/routes_controller_test.rb b/plugins/network/test/functional/routes_controller_test.rb index 68e1366..0bd377a 100644 --- a/plugins/network/test/functional/routes_controller_test.rb +++ b/plugins/network/test/functional/routes_controller_test.rb @@ -1,23 +1,23 @@ require File.expand_path(File.dirname(__FILE__) + "/../test_helper") require 'test/unit' -require 'rubygems' require 'route' -require "scr" +require 'rubygems' require 'mocha' require File.expand_path( File.join("test","plugin_basic_tests"), RailsParent.parent ) -class RoutesControllerTest < Test::Unit::TestCase +class RoutesControllerTest < ActionController::TestCase + + def setup + @model_class = Route + Route.stubs(:find).returns(Route.new({"via" => "42.42.42.42"})) + @controller = Network::RoutesController.new + @request = ActionController::TestRequest.new + # http://railsforum.com/viewtopic.php?id=1719 + @request.session[:account_id] = 1 # defined in fixtures + end - # this is test only for mocked data - not very useful - # we need to extend it to test both model and controller - # to extract routes data from all YaPI map - # - def test_show - # if this code raises, test-unit 2.0.3 and mocha 0.9.5 fail - # with PASSTHROUGH_EXCEPTIONS. version mismatch?! - Route.expects(:find).returns({:routes=>{:default=>{'via'=>'10.20.30.40'}}}) - Route.find - end + # some cases fail because PluginBasicTests expects a singular controller + #include PluginBasicTests end -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

12 years, 11 months

[yast-commit] <rest-service> jr_ntp : Moved the other controllers, adapted route ctrl test.

by Martin Vidner

ref: refs/heads/jr_ntp commit 038b2cadbf4208635f7c23e764bd4164d8cc12d9 Author: Martin Vidner <martin(a)vidner.net> Date: Tue Sep 1 16:12:25 2009 +0200 Moved the other controllers, adapted route ctrl test. --- plugins/network/app/controllers/dns_controller.rb | 45 ------------------ .../app/controllers/network/dns_controller.rb | 45 ++++++++++++++++++ .../app/controllers/network/routes_controller.rb | 49 ++++++++++++++++++++ .../network/app/controllers/routes_controller.rb | 49 -------------------- .../test/functional/routes_controller_test.rb | 26 +++++----- 5 files changed, 107 insertions(+), 107 deletions(-) diff --git a/plugins/network/app/controllers/dns_controller.rb b/plugins/network/app/controllers/dns_controller.rb deleted file mode 100644 index 6e92ee0..0000000 --- a/plugins/network/app/controllers/dns_controller.rb +++ /dev/null @@ -1,45 +0,0 @@ -# = Hostname controller -# Provides access to hostname settings for authenticated users. -# Main goal is checking permissions. -class DnsController < ApplicationController - - before_filter :login_required - - # Sets hostname settings. Requires write permissions for network YaPI. - def update - unless permission_check( "org.opensuse.yast.modules.yapi.network.write") - render ErrorResult.error(403, 1, "no permission") and return - end - - root = params[:hostname] - if root == nil - render ErrorResult.error(404, 2, "format or internal error") and return - end - - @dns = DNS.new(root) - @dns.save - render :show - end - - # See update - def create - update - end - - # Shows hostname settings. Requires read permission for network YaPI. - def show - - unless permission_check( "org.opensuse.yast.modules.yapi.network.read") - render ErrorResult.error( 403, 1, "no permission" ) and return - end - - @dns = DNS.find - - respond_to do |format| - format.html { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } - format.xml { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } - format.json { render :json => @dns.to_json } - end - end - -end diff --git a/plugins/network/app/controllers/network/dns_controller.rb b/plugins/network/app/controllers/network/dns_controller.rb new file mode 100644 index 0000000..fa13f91 --- /dev/null +++ b/plugins/network/app/controllers/network/dns_controller.rb @@ -0,0 +1,45 @@ +# = Hostname controller +# Provides access to hostname settings for authenticated users. +# Main goal is checking permissions. +class Network::DnsController < ApplicationController + + before_filter :login_required + + # Sets hostname settings. Requires write permissions for network YaPI. + def update + unless permission_check( "org.opensuse.yast.modules.yapi.network.write") + render ErrorResult.error(403, 1, "no permission") and return + end + + root = params[:hostname] + if root == nil + render ErrorResult.error(404, 2, "format or internal error") and return + end + + @dns = DNS.new(root) + @dns.save + render :show + end + + # See update + def create + update + end + + # Shows hostname settings. Requires read permission for network YaPI. + def show + + unless permission_check( "org.opensuse.yast.modules.yapi.network.read") + render ErrorResult.error( 403, 1, "no permission" ) and return + end + + @dns = DNS.find + + respond_to do |format| + format.html { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } + format.xml { render :xml => @dns.to_xml( :root => "dns", :dasherize => false ) } + format.json { render :json => @dns.to_json } + end + end + +end diff --git a/plugins/network/app/controllers/network/routes_controller.rb b/plugins/network/app/controllers/network/routes_controller.rb new file mode 100644 index 0000000..dbf734e --- /dev/null +++ b/plugins/network/app/controllers/network/routes_controller.rb @@ -0,0 +1,49 @@ +# = Routing controller +# Provides access to hostname settings for authenticated users. +# Main goal is checking permissions. +class Network::RoutesController < ApplicationController + + before_filter :login_required + + # Sets hostname settings. Requires write permissions for network YaPI. + def update + unless permission_check( "org.opensuse.yast.modules.yapi.network.write") + render ErrorResult.error(403, 1, "no permission") and return + end + + root = params[:routes] + if root == nil + render ErrorResult.error(404, 2, "format or internal error") and return + end + + @routing = Route.new(root) + @routing.save + render :show + end + + # See update + def create + update + end + + # Shows hostname settings. Requires read permission for network YaPI. + def show + + unless permission_check( "org.opensuse.yast.modules.yapi.network.read") + render ErrorResult.error( 403, 1, "no permission" ) and return + end + + @route = Route.find(params[:id]) + + respond_to do |format| + format.html { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } + format.xml { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } + format.json { render :json => @route.to_json } + end + end + + def index + show + end + +end diff --git a/plugins/network/app/controllers/routes_controller.rb b/plugins/network/app/controllers/routes_controller.rb deleted file mode 100644 index dea97a8..0000000 --- a/plugins/network/app/controllers/routes_controller.rb +++ /dev/null @@ -1,49 +0,0 @@ -# = Routing controller -# Provides access to hostname settings for authenticated users. -# Main goal is checking permissions. -class RoutesController < ApplicationController - - before_filter :login_required - - # Sets hostname settings. Requires write permissions for network YaPI. - def update - unless permission_check( "org.opensuse.yast.modules.yapi.network.write") - render ErrorResult.error(403, 1, "no permission") and return - end - - root = params[:routes] - if root == nil - render ErrorResult.error(404, 2, "format or internal error") and return - end - - @routing = Route.new(root) - @routing.save - render :show - end - - # See update - def create - update - end - - # Shows hostname settings. Requires read permission for network YaPI. - def show - - unless permission_check( "org.opensuse.yast.modules.yapi.network.read") - render ErrorResult.error( 403, 1, "no permission" ) and return - end - - @route = Route.find(params[:id]) - - respond_to do |format| - format.html { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } - format.xml { render :xml => @route.to_xml( :root => "route", :dasherize => false ) } - format.json { render :json => @route.to_json } - end - end - - def index - show - end - -end diff --git a/plugins/network/test/functional/routes_controller_test.rb b/plugins/network/test/functional/routes_controller_test.rb index 68e1366..0bd377a 100644 --- a/plugins/network/test/functional/routes_controller_test.rb +++ b/plugins/network/test/functional/routes_controller_test.rb @@ -1,23 +1,23 @@ require File.expand_path(File.dirname(__FILE__) + "/../test_helper") require 'test/unit' -require 'rubygems' require 'route' -require "scr" +require 'rubygems' require 'mocha' require File.expand_path( File.join("test","plugin_basic_tests"), RailsParent.parent ) -class RoutesControllerTest < Test::Unit::TestCase +class RoutesControllerTest < ActionController::TestCase + + def setup + @model_class = Route + Route.stubs(:find).returns(Route.new({"via" => "42.42.42.42"})) + @controller = Network::RoutesController.new + @request = ActionController::TestRequest.new + # http://railsforum.com/viewtopic.php?id=1719 + @request.session[:account_id] = 1 # defined in fixtures + end - # this is test only for mocked data - not very useful - # we need to extend it to test both model and controller - # to extract routes data from all YaPI map - # - def test_show - # if this code raises, test-unit 2.0.3 and mocha 0.9.5 fail - # with PASSTHROUGH_EXCEPTIONS. version mismatch?! - Route.expects(:find).returns({:routes=>{:default=>{'via'=>'10.20.30.40'}}}) - Route.find - end + # some cases fail because PluginBasicTests expects a singular controller + #include PluginBasicTests end -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

12 years, 11 months

[yast-commit] <web-client> master : catch exception, render error message

by Ladislav Slezak

ref: refs/heads/master commit 73d0d1bfe7f4db8080d27c5c8eeead3cb6801311 Author: Ladislav Slezak <lslezak(a)novell.com> Date: Tue Sep 1 16:00:27 2009 +0200 catch exception, render error message --- .../app/controllers/services_controller.rb | 8 +++++++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/plugins/services/app/controllers/services_controller.rb b/plugins/services/app/controllers/services_controller.rb index f4aa94b..31e5821 100644 --- a/plugins/services/app/controllers/services_controller.rb +++ b/plugins/services/app/controllers/services_controller.rb @@ -25,7 +25,13 @@ class ServicesController < ApplicationController def show_status return unless client_permissions - @response = @client.find(params[:id]) + begin + @response = @client.find(params[:id]) + rescue ActiveResource::ResourceNotFound => e + Rails.logger.error "Resource not found: #{e.to_s}: #{e.response.body}" + render :text => _('(cannot read status)') and return + end + render(:partial =>'status', :object => @response.status, :params => params) end -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

12 years, 11 months

[yast-commit] <web-client> network : catch exception, render error message

by Ladislav Slezak

ref: refs/heads/network commit 73d0d1bfe7f4db8080d27c5c8eeead3cb6801311 Author: Ladislav Slezak <lslezak(a)novell.com> Date: Tue Sep 1 16:00:27 2009 +0200 catch exception, render error message --- .../app/controllers/services_controller.rb | 8 +++++++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/plugins/services/app/controllers/services_controller.rb b/plugins/services/app/controllers/services_controller.rb index f4aa94b..31e5821 100644 --- a/plugins/services/app/controllers/services_controller.rb +++ b/plugins/services/app/controllers/services_controller.rb @@ -25,7 +25,13 @@ class ServicesController < ApplicationController def show_status return unless client_permissions - @response = @client.find(params[:id]) + begin + @response = @client.find(params[:id]) + rescue ActiveResource::ResourceNotFound => e + Rails.logger.error "Resource not found: #{e.to_s}: #{e.response.body}" + render :text => _('(cannot read status)') and return + end + render(:partial =>'status', :object => @response.status, :params => params) end -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

12 years, 11 months

[yast-commit] <web-client> basesystem : catch exception, render error message

by Ladislav Slezak

ref: refs/heads/basesystem commit 73d0d1bfe7f4db8080d27c5c8eeead3cb6801311 Author: Ladislav Slezak <lslezak(a)novell.com> Date: Tue Sep 1 16:00:27 2009 +0200 catch exception, render error message --- .../app/controllers/services_controller.rb | 8 +++++++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/plugins/services/app/controllers/services_controller.rb b/plugins/services/app/controllers/services_controller.rb index f4aa94b..31e5821 100644 --- a/plugins/services/app/controllers/services_controller.rb +++ b/plugins/services/app/controllers/services_controller.rb @@ -25,7 +25,13 @@ class ServicesController < ApplicationController def show_status return unless client_permissions - @response = @client.find(params[:id]) + begin + @response = @client.find(params[:id]) + rescue ActiveResource::ResourceNotFound => e + Rails.logger.error "Resource not found: #{e.to_s}: #{e.response.body}" + render :text => _('(cannot read status)') and return + end + render(:partial =>'status', :object => @response.status, :params => params) end -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

12 years, 11 months

[yast-commit] <rest-service> master : catch exceptions, return an error message

by Ladislav Slezak

ref: refs/heads/master commit e3f4f4eb81c5ffdc3c658f7838b0c91c53fed098 Author: Ladislav Slezak <lslezak(a)novell.com> Date: Tue Sep 1 15:59:33 2009 +0200 catch exceptions, return an error message --- .../app/controllers/services_controller.rb | 13 +++++++++++-- 1 files changed, 11 insertions(+), 2 deletions(-) diff --git a/plugins/services/app/controllers/services_controller.rb b/plugins/services/app/controllers/services_controller.rb index 246e822..fd3a471 100644 --- a/plugins/services/app/controllers/services_controller.rb +++ b/plugins/services/app/controllers/services_controller.rb @@ -8,7 +8,11 @@ class ServicesController < ApplicationController render ErrorResult.error(403, 1, "no permission") and return end - @services = Service.find_all params + begin + @services = Service.find_all params + rescue Exception => e + render ErrorResult.error(404, 107, e.to_s) and return + end end # GET /services/service_name @@ -20,7 +24,12 @@ class ServicesController < ApplicationController end @service = Service.new(params[:id]) - @service.read_status + + begin + @service.read_status + rescue Exception => e + render ErrorResult.error(404, 108, e.to_s) and return + end respond_to do |format| format.html { render :xml => @service.to_xml(:root => 'service', :dasherize => false), :location => "none" } #return xml only -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

12 years, 11 months

[yast-commit] <rest-service> network : catch exceptions, return an error message

by Ladislav Slezak

ref: refs/heads/network commit e3f4f4eb81c5ffdc3c658f7838b0c91c53fed098 Author: Ladislav Slezak <lslezak(a)novell.com> Date: Tue Sep 1 15:59:33 2009 +0200 catch exceptions, return an error message --- .../app/controllers/services_controller.rb | 13 +++++++++++-- 1 files changed, 11 insertions(+), 2 deletions(-) diff --git a/plugins/services/app/controllers/services_controller.rb b/plugins/services/app/controllers/services_controller.rb index 246e822..fd3a471 100644 --- a/plugins/services/app/controllers/services_controller.rb +++ b/plugins/services/app/controllers/services_controller.rb @@ -8,7 +8,11 @@ class ServicesController < ApplicationController render ErrorResult.error(403, 1, "no permission") and return end - @services = Service.find_all params + begin + @services = Service.find_all params + rescue Exception => e + render ErrorResult.error(404, 107, e.to_s) and return + end end # GET /services/service_name @@ -20,7 +24,12 @@ class ServicesController < ApplicationController end @service = Service.new(params[:id]) - @service.read_status + + begin + @service.read_status + rescue Exception => e + render ErrorResult.error(404, 108, e.to_s) and return + end respond_to do |format| format.html { render :xml => @service.to_xml(:root => 'service', :dasherize => false), :location => "none" } #return xml only -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

12 years, 11 months

Jump to page:

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

YaST Commits September 2009