Fix rubocop lint issues

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

diff --git a/test/integration/client_application_test.rb b/test/integration/client_application_test.rb
index 80da36deb38edb975e5753dcbf465c24fe9adf3a..9aa1bc56a1f3bd726aa46931129fabb8f4bfd4a5 100644 (file)
--- a/test/integration/client_application_test.rb
+++ b/test/integration/client_application_test.rb
@@ -1,6 +1,6 @@
-require File.dirname(__FILE__) + '/../test_helper'
+require 'test_helper'
 
-class ClientApplicationTest < ActionController::IntegrationTest
+class ClientApplicationTest < ActionDispatch::IntegrationTest
   fixtures :users, :client_applications
 
   ##
@@ -12,15 +12,18 @@ class ClientApplicationTest < ActionController::IntegrationTest
     assert_redirected_to "controller" => "user", "action" => "login", "cookie_test" => "true"
     follow_redirect!
     assert_response :success
-    post '/login', {'username' => "test@example.com", 'password' => "test", :referer => '/user/test2'}
+    post '/login', 'username' => "test@example.com", 'password' => "test", :referer => '/user/test2'
     assert_response :redirect
     follow_redirect!
     assert_response :success
     assert_template 'user/view'
+    get '/user/test2/account'
+    assert_response :success
+    assert_template 'user/account'
 
     # check that the form to allow new client application creations exists
-    assert_in_body do
-      assert_select "a[href='/user/test2/oauth_clients']"
+    assert_in_heading do
+      assert_select "ul.secondary-actions li a[href='/user/test2/oauth_clients']"
     end
 
     # now we follow the link to the oauth client list
@@ -33,10 +36,12 @@ class ClientApplicationTest < ActionController::IntegrationTest
     # now we follow the link to the new oauth client page
     get '/user/test2/oauth_clients/new'
     assert_response :success
-    assert_in_body do
+    assert_in_heading do
       assert_select "h1", "Register a new application"
+    end
+    assert_in_body do
       assert_select "form[action='/user/test2/oauth_clients']" do
-        [ :name, :url, :callback_url, :support_url ].each do |inp|
+        [:name, :url, :callback_url, :support_url].each do |inp|
           assert_select "input[name=?]", "client_application[#{inp}]"
         end
         ClientApplication.all_permissions.each do |perm|
@@ -45,11 +50,10 @@ class ClientApplicationTest < ActionController::IntegrationTest
       end
     end
 
-    post '/user/test2/oauth_clients', {
-      'client_application[name]' => 'My New App',
-      'client_application[url]' => 'http://my.new.app.org/',
-      'client_application[callback_url]' => 'http://my.new.app.org/callback',
-      'client_application[support_url]' => 'http://my.new.app.org/support'}
+    post '/user/test2/oauth_clients',       'client_application[name]' => 'My New App',
+                                            'client_application[url]' => 'http://my.new.app.org/',
+                                            'client_application[callback_url]' => 'http://my.new.app.org/callback',
+                                            'client_application[support_url]' => 'http://my.new.app.org/support'
     assert_response :redirect
     follow_redirect!
     assert_response :success
@@ -73,14 +77,17 @@ class ClientApplicationTest < ActionController::IntegrationTest
 
   ##
   # utility method to make the HTML screening easier to read.
-  def assert_in_body
-    assert_select "html:root" do
-      assert_select "body" do
-        assert_select "div#content" do
-          yield
-        end
-      end
+  def assert_in_heading
+    assert_select "div.content-heading" do
+      yield
     end
   end
 
+  ##
+  # utility method to make the HTML screening easier to read.
+  def assert_in_body
+    assert_select "div#content" do
+      yield
+    end
+  end
 end