Fix new ruboop warnings

[rails.git] / test / integration / page_locale_test.rb

diff --git a/test/integration/page_locale_test.rb b/test/integration/page_locale_test.rb
index 43810db997885e1815ab20d2d668869649bc322c..c8e5026d452d113802ed369a03f8131aa19d0a53 100644 (file)
--- a/test/integration/page_locale_test.rb
+++ b/test/integration/page_locale_test.rb
@@ -1,45 +1,41 @@
 require "test_helper"
 
 class PageLocaleTest < ActionDispatch::IntegrationTest
 require "test_helper"
 
 class PageLocaleTest < ActionDispatch::IntegrationTest

-  fixtures :users
-
-  def setup
-    I18n.locale = "en"
-  end
-
-  def teardown
-    I18n.locale = "en"
-  end
-

   def test_defaulting
   def test_defaulting

-    user = users(:second_public_user)
+    user = create(:user, :languages => [])

 
 

-    post_via_redirect "/login", :username => user.email, :password => "test"
+    get "/login"
+    follow_redirect!
+    post "/login", :params => { :username => user.email, :password => "test" }
+    follow_redirect!

 
 

-    get "/diary/new", {}
-    assert_equal [], User.find(user.id).languages
+    get "/diary/new"
+    assert_empty User.find(user.id).languages

     assert_select "html[lang=?]", "en"
 
     assert_select "html[lang=?]", "en"
 

-    get "/diary/new", {}, { "HTTP_ACCEPT_LANGUAGE" => "fr, en" }
-    assert_equal %w(fr en), User.find(user.id).languages
+    get "/diary/new", :headers => { "HTTP_ACCEPT_LANGUAGE" => "fr, en" }
+    assert_equal %w[fr en], User.find(user.id).languages

     assert_select "html[lang=?]", "fr"
   end
 
   def test_override
     assert_select "html[lang=?]", "fr"
   end
 
   def test_override

-    user = users(:german_user)
+    user = create(:user, :languages => ["de"])

 
     get "/diary"
     assert_select "html[lang=?]", "en"
 
 
     get "/diary"
     assert_select "html[lang=?]", "en"
 

-    get "/diary", :locale => "es"
+    get "/diary", :params => { :locale => "es" }

     assert_select "html[lang=?]", "es"
 
     assert_select "html[lang=?]", "es"
 

-    post_via_redirect "/login", :username => user.email, :password => "test"
+    get "/login"
+    follow_redirect!
+    post "/login", :params => { :username => user.email, :password => "test" }
+    follow_redirect!

 
     get "/diary"
     assert_select "html[lang=?]", "de"
 
 
     get "/diary"
     assert_select "html[lang=?]", "de"
 

-    get "/diary", :locale => "fr"
+    get "/diary", :params => { :locale => "fr" }

     assert_select "html[lang=?]", "fr"
   end
 end
     assert_select "html[lang=?]", "fr"
   end
 end