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

1 files changed, 4 insertions, 5 deletions

diff --git a/lib/gitlab_access_status.rb b/lib/gitlab_access_status.rb
index 14ec1ef..7a5f7d5 100644
--- a/lib/gitlab_access_status.rb
+++ b/lib/gitlab_access_status.rb
@@ -1,10 +1,9 @@
 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