fabrial 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf013a11226fff5bf52dea812c36fcc95e8912f05d8118a8973727a7864125d6
-  data.tar.gz: 663a33a75e7e52a7e5f51be1476d0a250dc892a47fe0f8909b673fdeb7b6c23b
+  metadata.gz: 86f03b9211c2441a8ca1e341e7bfb202e6d1bdf0cf588f8abc581aea8daf1a68
+  data.tar.gz: 9bc2b809e7fa65f979dd7aa268555c4b142d565dacb31cffb1e4805d5b6a89f9
 SHA512:
-  metadata.gz: 0b7bafadd06459481ddbc1e93e3598885976b9c1d93d44d8ac2319683554699f802f482988c3265c6dcf11df7ba27af4d45fafffecaf751dbc0b43ce9776f18f
-  data.tar.gz: 1949a259ec2c8f075241e54a11d9eebad8fde59473501af794b81e193772b26a331af245eeb36f14c247fe423f3b040881602a8115fce8d73a7263adb777cb01
+  metadata.gz: c214bb35c6d29c06ebfa3d21b5a93064da81abbc86f3e28e5713ae906b0ca476beaaf40aa60d011b37f7ef10ffffc37ed7a5b5bad3de05cedc704f58be171c44
+  data.tar.gz: ac7a67c81a346eebdf6a9440bed3627797ab576a0c15ff247acc7d3280af3c731a491e0ea0dd5a2d9a71d16871fc24fb65932fbc2946fce40cc1c0d673bd7fee

data/.gitignore

@@ -7,6 +7,9 @@
 /spec/reports/
 /tmp/
 
+# Gems should not commit `Gemfile.lock`
+Gemfile.lock
+
 # Appraisal gemfiles
 /gemfiles/.bundle/
 /gemfiles/*.gemfile.lock

data/fabrial.gemspec

@@ -47,6 +47,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'minitest', '~> 5.12', '>= 5.12.2'
   spec.add_development_dependency 'minitest-focus', '~> 1.1', '>= 1.1.2'
   spec.add_development_dependency 'minitest-reporters', '~> 1.3', '>= 1.3.8'
+  spec.add_development_dependency 'mocha'
   spec.add_development_dependency 'pry'
   spec.add_development_dependency 'pry-byebug'
   spec.add_development_dependency 'rake', '~> 13.0'

data/lib/fabrial/errors.rb

@@ -3,4 +3,11 @@
 module Fabrial
   class Error < RuntimeError; end
   class UnknownClassError < Error; end
+  class CreationError < Error
+    def message
+      return super unless cause
+
+      "#{super}: #{cause.message}"
+    end
+  end
 end

data/lib/fabrial/fabricate.rb

@@ -34,6 +34,28 @@ module Fabrial::Fabricate
     end
   end
 
+  def extract_child_records(klass, data)
+    children = data.select do |type, v|
+      # Must have nested data
+      [Array, Hash].any? { |c| v.is_a? c } &&
+        # Must be a class that we can instantiate
+        get_class(type) &&
+
+        # Even if it has the same name as a Model in the system, if it is also
+        # the name of a column in the table, assume the data is for a serialzed
+        # field and not a nested relation.  Ex: Requests have a serialized field
+        # called content and there is also a Content model in the system.
+        (
+          # If they are using a class as the key, then always choose the class
+          # over the field.
+          type.is_a?(Class) ||
+
+          !column_names(klass).include?(type.to_s)
+        )
+    end
+    data.extract!(*children.keys)
+  end
+
   private
 
   def contains_return?(objects)
@@ -121,7 +143,12 @@ module Fabrial::Fabricate
     type_col = klass.inheritance_column.try :to_sym
     type = data.delete(type_col).try :safe_constantize
     type ||= klass
-    create type, data.reverse_merge(associations)
+    begin
+      create type, data.reverse_merge(associations)
+    rescue
+      raise Fabrial::CreationError,
+        "Error creating #{type.name} with data: #{data}"
+    end
   end
 
   def collect_associations(klass, ancestors)
@@ -132,28 +159,6 @@ module Fabrial::Fabricate
     associations
   end
 
-  def extract_child_records(klass, data)
-    children = data.select do |type, v|
-      # Must have nested data
-      [Array, Hash].any? { |c| v.is_a? c } &&
-        # Must be a class that we can instantiate
-        get_class(type) &&
-
-        # Even if it has the same name as a Model in the system, if it is also
-        # the name of a column in the table, assume the data is for a serialzed
-        # field and not a nested relation.  Ex: Requests have a serialized field
-        # called content and there is also a Content model in the system.
-        (
-          # If they are using a class as the key, then always choose the class
-          # over the field.
-          type.is_a?(Class) ||
-
-            !column_names(klass).include?(type.to_s)
-        )
-    end
-    data.extract!(*children.keys)
-  end
-
   def column_names(klass)
     klass.column_names
 

data/lib/fabrial/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Fabrial
-  VERSION = '0.2.0'
+  VERSION = '0.3.0'
   public_constant :VERSION
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fabrial
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Jeremy Mickelson
@@ -140,6 +140,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 1.3.8
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
@@ -227,7 +241,6 @@ files:
 - ".travis.yml"
 - Appraisals
 - Gemfile
-- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile

data/Gemfile.lock

@@ -1,89 +0,0 @@
-PATH
-  remote: .
-  specs:
-    fabrial (0.1.0)
-      activerecord (> 4.2.0, < 6.0)
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    activemodel (5.2.3)
-      activesupport (= 5.2.3)
-    activerecord (5.2.3)
-      activemodel (= 5.2.3)
-      activesupport (= 5.2.3)
-      arel (>= 9.0)
-    activesupport (5.2.3)
-      concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 0.7, < 2)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-    ansi (1.5.0)
-    appraisal (2.2.0)
-      bundler
-      rake
-      thor (>= 0.14.0)
-    arel (9.0.0)
-    ast (2.4.0)
-    builder (3.2.3)
-    byebug (11.0.1)
-    coderay (1.1.2)
-    concurrent-ruby (1.1.5)
-    database_cleaner (1.7.0)
-    i18n (1.7.0)
-      concurrent-ruby (~> 1.0)
-    jaro_winkler (1.5.3)
-    method_source (0.9.2)
-    minitest (5.12.2)
-    minitest-focus (1.1.2)
-      minitest (>= 4, < 6)
-    minitest-reporters (1.3.8)
-      ansi
-      builder
-      minitest (>= 5.0)
-      ruby-progressbar
-    parallel (1.17.0)
-    parser (2.6.5.0)
-      ast (~> 2.4.0)
-    pry (0.12.2)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
-    pry-byebug (3.7.0)
-      byebug (~> 11.0)
-      pry (~> 0.10)
-    rainbow (3.0.0)
-    rake (13.0.0)
-    rubocop (0.75.0)
-      jaro_winkler (~> 1.5.1)
-      parallel (~> 1.10)
-      parser (>= 2.6)
-      rainbow (>= 2.2.2, < 4.0)
-      ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 1.7)
-    ruby-progressbar (1.10.1)
-    sqlite3 (1.3.13)
-    thor (0.20.3)
-    thread_safe (0.3.6)
-    tzinfo (1.2.5)
-      thread_safe (~> 0.1)
-    unicode-display_width (1.6.0)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  appraisal (~> 2.2, >= 2.2.0)
-  bundler (~> 1.0)
-  database_cleaner
-  fabrial!
-  minitest (~> 5.12, >= 5.12.2)
-  minitest-focus (~> 1.1, >= 1.1.2)
-  minitest-reporters (~> 1.3, >= 1.3.8)
-  pry
-  pry-byebug
-  rake (~> 13.0)
-  rubocop (~> 0.75.0)
-  sqlite3 (~> 1.3.0)
-
-BUNDLED WITH
-   1.17.3