2 tests fixed

Evandro Jr
1 parent 53af2595
Showing 1 changed file with 19 additions and 15 deletions Show diff stats
plugins/virtuoso/test/functional/virtuoso_plugin_admin_controller_test.rb
@@ -2,38 +2,42 @@ require File.dirname(__FILE__) + &#39;/../test_helper&#39;
 class VirtuosoPluginAdminControllerTest < ActionController::TestCase
+  attr_reader :environment
+  
   def setup
     @environment = Environment.default
     @profile = create_user('profile').person
     login_as(@profile.identifier)
+    post :index, :settings => mock_settings
   end
+  
+  def mock_settings
+    {
+      :virtuoso_uri=>"http://virtuoso.noosfero.com",
+                      :virtuoso_username=>"username", :virtuoso_password=>"password",
+                      :virtuoso_readonly_username=>"username",
+                      :virtuoso_readonly_password=>"password",
+                      :dspace_servers=>[
+                        {"dspace_uri"=>"http://dspace1.noosfero.com"},
+                        {"dspace_uri"=>"http://dspace2.noosfero.com"},
+                        {"dspace_uri"=>"http://dspace3.noosfero.com"}
+                      ]
+    }
+  end                 
-  attr_reader :environment
- 
   should 'save virtuoso plugin settings' do
-    post :index, :settings =>
-          {:virtuoso_uri=>"http://virtuoso.noosfero.com",
-           :virtuoso_username=>"username", :virtuoso_password=>"password",
-           :virtuoso_readonly_username=>"password",
-           :virtuoso_readonly_password=>"password",
-           :dspace_servers=>[ 
-                              {"dspace_uri"=>"http://dspace1.noosfero.com"},
-                              {"dspace_uri"=>"http://dspace2.noosfero.com"},
-                              {"dspace_uri"=>"http://dspace3.noosfero.com"}
-                            ]
-          }
     @settings = Noosfero::Plugin::Settings.new(environment.reload, VirtuosoPlugin)
     assert_equal 'http://virtuoso.noosfero.com', @settings.settings[:virtuoso_uri]
     assert_equal 'username', @settings.settings[:virtuoso_username]
     assert_equal 'password', @settings.settings[:virtuoso_password]
     assert_equal 'http://dspace1.noosfero.com', @settings.settings[:dspace_servers][0][:dspace_uri]
     assert_equal 'http://dspace2.noosfero.com', @settings.settings[:dspace_servers][1][:dspace_uri]
-    assert_equal 'http://dspace3.noosfero.com', @settings.settings[:dspace_servers][2][:dspace_uri]    
+    assert_equal 'http://dspace3.noosfero.com', @settings.settings[:dspace_servers][2][:dspace_uri]
     assert_redirected_to :action => 'index'
   end
   should 'redirect to index after save' do
-    post :index, :settings => {"virtuoso_uri" => 'http://virtuoso.noosfero.com'}
+    post :index, :settings => mock_settings
     assert_redirected_to :action => 'index'
   end