Merge pull request #277 from codetriage/user-should-be-logged-in-befo…

…re-creating-repo

User should be logged in before creating repo

Gemfile.lock

@@ -76,7 +76,7 @@ GEM
     dotenv-rails (0.10.0)
       dotenv (= 0.10.0)
     erubis (2.7.0)
-    excon (0.38.0)
+    excon (0.41.0)
     execjs (2.0.2)
     faraday (0.9.0)
       multipart-post (>= 1.2, < 3)
@@ -246,7 +246,7 @@ GEM
       rack (>= 1.0.0)
     warden (1.2.3)
       rack (>= 1.0)
-    webmock (1.13.0)
+    webmock (1.20.4)
       addressable (>= 2.2.7)
       crack (>= 0.3.2)
     websocket-driver (0.3.2)

app/controllers/repos_controller.rb

@@ -2,6 +2,8 @@
 
 class ReposController < RepoBasedController
 
+  before_filter :authenticate_user!, only: [:new, :create, :edit, :update]
+
   def index
     @repos = Repo.order_by_subscribers.order(:name).page(params[:page]).per_page(params[:per_page] || 50)
   end

test/functional/repos_controller_test.rb

@@ -10,4 +10,24 @@ class ReposControllerTest < ActionController::TestCase
       get :show, full_name: 'foo/bar'
     }
   end
+
+  test 'trying to create repo without logged in will redirect to login page' do
+    assert_no_difference -> { Repo.count } do
+      post :create, repo: { name: 'codetriage', user_name: 'codetriage' }
+    end
+
+    assert_redirected_to new_user_session_path
+  end
+
+  test 'logged in user can create repo' do
+    sign_in users(:mockstar)
+
+    VCR.use_cassette "create_repo_refinery", record: :once do
+      assert_difference -> { Repo.count } do
+        post :create, repo: { name: 'refinerycms', user_name: 'refinery' }
+      end
+    end
+
+    assert_redirected_to repo_path('refinery/refinerycms')
+  end
 end