temping 3.9.0 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 854682e673db78262df86716d1f51432af84aa72
-  data.tar.gz: f8e95dc7625c179eecf3a9891e01bfac673205ab
+SHA256:
+  metadata.gz: 17a3e61694c280ee41ecaba6d9ba29f7815dd0ae7126bb931872819669ee03e7
+  data.tar.gz: ce7e225a2e34ed393ca11fe8d5cb1a5695806142282d940339fbf991f24a45b9
 SHA512:
-  metadata.gz: 08cbb126b85f7452c3a986edf1679a99a18b4b8dfcd9b9e8af874208dea32ec3044bcd83239840ce0a88bfe323388a31b63c031248f15c1be0925fb97144e210
-  data.tar.gz: 65e0a023f58fcc3baae537f1c073565d74844bdcb192559ad41077b7f13d83dd9a1247031c123d331a9eaf44ce6d126d0e9f3ec0a43c56ab1377f0b77bfa19dd
+  metadata.gz: 9aaf7aee2a796eb3a19df188097c548619665b1d37923d86a702657a28d9f0339daa0e48d62dfd2b2b27b31fe5a27704ff3103b6c342ecd3eaeacaadc3c46906
+  data.tar.gz: f0406caf8f32fcf950a5328eae4f46bb46573d50e0b1e58247fdeb0622552471c1a6ed0faa047b0535e35674fd79fd1b8d6ea0a73a200cf158fb4b3dfc7c4675

data/lib/temping/model_factory.rb

@@ -0,0 +1,69 @@
+class Temping::ModelFactory
+  DEFAULT_OPTIONS = {temporary: true}
+
+  def initialize(name, namespace, options = {}, &block)
+    @name = name
+    @namespace = namespace
+    @options = options
+    klass.class_eval(&block) if block
+    klass.reset_column_information
+  end
+
+  def klass
+    @klass ||= @namespace.const_get(@name)
+  rescue NameError
+    @klass = build
+  end
+
+  private
+
+  def build
+    Class.new(parent_class_name).tap do |klass|
+      @namespace.const_set(@name, klass)
+      klass.primary_key = @options[:primary_key] || :id
+      create_table(@options)
+      add_methods
+      klass.namespace = @namespace
+    end
+  end
+
+  def parent_class_name
+    @options.fetch(:parent_class, default_parent_class_name)
+  end
+
+  def default_parent_class_name
+    if defined?(ApplicationRecord)
+      ApplicationRecord
+    else
+      ActiveRecord::Base
+    end
+  end
+
+  def create_table(options = {})
+    connection.create_table(table_name, **DEFAULT_OPTIONS.merge(options))
+  end
+
+  def add_methods
+    class << klass
+      attr_accessor :namespace
+
+      def with_columns
+        connection.change_table(table_name) do |table|
+          yield(table)
+        end
+      end
+
+      def table_exists?
+        true
+      end
+    end
+  end
+
+  def connection
+    klass.connection
+  end
+
+  def table_name
+    klass.table_name
+  end
+end

data/lib/temping/namespace_factory.rb

@@ -0,0 +1,24 @@
+class Temping::NamespaceFactory
+  def initialize(name)
+    @name = name
+  end
+
+  def klass
+    @klass ||= @name.split("::").reduce(Object) { |parent, name_part| build(parent, name_part) }
+  end
+
+  private
+
+  def build(parent, name_part)
+    parent.const_get(name_part)
+  rescue NameError
+    parent.const_set(
+      name_part,
+      Module.new do
+        def self.defined_by_temping?
+          true
+        end
+      end
+    )
+  end
+end

data/lib/temping.rb

@@ -1,94 +1,149 @@
 require "active_record"
 require "active_support/core_ext/string"
 
+class Temping; end
+
+require "temping/namespace_factory"
+require "temping/model_factory"
+
 class Temping
-  @model_klasses = []
+  @namespaces = []
+  @models = []
 
   class << self
-    def create(model_name, options = {}, &block)
-      factory = ModelFactory.new(model_name.to_s.classify, options, &block)
-      klass = factory.klass
-      @model_klasses << klass
-      klass
+    # Create a new temporary ActiveRecord model with a name specified by `name`.
+    #
+    # Provided `options` are all passed to the inner `create_table` call so anything
+    # acceptable by `create_table` method can be passed here.
+    # In addition `options` can include `parent_class` key to specify parent class for the model.
+    # When `block` is passed, it is evaluated in the context of the class. This means anything you
+    # do in an ActiveRecord model class body can be accomplished in `block` including method
+    # definitions, validations, module includes, etc.
+    # Additional database columns can be specified via `with_columns` method inside `block`,
+    # which uses Rails migration syntax.
+    def create(name, options = {}, &block)
+      namespace_name, model_name = split_name(name)
+      namespace = namespace_name ? NamespaceFactory.new(namespace_name).klass : Object
+      @namespaces << namespace if namespace_name
+      model = ModelFactory.new(model_name, namespace, options, &block).klass
+      @models << model
+      model
     end
 
+    # Completely destroy everything created by Temping.
+    #
+    # This includes:
+    # * removing all the records in the models created by Temping and dropping their tables
+    # from the database;
+    # * undefining model constants so they cannot be pointed to anymore in the code.
     def teardown
-      if @model_klasses.any?
-        @model_klasses.each do |klass|
-          if Object.const_defined?(klass.name)
-            klass.connection.drop_table(klass.table_name)
-            Object.send(:remove_const, klass.name)
-          end
-        end
-        @model_klasses.clear
-        ActiveSupport::Dependencies::Reference.clear!
+      if @models.any?
+        teardown_models
+        teardown_namespaces
+        ActiveSupport::Dependencies::Reference.clear! if ActiveRecord::VERSION::MAJOR < 7
       end
     end
 
+    # Destroy all records from each of the models created by Temping.
+    #
+    # This does not undefine the models themselves or drop their tables.
+    # This method is an alternative to `teardown` if you want to keep the models and tables.
     def cleanup
-      @model_klasses.each(&:destroy_all)
+      @models.reverse_each(&:destroy_all)
     end
-  end
 
-  class ModelFactory
-    def initialize(model_name, options = {}, &block)
-      @model_name = model_name
-      @options = options
-      klass.class_eval(&block) if block_given?
-      klass.reset_column_information
-    end
+    # Split the provided name finding the namespace (if any) and the model name without namespace.
+    def split_name(name)
+      classified_name = name.to_s.classify
+      name_parts = classified_name.split("::")
+      namespace_name = name_parts[0...-1].join("::")
+      return [nil, classified_name] if namespace_name.empty?
 
-    def klass
-      @klass ||= Object.const_get(@model_name)
-    rescue NameError
-      @klass = build
+      [namespace_name, name_parts.last]
     end
+    private :split_name
 
-    private
-
-    def build
-      Class.new(@options.fetch(:parent_class, default_parent_class)).tap do |klass|
-        Object.const_set(@model_name, klass)
-
-        klass.primary_key = @options[:primary_key] || :id
-        create_table(@options)
-        add_methods
+    # Iterate over `@models`, undefine model constants, drop tables, and remove the constants
+    # from the array one by one starting with the models defined last. (Models defined later
+    # can point to older models by using foreign keys so they have to be removed first).
+    def teardown_models
+      @models.reverse_each do |model|
+        model_name_without_namespace = model.name.split("::").last
+        if model.namespace.const_defined?(model_name_without_namespace)
+          model.connection.drop_table(model.table_name)
+          model.namespace.send(:remove_const, model_name_without_namespace)
+        end
       end
+      @models.clear
     end
+    private :teardown_models
 
-    def default_parent_class
-      if ActiveRecord::VERSION::MAJOR > 4 && defined?(ApplicationRecord)
-        ApplicationRecord
-      else
-        ActiveRecord::Base
+    # Iterate over `@namespaces`, undefine modules and remove them from the array one by one
+    # starting with the deepest ones first.
+    def teardown_namespaces
+      @namespaces.select! { |namespace| namespace_still_defined?(namespace) }
+      until @namespaces.empty?
+        namespace, index = @namespaces.each_with_index.max_by { |n, _i| n.name.split("::").length }
+        parts = namespace.name.split("::")
+        parent = parts.length == 1 ? Object : parts[0...-1].join("::").constantize
+        parent.send(:remove_const, parts.last) if namespace_removable?(namespace, parts, parent)
+        delete_or_trim_in_namespaces(parent, parts, index)
       end
     end
+    private :teardown_namespaces
 
-    DEFAULT_OPTIONS = { :temporary => true }
-    def create_table(options = {})
-      connection.create_table(table_name, DEFAULT_OPTIONS.merge(options))
-    end
-
-    def add_methods
-      class << klass
-        def with_columns
-          connection.change_table(table_name) do |table|
-            yield(table)
-          end
-        end
+    # Check if namespace is still defined.
+    # It could be already removed if it were inside a model created by Temping.
+    # Since `@models` are teared down first, it means that in such a case all modules that were
+    # inside that model are no longer defined.
+    def namespace_still_defined?(namespace)
+      parent = Object
+      outer_namespace_parts = []
+      namespace.to_s.split("::").each do |part|
+        return false unless parent.const_defined?(part)
 
-        def table_exists?
-          true
-        end
+        outer_namespace_parts.push(part)
+        parent = outer_namespace_parts.join("::").constantize
       end
+      true
     end
+    private :namespace_still_defined?
 
-    def connection
-      klass.connection
+    # Namespace can be removed only if it's still defined and if it was created by Temping,
+    # we use `defined_by_temping?` to indicate the latter.
+    def namespace_removable?(namespace, parts, parent)
+      parent.const_defined?(parts.last) && namespace.defined_by_temping?
+    rescue NoMethodError
+      false
     end
+    private :namespace_removable?
+
+    # Clean `@namespaces` array by either removing current namespace or replacing it with its
+    # parent.
+    #
+    # Case 1: @namespaces = [A, B, C, D]; index = 3
+    # This is an outer-most module, we just remove it from `@namespaces`.
+    #
+    # Case 2: @namespaces = [A::B, A::B::C, A::D]; index = 1
+    # Once we remove C from A::B::C, it becomes A::B, but we already have A::B, so just remove it.
+    #
+    # Case 3: @namespaces = [A::B, A::D]; index = 1
+    # Once we remove D from A::D, it becomes A, replace A::D with A.
+    def delete_or_trim_in_namespaces(parent, parts, index)
+      is_last_module = parts.length == 1
+      if is_last_module
+        @namespaces.delete_at(index)
+        return
+      end
+
+      parent_already_in_namespaces = @namespaces.include?(parent)
+      if parent_already_in_namespaces
+        @namespaces.delete_at(index)
+        return
+      end
 
-    def table_name
-      klass.table_name
+      @namespaces[index] = parent
     end
+    private :delete_or_trim_in_namespaces
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: temping
 version: !ruby/object:Gem::Version
-  version: 3.9.0
+  version: 4.0.0
 platform: ruby
 authors:
 - John Pignata
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-16 00:00:00.000000000 Z
+date: 2022-12-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,112 +16,170 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '5.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '5.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.1'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '5.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '5.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.1'
 - !ruby/object:Gem::Dependency
-  name: sqlite3
+  name: appraisal
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.10
+        version: '2.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.10
+        version: '2.2'
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.3'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.3'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: pg
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: mysql2
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.18.2
+        version: '0.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.18.2
+        version: '0.5'
 - !ruby/object:Gem::Dependency
-  name: mysql
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.9.1
+        version: '3.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.9.1
+        version: '3.12'
 - !ruby/object:Gem::Dependency
-  name: mysql2
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.18
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.18
+        version: '13.0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.4.0
+        version: '0.17'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.4.0
+        version: '0.17'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: standard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 10.0.4
+        version: 0.0.1
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 10.0.4
+        version: 0.0.1
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
 description: 
 email: john@pignata.com
 executables: []
@@ -129,8 +187,11 @@ extensions: []
 extra_rdoc_files: []
 files:
 - lib/temping.rb
+- lib/temping/model_factory.rb
+- lib/temping/namespace_factory.rb
 homepage: http://github.com/jpignata/temping
-licenses: []
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -140,15 +201,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.2.2
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.10
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Create temporary table-backed ActiveRecord models for use in tests