enums without .new

lauth/lib/lauth/determination/allowed.rb

@@ -4,7 +4,7 @@ class Allowed < Base
 
       ALLOWED = "allowed"
 
-      def type
+      def self.type
         ALLOWED
       end
 

lauth/lib/lauth/determination/base.rb

@@ -2,27 +2,33 @@ module Lauth
   module Determination
     class Base
 
-      def type
-        raise NotImplementedError
-      end
+      class << self
+        def type
+          raise NotImplementedError
+        end
 
-      def eql?(other)
-        type == other.to_s
-      end
-      alias == eql?
+        def eql?(other)
+          type == other.to_s
+        end
+        alias == eql?
 
-      def to_s
-        to_str
-      end
+        def to_s
+          to_str
+        end
 
-      def to_str
-        type
-      end
+        def to_str
+          type
+        end
 
-      def inspect
-        to_s
+        def inspect
+          to_s
+        end
       end
 
+      private
+
+      def new; end
+
     end
   end
 end

lauth/lib/lauth/determination/denied.rb

@@ -4,7 +4,7 @@ class Denied < Base
 
       DENIED = "denied"
 
-      def type
+      def self.type
         DENIED
       end
 

lauth/lib/lauth/ops/authorize.rb

@@ -14,9 +14,9 @@ def self.call(request:)
 
       def call
         determination = if grant_repo.for_user_and_uri(request.user, request.uri).any?
-          Determination::Allowed.new
+          Determination::Allowed
         else
-          Determination::Denied.new
+          Determination::Denied
         end
         Lauth::Access::Result.new(determination:)
       end

lauth/spec/determination_spec.rb

@@ -9,10 +9,6 @@
       expect(determination).to eq string.to_sym
     end
 
-    it "equals itself" do
-      expect(determination).to eq determination.class.new
-    end
-
     it "prints as a string" do
       expect(determination.to_s).to eql string
     end
@@ -24,12 +20,12 @@
   end
 
   describe Lauth::Determination::Allowed do
-    let(:determination) { Lauth::Determination::Allowed.new }
+    let(:determination) { Lauth::Determination::Allowed}
     include_examples "a determination", "allowed"
   end
 
   describe Lauth::Determination::Denied do
-    let(:determination) { Lauth::Determination::Denied.new }
+    let(:determination) { Lauth::Determination::Denied}
     include_examples "a determination", "denied"
   end
 

lauth/spec/ops/authorize_spec.rb

@@ -20,7 +20,7 @@
 
       result = Lauth::Ops::Authorize.call(request: request)
 
-      expect(result.determination).to eq Lauth::Determination::Denied.new
+      expect(result.determination).to eq Lauth::Determination::Denied
     end
   end
 
@@ -38,7 +38,7 @@
 
       result = Lauth::Ops::Authorize.new(request: request).call
 
-      expect(result.determination).to eq Lauth::Determination::Allowed.new
+      expect(result.determination).to eq Lauth::Determination::Allowed
     end
   end
 end

lauth/spec/requests/authorized_spec.rb

@@ -18,7 +18,7 @@
       get "/authorized", {user: "lauth-allowed", uri: "/restricted-by-username/"}
       body = JSON.parse(last_response.body, symbolize_names: true)
 
-      expect(body).to eq({determination: Lauth::Determination::Allowed.new})
+      expect(body).to eq({determination: Lauth::Determination::Allowed})
     end
   end
 end