foobara 0.5.10 → 0.6.0

data/projects/entities/projects/detached_entity/src/extensions/type_declarations/handlers/registered_type_declaration/validate_primary_key_present.rb

@@ -0,0 +1,30 @@
+module Foobara
+  module TypeDeclarations
+    module Handlers
+      class RegisteredTypeDeclaration < TypeDeclarationHandler
+        class ValidatePrimaryKeyPresent < TypeDeclarations::TypeDeclarationValidator
+          # TODO: seems like maybe we could actually check against types now...
+          # like make a type for primary_key: :symbol ??
+          class MissingPrimaryKeyError < TypeDeclarationError; end
+
+          def validation_errors(type_declaration)
+            if type_declaration.claimed_but_error?
+              klass = type_declaration.declaration_data
+
+              if klass.is_a?(::Class) && klass < ::Foobara::DetachedEntity
+                if klass.model_type.nil? && klass.primary_key_attribute.nil?
+                  build_error(
+                    message: "#{klass} doesn't have a primary key attribute set yet. " \
+                             "Did you forget something like `primary_key :id` in #{klass}?"
+                  )
+                end
+              end
+            end
+          end
+
+          def error_context(_value) ={}
+        end
+      end
+    end
+  end
+end

data/projects/entities/projects/detached_entity/src/remove_sensitive_values_transformer_extensions.rb

@@ -44,9 +44,9 @@ module Foobara
               already_sanitized << existing_type
               next
             else
-              # :nocov:
+              # simplecov:disable
               raise "Did not expect to be re-sanitizing #{entity_type.full_type_symbol}"
-              # :nocov:
+              # simplecov:enable
             end
 
           end

data/projects/entities/projects/entity/lib/foobara/entity.rb

@@ -23,7 +23,7 @@ module Foobara
       end
 
       def reset_all
-        Foobara.raise_if_production!("reset_all")
+        Foobara.raise_if_production!
         Entity::Concerns::Callbacks.reset_all
 
         install!

data/projects/entities/projects/entity/src/concerns/attributes.rb

@@ -26,21 +26,21 @@ module Foobara
 
             if attribute_name == primary_key_attribute
               if value.nil?
-                # :nocov:
+                # simplecov:disable
                 raise "Primary key cannot be set to a blank value"
-                # :nocov:
+                # simplecov:enable
               end
 
               if value.is_a?(::String) && value.empty?
-                # :nocov:
+                # simplecov:disable
                 raise "Primary key cannot be set to a blank value"
-                # :nocov:
+                # simplecov:enable
               end
 
               if value.is_a?(::Symbol) && value.to_s.empty?
-                # :nocov:
+                # simplecov:disable
                 raise "Primary key cannot be set to a blank value"
-                # :nocov:
+                # simplecov:enable
               end
 
               write_attribute!(attribute_name, value)

data/projects/entities/projects/entity/src/concerns/callbacks.rb

@@ -8,7 +8,7 @@ module Foobara
 
         class << self
           def reset_all
-            Foobara.raise_if_production!("reset_all")
+            Foobara.raise_if_production!
             Entity.instance_variable_set("@class_callback_registry", nil)
           end
         end

data/projects/entities/projects/entity/src/concerns/initialization.rb

@@ -29,9 +29,9 @@ module Foobara
 
             # TODO: is this possible?
             if record_id.nil?
-              # :nocov:
+              # simplecov:disable
               raise ArgumentError, "Primary key cannot be blank"
-              # :nocov:
+              # simplecov:enable
             end
 
             # check if tracked already...
@@ -58,9 +58,9 @@ module Foobara
             record = current_transaction_table.find_tracked(record_id)
 
             if record&.loaded?
-              # :nocov:
+              # simplecov:disable
               raise "Already loaded for #{attributes}. Bug maybe?"
-              # :nocov:
+              # simplecov:enable
             end
 
             record = __private_new__
@@ -68,9 +68,9 @@ module Foobara
             record.successfully_loaded(attributes)
 
             unless record.primary_key
-              # :nocov:
+              # simplecov:disable
               raise "Expected primary key #{primary_key_attribute} to be present!"
-              # :nocov:
+              # simplecov:enable
             end
 
             record.fire(:initialized)
@@ -104,9 +104,9 @@ module Foobara
 
         def successfully_loaded(attributes)
           if hard_deleted?
-            # :nocov:
+            # simplecov:disable
             raise "Not expecting to load a hard deleted record"
-            # :nocov:
+            # simplecov:enable
           end
 
           # TODO: why would we proceed if this is the case? Maybe raise?

data/projects/entities/projects/entity/src/concerns/mutations.rb

@@ -50,13 +50,13 @@ module Foobara
 
               object
             elsif type.extends?(BuiltinTypes[:tuple])
-              # :nocov:
+              # simplecov:disable
               raise "Tuple not yet supported"
-              # :nocov:
+              # simplecov:enable
             elsif type.extends?(BuiltinTypes[:associative_array])
-              # :nocov:
+              # simplecov:disable
               raise "Associated array not yet supported"
-              # :nocov:
+              # simplecov:enable
             elsif type.extends?(BuiltinTypes[:array])
               element_type = type.element_type
 

data/projects/entities/projects/entity/src/concerns/persistence.rb

@@ -62,7 +62,7 @@ module Foobara
 
           attribute_name = if attribute_name_or_attributes.is_a?(::Hash)
                              if attribute_name_or_attributes.keys.size == 1
-                               attribute_name_or_attributes.keys.first.to_sym
+                               attribute_name_or_attributes.keys[0].to_sym
                              end
                            elsif attribute_name_or_attributes
                              attribute_name_or_attributes.to_sym
@@ -75,9 +75,9 @@ module Foobara
           transaction = Foobara::Persistence::EntityBase::Transaction.open_transaction_for(self)
 
           unless transaction
-            # :nocov:
+            # simplecov:disable
             raise NoCurrentTransactionError, "Trying to load a #{entity_name} outside of a transaction."
-            # :nocov:
+            # simplecov:enable
           end
 
           unless transaction.loading?(self)

data/projects/entities/projects/entity/src/concerns/queries.rb

@@ -57,7 +57,7 @@ module Foobara
 
             if load_paths
               if load_paths.is_a?(::Array)
-                first = load_paths.first
+                first = load_paths[0]
 
                 if first.is_a?(::Symbol) || (first.is_a?(::String) && !first.include?("."))
                   load_paths = [load_paths]
@@ -99,8 +99,8 @@ module Foobara
           end
 
           def load_many(*record_ids)
-            if record_ids.size == 1 && record_ids.first.is_a?(::Array)
-              record_ids = record_ids.first
+            if record_ids.size == 1 && record_ids[0].is_a?(::Array)
+              record_ids = record_ids[0]
             end
 
             current_transaction_table.load_many(record_ids)
@@ -125,11 +125,11 @@ module Foobara
 
             unless containing_records.empty?
               if containing_records.size == 1
-                containing_records.first
+                containing_records[0]
               else
-                # :nocov:
+                # simplecov:disable
                 raise "Expected only one record to own #{record} but found #{containing_records.size}"
-                # :nocov:
+                # simplecov:enable
               end
             end
           end
@@ -155,7 +155,7 @@ module Foobara
             end until owning_entity_class
 
             if attribute_path.size == 1
-              attribute_path = attribute_path.first
+              attribute_path = attribute_path[0]
             end
 
             if owning_entity_class == self

data/projects/entities/projects/entity/src/extensions/domain/domain_module_extension.rb

@@ -34,15 +34,15 @@ module Foobara
                                             when 1, 2
                                               arg, other = args
 
-                                              if args.first.is_a?(::String)
+                                              if args[0].is_a?(::String)
                                                 [other, arg]
                                               else
                                                 args
                                               end
                                             else
-                                              # :nocov:
+                                              # simplecov:disable
                                               raise ArgumentError, "Too many arguments"
-                                              # :nocov:
+                                              # simplecov:enable
                                             end
 
             if model_base_class
@@ -59,7 +59,7 @@ module Foobara
             attributes_type = handler.type_for_declaration(attributes_type_declaration)
 
             # TODO: reuse the model_base_class primary key if it has one...
-            primary_key = attributes_type.element_types.keys.first
+            primary_key = attributes_type.element_types.keys[0]
 
             model_module = unless scoped_full_path.empty?
                              scoped_full_name

data/projects/entities/projects/entity/src/new_prepend.rb

@@ -11,9 +11,9 @@ module Foobara
 
       # rubocop:disable Lint/DuplicateMethods
       def new(...)
-        # :nocov:
+        # simplecov:disable
         raise "Cannot initialize a #{name}. Use .create, .thunk, .loaded, or .build instead."
-        # :nocov:
+        # simplecov:enable
       end
       # rubocop:enable Lint/DuplicateMethods
     end

data/projects/entities/projects/entity/src/not_found_error.rb

@@ -1,6 +1,6 @@
 module Foobara
   class Entity < DetachedEntity
-    class NotFoundError < Foobara::RuntimeError
+    class NotFoundError < Foobara::DataError
       class << self
         def not_found_error_class_name(data_path)
           error_class_name = data_path.path.map { |part| part == :"#" ? "Collection" : Util.classify(part) }.join
@@ -13,8 +13,8 @@ module Foobara
           # TODO: how would we avoid name collisions here??
           Util.make_class("#{mod.name}::#{error_class_name}", self) do
             # TODO: use Concern to change these into attr_accessor instead
-            singleton_class.define_method :data_path do
-              data_path
+            singleton_class.define_method :symbol do
+              :not_found
             end
 
             singleton_class.define_method :entity_class do
@@ -39,7 +39,7 @@ module Foobara
             data_path: data_path.to_s
           }
 
-          new(context:, message:)
+          new(context:, message:, path: data_path)
         end
 
         def data_path
@@ -54,15 +54,15 @@ module Foobara
       end
 
       def primary_key_attribute
-        # :nocov:
+        # simplecov:disable
         entity_class.primary_key_attribute
-        # :nocov:
+        # simplecov:enable
       end
 
       def full_entity_name
-        # :nocov:
+        # simplecov:disable
         entity_class.full_entity_name
-        # :nocov:
+        # simplecov:enable
       end
 
       def criteria

data/projects/entities/projects/entity/src/sensitive_value_removers/entity.rb

@@ -8,23 +8,26 @@ module Foobara
           elsif record.persisted?
             # We will assume that we do not need to clean up the primary key itself as
             # we will assume we don't allow sensitive primary keys for now.
-            thunkish = to_type.target_class.send(build_method, record.class.primary_key_attribute => record.primary_key)
+            thunkish = to_type.target_class.__send__(
+              build_method,
+              record.class.primary_key_attribute => record.primary_key
+            )
             thunkish.skip_validations = true
             thunkish.mutable = false
             thunkish
           else
-            # :nocov:
+            # simplecov:disable
             raise "Not sure what to do with a record that isn't loaded, created, or persisted"
-            # :nocov:
+            # simplecov:enable
           end
         end
 
         def build_method
           if to_type.extends?(BuiltinTypes[:entity])
             # TODO: figure out a way to test this path
-            # :nocov:
+            # simplecov:disable
             :build
-            # :nocov:
+            # simplecov:enable
           else
             :new
           end

data/projects/entities/projects/in_memory_crud_driver_minimal/src/in_memory_minimal.rb

@@ -50,9 +50,9 @@ module Foobara
             record_id = record_id_for(attributes)
 
             unless exists?(record_id)
-              # :nocov:
+              # simplecov:disable
               raise CannotUpdateError.new(record_id, "does not exist")
-              # :nocov:
+              # simplecov:enable
             end
 
             records[record_id] = Util.deep_dup(attributes)
@@ -61,9 +61,9 @@ module Foobara
 
           def hard_delete(record_id)
             unless exists?(record_id)
-              # :nocov:
+              # simplecov:disable
               raise CannotUpdateError.new(record_id, "does not exist")
-              # :nocov:
+              # simplecov:enable
             end
 
             records.delete(record_id)

data/projects/entities/projects/model_attribute_helpers/lib/foobara/model_attribute_helpers.rb

@@ -8,8 +8,6 @@ module Foobara
       def install!
         Model.include Concerns::AttributeHelpers
         Model.include Concerns::AttributeHelperAliases
-        Entity.include Concerns::AttributeHelpers
-        Entity.include Concerns::AttributeHelperAliases
       end
     end
   end

data/projects/entities/projects/model_attribute_helpers/src/attribute_helper_aliases.rb

@@ -20,7 +20,7 @@ module Foobara
             :type_declaration_value_at
           ].each do |method_name|
             define_method method_name do |*args, **opts, &block|
-              send("foobara_#{method_name}", *args, **opts, &block)
+              __send__("foobara_#{method_name}", *args, **opts, &block)
             end
           end
         end

data/projects/entities/projects/model_attribute_helpers/src/attribute_helpers.rb

@@ -170,9 +170,9 @@ module Foobara
                 )
               end
             else
-              # :nocov:
+              # simplecov:disable
               raise "Bad path part #{path_part}"
-              # :nocov:
+              # simplecov:enable
             end
           end
         end

data/projects/entities/projects/nested_transactionable/lib/foobara/nested_transactionable.rb

@@ -34,9 +34,9 @@ module Foobara
     end
 
     def relevant_entity_classes
-      # :nocov:
-      raise "subclass responsibility"
-      # :nocov:
+      # simplecov:disable
+      raise NotImplementedError
+      # simplecov:enable
     end
 
     def transactions
@@ -118,9 +118,9 @@ module Foobara
         if transaction.currently_open?
           # Hard to test this because halting and other exceptions rollback the transactions via
           # block form but to be safe keeping this
-          # :nocov:
+          # simplecov:disable
           transaction.rollback!
-          # :nocov:
+          # simplecov:enable
         end
       end
     end

data/projects/entities/projects/persistence/lib/foobara/persistence.rb

@@ -4,7 +4,7 @@ module Foobara
   module Persistence
     class << self
       def reset_all
-        Foobara.raise_if_production!("reset_all")
+        Foobara.raise_if_production!
         @tables_for_entity_class_name = @bases = @default_crud_driver = @default_base = nil
         EntityBase::Transaction::Concerns::EntityCallbackHandling.reset_all
         EntityBase::Transaction.reset_all

data/projects/entities/projects/persistence/src/entity_attributes_crud_driver.rb

@@ -78,9 +78,9 @@ module Foobara
             match = /^Cannot(\w+)Error$/.match(Util.non_full_name(self.class))
 
             unless match
-              # :nocov:
+              # simplecov:disable
               raise "Bad error name for #{self.class.name}"
-              # :nocov:
+              # simplecov:enable
             end
 
             Util.underscore(match[1])
@@ -121,15 +121,15 @@ module Foobara
 
         # CRUD
         def select(_query_declaration)
-          # :nocov:
-          raise "subclass responsibility"
-          # :nocov:
+          # simplecov:disable
+          raise NotImplementedError
+          # simplecov:enable
         end
 
         def all(page_size: nil)
-          # :nocov:
-          raise "subclass responsibility"
-          # :nocov:
+          # simplecov:disable
+          raise NotImplementedError
+          # simplecov:enable
         end
 
         def first
@@ -137,9 +137,9 @@ module Foobara
         end
 
         def find(_record_id)
-          # :nocov:
-          raise "subclass responsibility"
-          # :nocov:
+          # simplecov:disable
+          raise NotImplementedError
+          # simplecov:enable
         end
 
         def find!(record_id)
@@ -184,9 +184,7 @@ module Foobara
 
         def find_all_by_attribute_containing_any_of(attribute_name, values)
           all.select do |attributes|
-            attributes[attribute_name]&.any? do |attribute_value|
-              values.include?(attribute_value)
-            end
+            attributes[attribute_name]&.intersect?(values)
           end
         end
 
@@ -228,51 +226,51 @@ module Foobara
         end
 
         def insert(_attributes)
-          # :nocov:
-          raise "subclass responsibility"
-          # :nocov:
+          # simplecov:disable
+          raise NotImplementedError
+          # simplecov:enable
         end
 
         def insert_many(attributes_array)
           # TODO: add a test for a driver that doesn't override this and remove these :nocov: comments
-          # :nocov:
+          # simplecov:disable
           attributes_array.each.lazy.map do |attributes|
             insert(attributes)
           end
-          # :nocov:
+          # simplecov:enable
         end
 
         def update(_record)
-          # :nocov:
-          raise "subclass responsibility"
-          # :nocov:
+          # simplecov:disable
+          raise NotImplementedError
+          # simplecov:enable
         end
 
         def hard_delete(_record_id)
-          # :nocov:
-          raise "subclass responsibility"
-          # :nocov:
+          # simplecov:disable
+          raise NotImplementedError
+          # simplecov:enable
         end
 
         def hard_delete_many(record_ids)
           # TODO: add a test for a driver that doesn't override this and remove these :nocov: comments
-          # :nocov:
+          # simplecov:disable
           record_ids.each.lazy.map do |record_id|
             delete(record_id)
           end
-          # :nocov:
+          # simplecov:enable
         end
 
         def hard_delete_all!
-          # :nocov:
-          raise "subclass responsibility"
-          # :nocov:
+          # simplecov:disable
+          raise NotImplementedError
+          # simplecov:enable
         end
 
         def count
-          # :nocov:
-          raise "subclass responsibility"
-          # :nocov:
+          # simplecov:disable
+          raise NotImplementedError
+          # simplecov:enable
         end
 
         def exists?(record_id)
@@ -301,12 +299,12 @@ module Foobara
             end
           elsif type.extends?(BuiltinTypes[:tuple])
             # TODO: test this code path
-            # :nocov:
+            # simplecov:disable
             object.map.with_index do |value, index|
               element_type = type.element_types[index]
               restore_attributes(value, element_type)
             end
-            # :nocov:
+            # simplecov:enable
           elsif type.extends?(BuiltinTypes[:array])
             element_type = type.element_type
             object.map { |value| restore_attributes(value, element_type) }
@@ -333,9 +331,9 @@ module Foobara
               outcome.result
             else
               # TODO: figure out how to test this code path
-              # :nocov:
+              # simplecov:disable
               object
-              # :nocov:
+              # simplecov:enable
             end
           end
         end