[api] make /about public available. it is used for live checks and is…

… a route causing minimal load.
openSUSE · Sep 18, 2013 · 182ce38 · 182ce38
1 parent 664fae7
commit 182ce38
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/api/app/controllers/about_controller.rb b/src/api/app/controllers/about_controller.rb
@@ -1,6 +1,7 @@
 class AboutController < ApplicationController
 
   validate_action :index => {:method => :get, :response => :about}
+  skip_before_action :extract_user
 
   def index
     @api_revision = CONFIG['version'].to_s

diff --git a/src/api/test/functional/about_controller_test.rb b/src/api/test/functional/about_controller_test.rb
@@ -2,17 +2,21 @@
 
 class AboutControllerTest < ActionDispatch::IntegrationTest 
 
-  def setup
+  def test_about
     prepare_request_valid_user
+    get "/about"
+    assert_response :success
+    assert_xml_tag( :tag => "about", :descendant => { :tag => "revision" } )
   end
- 
-  def test_about
+
+  def test_about_anonymous
     get "/about"
     assert_response :success
     assert_xml_tag( :tag => "about", :descendant => { :tag => "revision" } )
   end
 
   def test_application_controller
+    prepare_request_valid_user
     get "/about?user[asd]=yxc"
     assert_response 400
     assert_xml_tag( :tag => "status", :attributes => { :code => "invalid_parameter" } )