Remove skip() in tests & Fix test

+ 失敗していたテストのスキップを外した
+ test/test_helper.rb のmock作成時に uid を user.id
  にしていた箇所を user.uid に変更することで，ほとんどの
  テストが成功するようになった
+ 上記変更後も，should_destroy_user は失敗する
+ 以前の議論により，ユーザは基本的に削除しない方針になった
  ため，should_destroy_user を含めたユーザ削除に関する
  テストを削除した．

Author: Wistlap

test/controllers/tasks_controller_test.rb

@@ -56,7 +56,6 @@ class TasksControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "should update task" do
-    skip ''
     log_in_as(@user)
     patch task_url(@task), params: { task: { content: @task.content, creator_id: @task.creator_id, task_state_id: @task.task_state_id } }
     assert_redirected_to tasks_url

test/controllers/users_controller_test.rb

@@ -24,7 +24,6 @@ class UsersControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "should get edit" do
-    skip ''
     log_in_as(@user)
     get edit_user_url(@user)
     assert_response :success
@@ -36,7 +35,6 @@ class UsersControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "should update user" do
-    skip ''
     log_in_as(@user)
     patch user_url(@user), params: { user: { name: @user.name } }
     assert_redirected_to user_url(@user)
@@ -46,19 +44,4 @@ class UsersControllerTest < ActionDispatch::IntegrationTest
     patch user_url(@user), params: { user: { name: @user.name } }
     assert_redirected_to welcome_path
   end
-
-  test "should destroy user" do
-    skip ''
-    log_in_as(@user)
-    assert_difference('User.count', -1) do
-      delete user_url(@user)
-    end
-
-    assert_redirected_to projects_url
-  end
-
-  test "should redirect destroy to login" do
-    delete user_url(@user)
-    assert_redirected_to welcome_path
-  end
 end

test/test_helper.rb

@@ -19,7 +19,7 @@ def log_in_as(user)
     OmniAuth.config.mock_auth[:github] =
       OmniAuth::AuthHash.new({
                                :provider => 'github',
-                               :uid => user.id,
+                               :uid => user.uid,
                                :credentials => {
                                  :token => 'mock_token',
                                  :active_member => 'mock_active_member'