enumbler 0.8.0 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a3b8d2a2e38789cb62f2be5c895aa0fb2776b26b8efbbfe3386e88feaf041393
-  data.tar.gz: d981a2476760ab1d0afb09bd2eb790c125cf54ba54cad3493a5865f52163648b
+  metadata.gz: ca8de5b8e3b88d4321e0df6108f41245142f95489930e57e6d0209c260335b3d
+  data.tar.gz: ab12a41dc3564dd62f78cb0583ec12ac6602ba5faf6bf890be80864096f4a65d
 SHA512:
-  metadata.gz: 65c1b774b7fd7fe86e3038c5bfea649608708e57ef626529c933e82e31249a43716b50e54d2a822640d11d6b98321832f032dfedf2efce9a750aebc1e25fd26c
-  data.tar.gz: c223ff2703dd5693ee9925dd4ffc025c890138962001acca6619f8c1d30b7e6056a5275fa84f5c159f163c052b8fbc546c1f9132917b93a96956c6aba9d1e72e
+  metadata.gz: 948e3cab58f8065ad78b67680ecc9ca0f3de53be0169ca13502a69f145af27e855707b331cf760e925859789f26f68a801ee7f6c02562d393ca6a720c30497fd
+  data.tar.gz: c50f6dc4282fa0ace1cdc44b97dd79f64f6a1667074bd4fe93f5ed580decfa55983b412317d3c3a4f18f83efa1ef83c16fad75177f0e575ccf0ba005f6275cf1

data/.rubocop.yml

@@ -1,7 +1,6 @@
 AllCops:
-  TargetRubyVersion: 2.6
-  Exclude:
-    - 'bin/pre_commit_rubocop.rb'
+  NewCops: enable
+  TargetRubyVersion: 2.5
 
 Layout/ArgumentAlignment:
   Enabled: true
@@ -11,16 +10,36 @@ Layout/FirstHashElementIndentation:
   Enabled: true
   EnforcedStyle: consistent
 
+Layout/HashAlignment:
+  Enabled: true
+  # This allows "implicit" hash arguments (without curly braces) to be aligned
+  # differently.
+  EnforcedLastArgumentHashStyle: ignore_implicit
+
 Layout/LineLength:
   AllowURI: true
   Max: 119
 
+Layout/MultilineMethodCallIndentation:
+  Enabled: true
+  EnforcedStyle: indented
+
 Lint/RaiseException:
   Enabled: true
 
 Lint/StructNewOverride:
   Enabled: true
 
+Metrics:
+  Enabled: false
+
+Naming/MethodParameterName:
+  Enabled: true
+  AllowedNames: [at, by, db, id, in, ip, of, on, to, tz]
+
+Naming/PredicateName:
+  Enabled: true
+
 Style/Documentation:
   Enabled: false
 
@@ -33,6 +52,9 @@ Style/HashTransformKeys:
 Style/HashTransformValues:
   Enabled: true
 
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
+
 Style/TrailingCommaInHashLiteral:
   Enabled: true
   EnforcedStyleForMultiline: consistent_comma
@@ -41,5 +63,5 @@ Style/TrailingCommaInArrayLiteral:
   Enabled: true
   EnforcedStyleForMultiline: consistent_comma
 
-Metrics:
-  Enabled: false
+Style/TrailingCommaInArguments:
+  EnforcedStyleForMultiline: consistent_comma

data/Gemfile

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in enumbler.gemspec
 gemspec

data/Gemfile.lock

@@ -1,9 +1,9 @@
 PATH
   remote: .
   specs:
-    enumbler (0.8.0)
-      activerecord (>= 5.2.3, < 6.1)
-      activesupport (>= 5.2.3, < 6.1)
+    enumbler (0.9.1)
+      activerecord (>= 5.2.3, < 7)
+      activesupport (>= 5.2.3, < 7)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -170,14 +170,12 @@ To install this gem onto your local machine, run `bundle exec rake install`. To
 
 ## Roadmap
 
-* We need to add in support for additional attributes/columns in the enumbled table.  For example, following the `Color` concept, we may want to have a column which is `hex` and stores the colors `hex` value (e.g., `FFFFFF`).  This should be supported.
-* Ideally, we could make this work more like a traditional `enum`; for example, overriding the `.where` method by allowing something like: `House.where(color: :blue)` instead of `House.where_color(:blue)`.  But right now am in a rush and not sure how to go about doing that properly.
+* Ideally, we could make this work more like a traditional `enum`; for example, overriding the `.where` method by allowing something like: `House.where(color: :blue)` instead of `House.color(:blue)`.  But right now am in a rush and not sure how to go about doing that properly.
 
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/linguabee/enumbler.
 
-
 ## License
 
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/Rakefile

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'bundler/gem_tasks'
-require 'rspec/core/rake_task'
+require "bundler/gem_tasks"
+require "rspec/core/rake_task"
 
 RSpec::Core::RakeTask.new(:spec)
 

data/bin/console

@@ -1,8 +1,8 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
-require 'bundler/setup'
-require 'enumbler'
+require "bundler/setup"
+require "enumbler"
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.
@@ -11,5 +11,5 @@ require 'enumbler'
 # require "pry"
 # Pry.start
 
-require 'irb'
+require "irb"
 IRB.start(__FILE__)

data/bin/pre_commit_rubocop.rb

@@ -1,8 +1,10 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
+
 # Put this file into your path and use `<file> install` to add a new hook
 # or use it as a binary to check changed files
 
-require 'shellwords'
+require "shellwords"
 
 if ARGV == ["install"]
   exec "ln", "-sf", File.expand_path(File.basename(__FILE__), __dir__), ".git/hooks/pre-commit"
@@ -10,15 +12,23 @@ else
   raise unless ARGV == []
 end
 
-changed = `git status --porcelain`.
-  split("\n").
-  map { |l| l.split(" ", 2) }.
-  select { |status, _| ["A", "AM", "M"].include?(status) }.
-  map { |_, file| file.delete('"') }
+changed = `git status --porcelain`
+  .split("\n")
+  .map { |l| l.split(" ", 2) }
+  .select { |status, _| %w[A AM M].include?(status) }
+  .map { |_, file| file.delete('"') }
 
 exit if changed.empty?
 
-parallel = ((File.read(".rubocop.yml").include?("UseCache: false") rescue false) ? "" : " --parallel")
+parallel = (if begin
+  File.read(".rubocop.yml").include?("UseCache: false")
+rescue StandardError
+  false
+end
+              ""
+            else
+              " --parallel"
+            end)
 result = `bundle exec rubocop #{parallel} --color --force-exclusion #{changed.shelljoin}`
-puts result unless $?.success?
-exit $?.exitstatus
+puts result unless $CHILD_STATUS.success?
+exit $CHILD_STATUS.exitstatus

data/enumbler.gemspec

@@ -1,42 +1,42 @@
 # frozen_string_literal: true
 
-require_relative 'lib/enumbler/version'
+require_relative "lib/enumbler/version"
 
 Gem::Specification.new do |spec|
-  spec.name          = 'enumbler'
+  spec.name          = "enumbler"
   spec.version       = Enumbler::VERSION
-  spec.authors       = ['Damon Timm']
-  spec.email         = ['damon@linguabee.com']
+  spec.authors       = ["Damon Timm"]
+  spec.email         = ["damon@linguabee.com"]
 
   spec.summary       = "Enums are terrific, but lack integrity.  Let's add some!"
-  spec.description   = 'A more complete description is forthcoming.'
-  spec.homepage      = 'https://github.com/linguabee/enumbler'
-  spec.license       = 'MIT'
-  spec.required_ruby_version = Gem::Requirement.new('>= 2.5.0')
+  spec.description   = "A more complete description is forthcoming."
+  spec.homepage      = "https://github.com/linguabee/enumbler"
+  spec.license       = "MIT"
+  spec.required_ruby_version = Gem::Requirement.new(">= 2.5.0")
 
-  spec.metadata['allowed_push_host'] = 'https://rubygems.org'
+  spec.metadata["allowed_push_host"] = "https://rubygems.org"
 
-  spec.metadata['homepage_uri'] = spec.homepage
-  spec.metadata['source_code_uri'] = spec.homepage
-  spec.metadata['changelog_uri'] = spec.homepage
+  spec.metadata["homepage_uri"] = spec.homepage
+  spec.metadata["source_code_uri"] = spec.homepage
+  spec.metadata["changelog_uri"] = spec.homepage
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
   spec.files = Dir.chdir(File.expand_path(__dir__)) do
     `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   end
-  spec.bindir        = 'exe'
+  spec.bindir        = "exe"
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ['lib']
-
-  spec.add_dependency 'activerecord', ['>= 5.2.3', '< 6.1']
-  spec.add_dependency 'activesupport', ['>= 5.2.3', '< 6.1']
-
-  spec.add_development_dependency 'database_cleaner-active_record', '~> 1.8.0'
-  spec.add_development_dependency 'fuubar', '~> 2.5'
-  spec.add_development_dependency 'pry'
-  spec.add_development_dependency 'rake', '~> 12.0'
-  spec.add_development_dependency 'rspec', '~> 3.9.0'
-  spec.add_development_dependency 'rubocop', '~> 0.91.0'
-  spec.add_development_dependency 'sqlite3', '~> 1.4.0'
+  spec.require_paths = ["lib"]
+
+  spec.add_dependency "activerecord", [">= 5.2.3", "< 7"]
+  spec.add_dependency "activesupport", [">= 5.2.3", "< 7"]
+
+  spec.add_development_dependency "database_cleaner-active_record", "~> 1.8.0"
+  spec.add_development_dependency "fuubar", "~> 2.5"
+  spec.add_development_dependency "pry"
+  spec.add_development_dependency "rake", "~> 12.0"
+  spec.add_development_dependency "rspec", "~> 3.9.0"
+  spec.add_development_dependency "rubocop", "~> 0.91.0"
+  spec.add_development_dependency "sqlite3", "~> 1.4.0"
 end

data/lefthook.yml

@@ -0,0 +1,12 @@
+# EXAMPLE USAGE
+# Refer for explanation to following link:
+# https://github.com/Arkweid/lefthook/blob/master/docs/full_guide.md
+
+pre-commit:
+  parallel: true
+  commands:
+    rubocop:
+      tags: backend style
+      glob: "*.rb"
+      exclude: "application.rb|routes.rb"
+      run: bundle exec rubocop --force-exclusion {all_files}

data/lib/enumbler.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
-require 'enumbler/core_ext/symbol/case_equality_operator'
+require "enumbler/core_ext/symbol/case_equality_operator"
 
-require 'enumbler/collection'
-require 'enumbler/enumble'
-require 'enumbler/enabler'
-require 'enumbler/version'
+require "enumbler/collection"
+require "enumbler/enumble"
+require "enumbler/enabler"
+require "enumbler/version"
 
-require 'active_support/concern'
-require 'active_support/inflector'
+require "active_support/concern"
+require "active_support/inflector"
 
 # The Enumbler add integrity to our enum implementation!
 module Enumbler

data/lib/enumbler/collection.rb

@@ -16,7 +16,7 @@ module Enumbler
       super
     end
 
-    def respond_to_missing?(method_name, include_private = false)
+    def respond_to_missing?(method_name, *args, &block)
       enumble = find { |e| e.enum == method_name }
       enumble.present? || super
     end

data/lib/enumbler/enabler.rb

@@ -12,7 +12,7 @@ module Enumbler
     def enumble
       @enumble = self.class.find_enumble(id)
 
-      raise Error, 'An enumble is not defined for this record!' if @enumble.nil?
+      raise Error, "An enumble is not defined for this record!" if @enumble.nil?
 
       @enumble
     end
@@ -227,6 +227,8 @@ module Enumbler
             enumble =
               if arg.is_a?(Symbol)
                 @enumbled_model.enumbles.find { |e| e.enum == arg }
+              elsif arg.is_a?(Enumbler::Enumble)
+                @enumbled_model.enumbles.find { |e| e.enum == arg.enum }
               elsif arg.is_a?(String)
                 @enumbled_model.enumbles.find do |e|
                   if case_sensitive
@@ -330,7 +332,7 @@ module Enumbler
       #   longer defined (default: false)
       # @param validate [Boolean] validate on save?
       def seed_the_enumbler(delete_missing_records: false, validate: true)
-        max_database_id = all.order('id desc').take&.id || 0
+        max_database_id = all.order("id desc").take&.id || 0
         max_enumble_id = @enumbles.map(&:id).max
 
         # If we are not deleting records, we just need to update each listed
@@ -375,11 +377,21 @@ module Enumbler
         method_name = "#{enumble.enum}?"
         not_method_name = "not_#{enumble.enum}?"
         alias_method_name = "is_#{enumble.enum}"
+        any_method_name = "any_#{enumble.enum}?"
+
+        [method_name, not_method_name, alias_method_name].each do |mname|
+          detect_enumbler_conflict(enumble.enum, mname)
+        end
+
+        [enumble.enum, any_method_name].each do |mname|
+          detect_enumbler_conflict(enumble.enum, mname, klass_method: true)
+        end
 
         const_set(enumble.enum.to_s.upcase, enumble.id)
         define_method(method_name) { id == enumble.id }
         define_method(not_method_name) { id != enumble.id }
         alias_method alias_method_name, method_name
+
         define_singleton_method(enumble.enum) do |attr = nil|
           return find(enumble.id) if attr.nil?
 
@@ -388,13 +400,42 @@ module Enumbler
           raise Enumbler::Error, "The attribute #{attr} is not supported on this Enumble."
         end
 
-        define_singleton_method("any_#{enumble.enum}?") do
-          where(id: enumble.id).exists?
+        define_singleton_method(any_method_name) do
+          exists?(id: enumble.id)
         rescue NoMethodError
           raise Enumbler::Error, "The attribute #{attr} is not supported on this Enumble."
         end
       end
 
+      # This idea sourced lovingly from ActiveRecord::Enum
+      ENUMBLER_CONFLICT_MESSAGE = <<~TEXT.squish
+        You tried to define the enumble :%<enum>s on the model %<klass>s, but
+        this will generate a %<type>s method `%<method>s`, which is already defined
+        by %<source>s.
+      TEXT
+
+      def detect_enumbler_conflict(enumble_name, method_name, klass_method: false)
+        if klass_method && dangerous_class_method?(method_name)
+          raise_conflict_error(enumble_name, method_name, type: "class")
+        elsif klass_method && method_defined_within?(method_name, ActiveRecord::Relation)
+          raise_conflict_error(enumble_name, method_name, type: "class", source: ActiveRecord::Relation.name)
+        elsif !klass_method && dangerous_attribute_method?(method_name)
+          raise_conflict_error(enumble_name, method_name)
+        end
+      end
+
+      def raise_conflict_error(enumble_name, method_name, type: "instance", source: "ActiveRecord")
+        raise Error,
+          format(
+            ENUMBLER_CONFLICT_MESSAGE,
+            enum: enumble_name,
+            klass: name,
+            type: type,
+            method: method_name,
+            source: source,
+          )
+      end
+
       # I accidentally forgot to provide an id one time and it was confusing as
       # the last argument became the hash of options.  This should help.
       def validate_id_is_numeric(enum, id)
@@ -411,11 +452,17 @@ module Enumbler
 
         return if unsupported_attrs.blank?
 
+        ActiveRecord::Migration.check_pending!
+
         raise Enumbler::Error,
           "The model #{self} does not support the attribute(s): #{unsupported_attrs.keys.map(&:to_s).to_sentence}"
+      rescue ActiveRecord::PendingMigrationError
+        warn "[Enumbler Warning] => The model #{self} does not currently support the attribute(s): #{unsupported_attrs.keys.map(&:to_s).to_sentence}." \
+          " You have a pending migration which hopefully would remedy this!  If not, you need to add a migration for this attibrute or" \
+          " remove it from the Enumbler."
       rescue ActiveRecord::StatementInvalid
         warn "[Enumbler Warning] => Unable to find a table for #{self}."\
-          'This is to be expected if there is a pending migration; however, if there is not then something is amiss.'
+          "This is to be expected if there is a pending migration; however, if there is not then something is amiss."
       end
     end
   end

data/lib/enumbler/enumble.rb

@@ -11,7 +11,7 @@ module Enumbler
       @label_column_name = label_column_name
       @label = (label_col_specified? ? attributes[label_column_name] : label) || enum.to_s.dasherize
       @additional_attributes = attributes || {}
-      @additional_attributes.merge!({ label: label }) if label_col_specified?
+      @additional_attributes.merge!({ label: label }) unless label.nil?
     end
 
     def ==(other)
@@ -21,7 +21,6 @@ module Enumbler
 
     def attributes
       hash = { id: id, label_column_name => label }
-      hash.merge!({ label: @additional_attributes[:label] }) if label_col_specified?
       @additional_attributes.merge(hash)
     end
 

data/lib/enumbler/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Enumbler
-  VERSION = '0.8.0'
+  VERSION = "0.9.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: enumbler
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.9.2
 platform: ruby
 authors:
 - Damon Timm
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-10-26 00:00:00.000000000 Z
+date: 2021-05-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -19,7 +19,7 @@ dependencies:
         version: 5.2.3
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,7 +29,7 @@ dependencies:
         version: 5.2.3
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '7'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
@@ -39,7 +39,7 @@ dependencies:
         version: 5.2.3
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -49,7 +49,7 @@ dependencies:
         version: 5.2.3
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '7'
 - !ruby/object:Gem::Dependency
   name: database_cleaner-active_record
   requirement: !ruby/object:Gem::Requirement
@@ -169,6 +169,7 @@ files:
 - bin/pre_commit_rubocop.rb
 - bin/setup
 - enumbler.gemspec
+- lefthook.yml
 - lib/enumbler.rb
 - lib/enumbler/collection.rb
 - lib/enumbler/core_ext/symbol/case_equality_operator.rb
@@ -198,7 +199,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Enums are terrific, but lack integrity.  Let's add some!