validates_timeliness 7.0.0.beta1 → 7.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 284d18473a0007c273fc4608579b619afac6a3809dfdcd12864d6aeea30c6f17
-  data.tar.gz: 12d2db9663bb6c80a643d7c7365fc4d44218a64a9b69feacb6d060dc737ddd93
+  metadata.gz: 506a3b1b6bf8487da01734e154c8bbd8c4afd5ab308dc11b4928b26c30d1a658
+  data.tar.gz: 3c380ad2708dc2946a274c473451bb1db6c1d71d0e5da462b658dad735cc9560
 SHA512:
-  metadata.gz: 77eb167f597fe224c293b6a9e9143c0b5eb8b443ed54539dc89e34c28e5cb7c9517e826c25660641281d6b477451cb2ff4b1187f0332e33fd0d94d7618223204
-  data.tar.gz: de5c7cb989b1148acf4a7af524171c7eead44663b43088da99f79bb5af7c7b1c120f28ace62cb62bcfe533979fbe9975763e59aa463acccf7ed1a951496e1f39
+  metadata.gz: 2ef5523111dee8a055989d0fa7c751d6dc34e959e2442ef99b0b46050097a31983a350ab4d6a1d3bc00ac4ebc4d00320622c9cc7aaad799535bd6dcb2084849f
+  data.tar.gz: b15a9e6cd84b59830b45a3bce59d9cc664b2a0a5504423a355cea340614ae2a136a41ee37fc180f00b70398cd50deb67d20a1459165945fccc28eb2d79ee73e2

data/.github/workflows/ci.yml

@@ -14,6 +14,22 @@ jobs:
           - gemfile: rails_7_0
             ruby: 3.1
 
+          - gemfile: rails_7_1
+            ruby: 2.7
+          - gemfile: rails_7_1
+            ruby: 3.0
+          - gemfile: rails_7_1
+            ruby: 3.1
+          - gemfile: rails_7_1
+            ruby: 3.2
+
+          - gemfile: rails_7_2
+            ruby: 3.1
+          - gemfile: rails_7_2
+            ruby: 3.2
+          - gemfile: rails_7_2
+            ruby: 3.3
+
     name: ${{ matrix.gemfile }}, ruby ${{ matrix.ruby }}
     env:
       BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile

data/README.md

@@ -30,7 +30,7 @@ Older Rails versions:
 
 In Gemfile
 ```ruby
-gem 'validates_timeliness', '~> 7.0.0.beta1'
+gem 'validates_timeliness', '~> 7.0.0'
 ```
 
 Run bundler:

data/gemfiles/rails_7_0.gemfile

@@ -1,13 +1,13 @@
 # This file was generated by Appraisal
 
-source "http://rubygems.org"
+source "https://rubygems.org"
 
 gem "rails", "~> 7.0.0"
 gem "rspec"
 gem "rspec-rails", "~> 6.0"
-gem "sqlite3"
+gem "sqlite3", "~> 1.4"
 gem "byebug"
 gem "appraisal"
-gem "nokogiri", "~> 1.8"
+gem "nokogiri"
 
 gemspec path: "../"

data/gemfiles/rails_7_1.gemfile

@@ -0,0 +1,13 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 7.1.0"
+gem "rspec"
+gem "rspec-rails", "~> 6.0"
+gem "sqlite3", "~> 1.4"
+gem "byebug"
+gem "appraisal"
+gem "nokogiri"
+
+gemspec path: "../"

data/gemfiles/rails_7_2.gemfile

@@ -0,0 +1,13 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 7.2.0"
+gem "rspec"
+gem "rspec-rails", "~> 6.0"
+gem "sqlite3", "~> 1.4"
+gem "byebug"
+gem "appraisal"
+gem "nokogiri"
+
+gemspec path: "../"

data/lib/validates_timeliness/extensions/date_time_select.rb

@@ -1,6 +1,6 @@
 module ValidatesTimeliness
   module Extensions
-    module TimelinessDateTimeSelect
+    module DateTimeSelect
       # Intercepts the date and time select helpers to reuse the values from
       # the params rather than the parsed value. This allows invalid date/time
       # values to be redisplayed instead of blanks to aid correction by the user.

data/lib/validates_timeliness/extensions.rb

@@ -1,10 +1,11 @@
 module ValidatesTimeliness
   module Extensions
-    autoload :TimelinessDateTimeSelect, 'validates_timeliness/extensions/date_time_select'
+    autoload :DateTimeSelect, 'validates_timeliness/extensions/date_time_select'
   end
 
   def self.enable_date_time_select_extension!
-    ::ActionView::Helpers::Tags::DateSelect.send(:prepend, ValidatesTimeliness::Extensions::TimelinessDateTimeSelect)
+    require 'uri' # Do we need this? No, but the test suite fails without it.
+    ::ActionView::Helpers::Tags::DateSelect.send(:prepend, ValidatesTimeliness::Extensions::DateTimeSelect)
   end
 
   def self.enable_multiparameter_extension!

data/lib/validates_timeliness/helper_methods.rb

@@ -3,25 +3,19 @@ module ActiveModel
 
     module HelperMethods
       def validates_date(*attr_names)
-        timeliness_validation_for attr_names, :date
+        validates_with TimelinessValidator, _merge_attributes(attr_names).merge(type: :date)
       end
 
       def validates_time(*attr_names)
-        timeliness_validation_for attr_names, :time
+        validates_with TimelinessValidator, _merge_attributes(attr_names).merge(type: :time)
       end
 
       def validates_datetime(*attr_names)
-        timeliness_validation_for attr_names, :datetime
+        validates_with TimelinessValidator, _merge_attributes(attr_names).merge(type: :datetime)
       end
 
       def validates_timeliness_of(*attr_names)
-        timeliness_validation_for attr_names
-      end
-
-      def timeliness_validation_for(attr_names, type=nil)
-        options = _merge_attributes(attr_names)
-        options.update(:type => type) if type
-        validates_with TimelinessValidator, options
+        validates_with TimelinessValidator, _merge_attributes(attr_names)
       end
     end
 

data/lib/validates_timeliness/orm/active_model.rb

@@ -3,67 +3,8 @@ module ValidatesTimeliness
     module ActiveModel
       extend ActiveSupport::Concern
 
-      module ClassMethods
-        public
-
-        def define_attribute_methods(*attr_names)
-          super.tap { define_timeliness_methods }
-        end
-
-        def undefine_attribute_methods
-          super.tap { undefine_timeliness_attribute_methods }
-        end
-
-        def define_timeliness_methods(before_type_cast=false)
-          return if timeliness_validated_attributes.blank?
-          timeliness_validated_attributes.each do |attr_name|
-            define_attribute_timeliness_methods(attr_name, before_type_cast)
-          end
-        end
-
-        def generated_timeliness_methods
-          @generated_timeliness_methods ||= Module.new { |m|
-            extend Mutex_m
-          }.tap { |mod| include mod }
-        end
-
-        def undefine_timeliness_attribute_methods
-          generated_timeliness_methods.module_eval do
-            instance_methods.each { |m| undef_method(m) }
-          end
-        end
-
-        def define_attribute_timeliness_methods(attr_name, before_type_cast=false)
-          define_timeliness_write_method(attr_name)
-          define_timeliness_before_type_cast_method(attr_name) if before_type_cast
-        end
-
-        def define_timeliness_write_method(attr_name)
-          generated_timeliness_methods.module_eval <<-STR, __FILE__, __LINE__ + 1
-            def #{attr_name}=(value)
-              @timeliness_cache ||= {}
-              @timeliness_cache['#{attr_name}'] = value
-
-              @attributes['#{attr_name}'] = super
-            end
-          STR
-        end
-
-        def define_timeliness_before_type_cast_method(attr_name)
-          generated_timeliness_methods.module_eval <<-STR, __FILE__, __LINE__ + 1
-            def #{attr_name}_before_type_cast
-              read_timeliness_attribute_before_type_cast('#{attr_name}')
-            end
-          STR
-        end
-      end
-
       def read_timeliness_attribute_before_type_cast(attr_name)
-        @timeliness_cache && @timeliness_cache[attr_name] || @attributes[attr_name]
-      end
-
-      def _clear_timeliness_cache
-        @timeliness_cache = {}
+        @attributes[attr_name].value_before_type_cast
       end
 
     end

data/lib/validates_timeliness/validator.rb

@@ -95,8 +95,9 @@ module ValidatesTimeliness
     end
 
     def attribute_raw_value(record, attr_name)
-      record.respond_to?(:read_timeliness_attribute_before_type_cast) &&
+      if record.respond_to?(:read_timeliness_attribute_before_type_cast)
         record.read_timeliness_attribute_before_type_cast(attr_name.to_s)
+      end
     end
 
     def time_zone_aware?(record, attr_name)

data/lib/validates_timeliness/version.rb

@@ -1,3 +1,3 @@
 module ValidatesTimeliness
-  VERSION = '7.0.0.beta1'
+  VERSION = '7.0.0'
 end

data/spec/spec_helper.rb

@@ -33,8 +33,6 @@ I18n.available_locales = ['en', 'es']
 # Extend TestModel as you would another ORM/ODM module
 module TestModelShim
   extend ActiveSupport::Concern
-  include ValidatesTimeliness::AttributeMethods
-  include ValidatesTimeliness::ORM::ActiveModel
 
   module ClassMethods
     # Hook into native time zone handling check, if any
@@ -46,11 +44,10 @@ end
 
 class Person
   include TestModel
+
   attribute :birth_date, :date
   attribute :birth_time, :time
   attribute :birth_datetime, :datetime
-
-  define_attribute_methods model_attributes.keys
 end
 
 class PersonWithShim < Person

data/spec/support/model_helpers.rb

@@ -1,22 +1,29 @@
 module ModelHelpers 
 
   # Some test helpers from Rails source 
-  def invalid!(attr_name, values, error = nil)
-    with_each_person_value(attr_name, values) do |record, value|
-      expect(record).to be_invalid
+  def invalid!(attr_name, values, error = nil, model_class = Person)
+    with_each_model_value(attr_name, values, model_class) do |record, value|
+      expect(record).to_not be_valid
       expect(record.errors[attr_name].size).to be >= 1
-      expect(record.errors[attr_name].first).to eq(error) if error
+
+      return unless error
+
+      if error.is_a?(Regexp)
+        expect(record.errors[attr_name].first).to match(error)
+      else
+        expect(record.errors[attr_name].first).to eq(error)
+      end
     end
   end
 
-  def valid!(attr_name, values)
-    with_each_person_value(attr_name, values) do |record, value|
+  def valid!(attr_name, values, model_class = Person)
+    with_each_model_value(attr_name, values, model_class) do |record, value|
       expect(record).to be_valid
     end
   end
 
-  def with_each_person_value(attr_name, values)
-    record = Person.new
+  def with_each_model_value(attr_name, values, model_class)
+    record = model_class.new
     Array.wrap(values).each do |value|
       record.send("#{attr_name}=", value)
       yield record, value

data/spec/support/test_model.rb

@@ -1,61 +1,13 @@
 module TestModel
   extend ActiveSupport::Concern
-  extend ActiveModel::Translation
+  include ActiveModel::Model
+  include ActiveModel::Attributes
   include ActiveModel::Validations
-  include ActiveModel::AttributeMethods
+  include ValidatesTimeliness::AttributeMethods
+  include ValidatesTimeliness::ORM::ActiveModel
 
   included do
     attribute_method_suffix "="
-    cattr_accessor :model_attributes
-  end
-
-  module ClassMethods
-    def attribute(name, type)
-      self.model_attributes ||= {}
-      self.model_attributes[name] = type
-    end
-
-    def define_method_attribute=(attr_name, owner: nil)
-      generated_attribute_methods.module_eval("def #{attr_name}=(new_value); @attributes['#{attr_name}']=self.class.type_cast('#{attr_name}', new_value); end", __FILE__, __LINE__)
-    end
-
-    def define_method_attribute(attr_name, owner: nil)
-      generated_attribute_methods.module_eval("def #{attr_name}; @attributes['#{attr_name}']; end", __FILE__, __LINE__)
-    end
-
-    def type_cast(attr_name, value)
-      return value unless value.is_a?(String)
-      type_name = model_attributes[attr_name.to_sym]
-      type = ActiveModel::Type.lookup(type_name)
-      type.cast(value)
-    end
-  end
-
-  def initialize(attributes = nil)
-    @attributes = self.class.model_attributes.keys.inject({}) do |hash, column|
-      hash[column.to_s] = nil
-      hash
-    end
-    self.attributes = attributes unless attributes.nil?
-  end
-
-  def attributes
-    @attributes
-  end
-
-  def attributes=(new_attributes={})
-    new_attributes.each do |key, value|
-      send "#{key}=", value
-    end
-  end
-
-  def method_missing(method_id, *args, &block)
-    if !matched_attribute_method(method_id.to_s).nil?
-      self.class.define_attribute_methods self.class.model_attributes.keys
-      send(method_id, *args, &block)
-    else
-      super
-    end
   end
 end
 

data/spec/validates_timeliness/attribute_methods_spec.rb

@@ -44,9 +44,11 @@ RSpec.describe ValidatesTimeliness::AttributeMethods do
       class PersonWithParser
         include TestModel
         include TestModelShim
+
         attribute :birth_date, :date
         attribute :birth_time, :time
         attribute :birth_datetime, :datetime
+
         validates_date :birth_date
         validates_time :birth_time
         validates_datetime :birth_datetime
@@ -54,8 +56,10 @@ RSpec.describe ValidatesTimeliness::AttributeMethods do
 
       it 'should parse a string value' do
         expect(Timeliness::Parser).to receive(:parse)
+
         r = PersonWithParser.new
         r.birth_date = '2010-01-01'
+        r.birth_date # parsing happens on read not write
       end
 
     end

data/spec/validates_timeliness/validator/format_spec.rb

@@ -0,0 +1,57 @@
+RSpec.describe ValidatesTimeliness::Validator, ":format option" do
+  with_config(:use_plugin_parser, true)
+
+  describe "for date type" do
+    before do
+      Person.validates_date :birth_date, format: "yyyy-mm-dd"
+    end
+
+    it "should not be valid for string given in the wrong format" do
+      invalid!(:birth_date, '23/12/2023', /is not a valid date/)
+    end
+
+    it "should be valid for string given in the right format" do
+      valid!(:birth_date, '2023-12-23')
+    end
+
+    it "should be valid for date instance" do
+      valid!(:birth_date, Date.new(2022,12,23))
+    end
+  end
+
+  describe "for time type" do
+    before do
+      Person.validates_time :birth_time, format: "hh:nn:ss"
+    end
+
+    it "should not be valid for string given in the wrong format" do
+      invalid!(:birth_time, "00-00-00", /is not a valid time/)
+    end
+
+    it "should be valid for string given in the right format" do
+      valid!(:birth_time, "00:00:00")
+    end
+
+    it "should be valid for date instance" do
+      valid!(:birth_time, Time.new(2010, 1, 1, 0, 0, 0))
+    end
+  end
+
+  describe "for datetime type" do
+    before do
+      Person.validates_datetime :birth_datetime, format: "yyyy-mm-dd hh:nn:ss"
+    end
+
+    it "should not be valid for string given in the wrong format" do
+      invalid!(:birth_datetime, "01-01-2010 00-00-00", /is not a valid datetime/)
+    end
+
+    it "should be valid for string given in the right format" do
+      valid!(:birth_datetime, "2010-01-01 00:00:00")
+    end
+
+    it "should be valid for date instance" do
+      valid!(:birth_datetime, DateTime.new(2010, 1, 1, 0, 0, 0))
+    end
+  end
+end

data/validates_timeliness.gemspec

@@ -21,6 +21,7 @@ Gem::Specification.new do |s|
   s.extra_rdoc_files = ["README.md", "CHANGELOG.md", "LICENSE"]
 
   s.metadata = {
+    "rubygems_mfa_required" => "true",
     "bug_tracker_uri" => "#{github_url}/issues",
     "changelog_uri"   => "#{github_url}/blob/master/CHANGELOG.md",
     "source_code_uri" => "#{github_url}",

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: validates_timeliness
 version: !ruby/object:Gem::Version
-  version: 7.0.0.beta1
+  version: 7.0.0
 platform: ruby
 authors:
 - Adam Meehan
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-11-21 00:00:00.000000000 Z
+date: 2024-11-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -67,7 +67,8 @@ files:
 - README.md
 - Rakefile
 - gemfiles/rails_7_0.gemfile
-- gemfiles/rails_edge.gemfile
+- gemfiles/rails_7_1.gemfile
+- gemfiles/rails_7_2.gemfile
 - init.rb
 - lib/generators/validates_timeliness/install_generator.rb
 - lib/generators/validates_timeliness/templates/en.yml
@@ -99,6 +100,7 @@ files:
 - spec/validates_timeliness/railtie_spec.rb
 - spec/validates_timeliness/validator/after_spec.rb
 - spec/validates_timeliness/validator/before_spec.rb
+- spec/validates_timeliness/validator/format_spec.rb
 - spec/validates_timeliness/validator/is_at_spec.rb
 - spec/validates_timeliness/validator/on_or_after_spec.rb
 - spec/validates_timeliness/validator/on_or_before_spec.rb
@@ -109,6 +111,7 @@ homepage: https://github.com/adzap/validates_timeliness
 licenses:
 - MIT
 metadata:
+  rubygems_mfa_required: 'true'
   bug_tracker_uri: https://github.com/adzap/validates_timeliness/issues
   changelog_uri: https://github.com/adzap/validates_timeliness/blob/master/CHANGELOG.md
   source_code_uri: https://github.com/adzap/validates_timeliness
@@ -124,11 +127,11 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.3.27
 signing_key:
 specification_version: 4
 summary: Date and time validation plugin for Rails which allows custom formats
@@ -148,6 +151,7 @@ test_files:
 - spec/validates_timeliness/railtie_spec.rb
 - spec/validates_timeliness/validator/after_spec.rb
 - spec/validates_timeliness/validator/before_spec.rb
+- spec/validates_timeliness/validator/format_spec.rb
 - spec/validates_timeliness/validator/is_at_spec.rb
 - spec/validates_timeliness/validator/on_or_after_spec.rb
 - spec/validates_timeliness/validator/on_or_before_spec.rb

data/gemfiles/rails_edge.gemfile

@@ -1,13 +0,0 @@
-# This file was generated by Appraisal
-
-source "http://rubygems.org"
-
-gem "rails", git: "https://github.com/rails/rails.git", branch: "main"
-gem "rspec"
-gem "rspec-rails", "~> 3.7"
-gem "sqlite3"
-gem "byebug"
-gem "appraisal"
-gem "nokogiri", "~> 1.8"
-
-gemspec path: "../"