Merge branch 'simplify-access-status' into 'master'

Simplify the GitAccessStatus class

Make the optional 'message' argument required. Remove unused 'to_json'
method.

See merge request !65

lib/gitlab_access_status.rb

 require 'json'
 
 class GitAccessStatus
-  attr_accessor :status, :message
-  alias_method :allowed?, :status
+  attr_reader :message
 
-  def initialize(status, message = '')
+  def initialize(status, message)
     @status = status
     @message = message
   end
@@ -14,7 +13,7 @@ class GitAccessStatus
     self.new(values["status"], values["message"])
   end
 
-  def to_json
-    { status: @status, message: @message }.to_json
+  def allowed?
+    @status
   end
 end

spec/gitlab_access_spec.rb

@@ -7,7 +7,7 @@ describe GitlabAccess do
   let(:repo_path)  { File.join(repository_path, repo_name) + ".git" }
   let(:api) do
     double(GitlabNet).tap do |api|
-      api.stub(check_access: GitAccessStatus.new(true))
+      api.stub(check_access: GitAccessStatus.new(true, 'ok'))
     end
   end
   subject do
@@ -38,7 +38,7 @@ describe GitlabAccess do
     context "access is denied" do
 
       before do
-        api.stub(check_access: GitAccessStatus.new(false))
+        api.stub(check_access: GitAccessStatus.new(false, 'denied'))
       end
 
       it "returns false" do

spec/gitlab_shell_spec.rb

@@ -22,7 +22,7 @@ describe GitlabShell do
   let(:api) do
     double(GitlabNet).tap do |api|
       api.stub(discover: { 'name' => 'John Doe' })
-      api.stub(check_access: GitAccessStatus.new(true))
+      api.stub(check_access: GitAccessStatus.new(true, 'ok'))
     end
   end
 
@@ -230,7 +230,7 @@ describe GitlabShell do
     end
 
     it "should disallow access and log the attempt if check_access returns false status" do
-      api.stub(check_access: GitAccessStatus.new(false))
+      api.stub(check_access: GitAccessStatus.new(false, 'denied'))
       message = "gitlab-shell: Access denied for git command <git-upload-pack gitlab-ci.git> "
       message << "by user with key #{key_id}."
       $logger.should_receive(:warn).with(message)