enumy 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a8f5d0cfe99364e910e316dd0a1080b7bdb0ec998b1a9005e07605a8060a491
-  data.tar.gz: a55d33d3c9b41bb26bd6392fb61f34fd2ef5466964320d4788d285080962126f
+  metadata.gz: 1c98be619a05adaaa0eb0dc1e272912eacbf374bf303d79b134fa7bcf8249ebf
+  data.tar.gz: bc317476028ef8504f2ec6aac90a7f6a240e85844f588e9a8031f9bf10a57ca6
 SHA512:
-  metadata.gz: bc82b788dbe3f917aacb120eda3e36dbcb7bb24c832e077b2379a8393f03d94e765dab0d8a16062d9c3fdad1ed181b259292a55bf592a8899372b236e2c86093
-  data.tar.gz: 453d1f643652560edafe31b52d0fc0a7092b7c034175c0549f4b6b6d87504b2187608e6e0d88cb1b83b2f86979ee8a67119326a3cebf1fa135aac5f596a6608b
+  metadata.gz: c9300102398830ab9e00f464a53920c35ab81385dfaccb90b7f00510893588c344cd91d8ab9612ad54140dfb9dc4688bac64b9b3e07a3676eb36c8d21dad2b26
+  data.tar.gz: ad8969cf282ca4921d18c69f8d40dfaf361a734b1e12a7bee43f188c5688d33576164195152d2e07c92b9b1fdad235a51315933ad6112ce986d9514490ad3ddb

data/lib/enumy/base.rb

@@ -3,13 +3,16 @@ require_relative 'errors'
 module Enumy
   class Base
     include ActiveModel::Model
+    include ActiveModel::Attributes
 
-    attr_accessor :id
-    attr_accessor :name
+    attribute :id, :integer
+    attribute :name
 
-    def initialize(id = nil, name = nil)
-      @id = id
-      @name = name
+    def initialize(id, name, **attrs)
+      raise ArgumentError, "#{self.class.name.upcase} enum requires an `id`" if id.nil?
+      raise ArgumentError, "#{self.class.name.upcase} enum requires a `name`" if name.nil?
+
+      super(id: id, name: name, **attrs)
     end
 
     def human_attribute_name(attribute)
@@ -20,7 +23,7 @@ module Enumy
 
       def register(enum)
         if instances[enum.id]
-          warn "Enum `#{enum.name.to_s.upcase}` is using a duplicate id `#{enum.id}` for Enum '#{enum.class.name}'. Ignoring the duplicate."
+          raise ArgumentError, "Enum `#{enum.name.to_s.upcase}` is using a duplicate id `#{enum.id}` for Enum '#{enum.class.name}'."
         else
           instances[enum.id] = enum
         end
@@ -31,7 +34,7 @@ module Enumy
       end
 
       def find(id)
-        find_by_id(id) || raise(Errors::EnumNotFoundError.new)
+        find_by_id(id) || raise(Errors::EnumNotFoundError.new("Could not find a `#{name}` enum with an `id` of #{id}"))
       end
 
       def find_by_id(id)
@@ -45,7 +48,7 @@ module Enumy
       end
 
       def model_name
-        @model_name ||= ActiveModel::Name.new(self.class, nil, name.to_s)
+        @model_name ||= ActiveModel::Name.new(self, nil, name.to_s)
       end
 
       def i18n_scope
@@ -61,4 +64,4 @@ module Enumy
     end
 
   end
-end
+end

data/lib/enumy/model.rb

@@ -5,21 +5,35 @@ module Enumy
   module Model
     def belongs_to_enum(name, class_name: name, default: nil)
       enum_class = class_name.to_s.classify.safe_constantize
-      raise Errors::InvalidEnumClassError if enum_class.nil? || !enum_class.ancestors.include?(Enumy::Base)
+      
+      if enum_class.nil?
+        raise Errors::InvalidEnumClassError, "Could not resolve class `#{class_name.to_s.classify}`"
+      elsif !enum_class.ancestors.include?(Enumy::Base)
+        raise Errors::InvalidEnumClassError, "`#{enum_class.name}` does not inherit from `Enumy::Base`"
+      end
 
       define_method "#{name}=" do |enum|
-        return nil unless self.has_attribute?("#{name}_id")
-        instance_variable_set("@#{name}", self.send("#{name}_id=", enum.id))
+        raise NameError, "Enum `#{enum_class.name}` expected #{self.class.name} to have attribute `#{name}_id`." unless self.has_attribute?("#{name}_id")
+        raise Errors::InvalidEnumClassError, "Expected an enum of type `#{enum_class.name}`, got `#{enum.class.name}`" if enum_class != enum.class
+        self.send("#{name}_id=", enum.id)
+        instance_variable_set("@#{name}", enum)
       end
 
       define_method "#{name}" do
         enum = instance_variable_get("@#{name}")
         return enum if enum
-        return nil unless self.has_attribute?("#{name}_id") && self.send("#{name}_id")
+
+        raise NameError, "Enum `#{enum_class.name}` expected #{self.class.name} to have attribute `#{name}_id`." unless self.has_attribute?("#{name}_id")
+        return nil unless self.send("#{name}_id")
         instance_variable_set("@#{name}", enum_class.find(self.send("#{name}_id")))
       end
 
-      after_initialize -> { self.send("#{name}=", default) if self.send("#{name}").nil? && default.present? }
+      define_method "#{name}_id=" do |new_id|
+        super(new_id)
+        instance_variable_set("@#{name}", enum_class.find(new_id))
+      end
+
+      after_initialize -> { self.send("#{name}=", default) if self.has_attribute?("#{name}_id") && self.send("#{name}").nil? && default.present? }
     end
   end
-end
+end

data/lib/enumy/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Enumy
-  VERSION = "0.1.2"
+  VERSION = "0.1.3"
 end

data/lib/generators/enumy/install/templates/app/enums/application_enum.rb

@@ -2,7 +2,7 @@ class ApplicationEnum < Enumy::Base
   include ActiveModel::Attributes
 
   attribute :id, :integer
-  attribute :name, :symbol
+  attribute :name
 
   def initialize(id, name, **attrs)
     super(id: id, name: name, **attrs)

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: enumy
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Bevan Holborn
 bindir: bin
 cert_chain: []
-date: 2025-01-14 00:00:00.000000000 Z
+date: 2025-01-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake