foobara 0.5.10 → 0.6.0

data/projects/command_connectors/src/desugarizers/attributes.rb

@@ -5,15 +5,15 @@ module Foobara
     module Desugarizers
       class Attributes < Desugarizer
         def desugarizer_symbol
-          # :nocov:
-          raise "subclass responsibility"
-          # :nocov:
+          # simplecov:disable
+          raise NotImplementedError
+          # simplecov:enable
         end
 
         def opts_key
-          # :nocov:
-          raise "subclass responsibility"
-          # :nocov:
+          # simplecov:disable
+          raise NotImplementedError
+          # simplecov:enable
         end
 
         def applicable?(args_and_opts)
@@ -40,13 +40,13 @@ module Foobara
           transformers = transformers.map do |transformer|
             if transformer.is_a?(::Hash) && transformer.key?(desugarizer_symbol)
               params = transformer[desugarizer_symbol]
-              AttributesTransformers.send(transformer_method, *params)
+              AttributesTransformers.__send__(transformer_method, *params)
             else
               transformer
             end
           end
 
-          transformers = transformers.first unless is_array
+          transformers = transformers[0] unless is_array
 
           opts = opts.merge(opts_key => transformers)
 

data/projects/command_connectors/src/desugarizers/auth.rb

@@ -3,7 +3,7 @@ module Foobara
     module Desugarizers
       class Auth < Desugarizer
         def applicable?(args_and_opts)
-          args_and_opts.last.key?(:auth)
+          args_and_opts[-1].key?(:auth)
         end
 
         def desugarize(args_and_opts)

data/projects/command_connectors/src/desugarizers/set_inputs.rb

@@ -30,13 +30,13 @@ module Foobara
               AttributesTransformers.set(transformer[:set])
             else
               # TODO: add a test for this
-              # :nocov:
+              # simplecov:disable
               transformer
-              # :nocov:
+              # simplecov:enable
             end
           end
 
-          transformers = transformers.first unless is_array
+          transformers = transformers[0] unless is_array
 
           opts = opts.merge(inputs_transformers: transformers)
 

data/projects/command_connectors/src/desugarizers/symbols_to_true.rb

@@ -20,9 +20,9 @@ module Foobara
             if arg.is_a?(::Symbol)
               new_opts[arg] = true
             else
-              # :nocov:
+              # simplecov:disable
               raise "Was not expecting non-symbol arg: #{arg}"
-              # :nocov:
+              # simplecov:enable
             end
           end
 

data/projects/command_connectors/src/request_mutator.rb

@@ -2,9 +2,9 @@ module Foobara
   module CommandConnectors
     class RequestMutator < Foobara::Value::Mutator
       def inputs_type_declaration_from(_inputs_type)
-        # :nocov:
-        raise "subclass responsibility"
-        # :nocov:
+        # simplecov:disable
+        raise NotImplementedError
+        # simplecov:enable
       end
 
       def inputs_type_from(inputs_type)
@@ -13,9 +13,9 @@ module Foobara
       end
 
       def mutate
-        # :nocov:
-        raise "subclass responsibility"
-        # :nocov:
+        # simplecov:disable
+        raise NotImplementedError
+        # simplecov:enable
       end
 
       alias request declaration_data

data/projects/command_connectors/src/response_mutator.rb

@@ -2,9 +2,9 @@ module Foobara
   module CommandConnectors
     class ResponseMutator < Foobara::Value::Mutator
       def result_type_declaration_from(_result_type)
-        # :nocov:
-        raise "subclass responsibility"
-        # :nocov:
+        # simplecov:disable
+        raise NotImplementedError
+        # simplecov:enable
       end
 
       def result_type_from(result_type)
@@ -13,9 +13,9 @@ module Foobara
       end
 
       def mutate
-        # :nocov:
-        raise "subclass responsibility"
-        # :nocov:
+        # simplecov:disable
+        raise NotImplementedError
+        # simplecov:enable
       end
 
       alias response declaration_data

data/projects/command_connectors/src/transformed_command.rb

@@ -19,6 +19,7 @@ module Foobara
                     :request_mutators,
                     :allowed_rule,
                     :requires_authentication,
+                    :authentication_optional,
                     :authenticator,
                     :subclassed_in_namespace,
                     :suffix,
@@ -39,6 +40,7 @@ module Foobara
         allowed_rule:,
         requires_authentication:,
         authenticator:,
+        authentication_optional: nil,
         suffix: nil,
         capture_unknown_error: false
       )
@@ -56,6 +58,7 @@ module Foobara
           klass.request_mutators = Util.array(request_mutators)
           klass.allowed_rule = allowed_rule
           klass.requires_authentication = requires_authentication
+          klass.authentication_optional = authentication_optional
           klass.authenticator = authenticator
           klass.subclassed_in_namespace = scoped_namespace
           klass.suffix = suffix
@@ -63,15 +66,15 @@ module Foobara
       end
 
       def domain
-        # :nocov:
+        # simplecov:disable
         command_class.domain
-        # :nocov:
+        # simplecov:enable
       end
 
       def organization
-        # :nocov:
+        # simplecov:disable
         command_class.organization
-        # :nocov:
+        # simplecov:enable
       end
 
       def description
@@ -307,9 +310,9 @@ module Foobara
             if type.registered?
               # TODO: if we ever change from attributes-only inputs type
               # then this will be handy
-              # :nocov:
+              # simplecov:disable
               types |= [type]
-              # :nocov:
+              # simplecov:enable
             end
 
             types |= type.types_depended_on
@@ -321,9 +324,9 @@ module Foobara
             if type.registered?
               # TODO: if we ever change from attributes-only inputs type
               # then this will be handy
-              # :nocov:
+              # simplecov:disable
               types |= [type]
-              # :nocov:
+              # simplecov:enable
             end
 
             types |= type.types_depended_on
@@ -447,7 +450,7 @@ module Foobara
         end
 
         @inputs_transformer = if transformers.size == 1
-                                transformers.first
+                                transformers[0]
                               else
                                 Value::Processor::Pipeline.new(processors: transformers)
                               end
@@ -468,7 +471,7 @@ module Foobara
           transformers = transformers_to_processors(response_mutators, result_type_from_transformers, direction: :from)
 
           if transformers.size == 1
-            transformers.first
+            transformers[0]
           else
             Value::Processor::Pipeline.new(processors: transformers)
           end
@@ -490,7 +493,7 @@ module Foobara
           transformers = transformers_to_processors(request_mutators, inputs_type_from_transformers, direction: :to)
 
           if transformers.size == 1
-            transformers.first
+            transformers[0]
           else
             Value::Processor::Pipeline.new(processors: transformers)
           end
@@ -515,7 +518,7 @@ module Foobara
           transformers = transformers_to_processors(result_transformers, command_class.result_type, direction: :from)
 
           if transformers.size == 1
-            transformers.first
+            transformers[0]
           else
             Value::Processor::Pipeline.new(processors: transformers)
           end
@@ -540,9 +543,9 @@ module Foobara
           elsif transformer.respond_to?(:call)
             Value::Transformer.create(transform: transformer)
           else
-            # :nocov:
+            # simplecov:disable
             raise "Not sure how to apply #{transformer}"
-            # :nocov:
+            # simplecov:enable
           end
         end
       end
@@ -575,7 +578,7 @@ module Foobara
               klass.foobara_manifest_reference
               # TODO: Delete this nocov block
               # TODO: make anonymous scoped path's have better names instead of random hexadecimal
-              # :nocov:
+              # simplecov:disable
             elsif processor.respond_to?(:symbol) && processor.symbol
               processor.symbol
             else
@@ -587,14 +590,14 @@ module Foobara
               end
 
               "Anonymous#{Util.non_full_name(name)}"
-              # :nocov:
+              # simplecov:enable
             end
           elsif processor.is_a?(::Proc)
             "Proc"
           else
-            # :nocov:
+            # simplecov:disable
             "Unknown"
-            # :nocov:
+            # simplecov:enable
           end
         end
       end
@@ -610,15 +613,15 @@ module Foobara
     end
 
     def full_command_name
-      # :nocov:
+      # simplecov:disable
       self.class.full_command_name
-      # :nocov:
+      # simplecov:enable
     end
 
     def command_name
-      # :nocov:
+      # simplecov:disable
       self.class.command_name
-      # :nocov:
+      # simplecov:enable
     end
 
     def command_class
@@ -630,15 +633,15 @@ module Foobara
     end
 
     def inputs_transformers
-      # :nocov:
+      # simplecov:disable
       self.class.inputs_transformers
-      # :nocov:
+      # simplecov:enable
     end
 
     def result_transformers
-      # :nocov:
+      # simplecov:disable
       self.class.result_transformers
-      # :nocov:
+      # simplecov:enable
     end
 
     def errors_transformers
@@ -729,7 +732,7 @@ module Foobara
       transformers = self.class.transformers_to_processors(serializers, nil, declaration_data: self)
 
       if transformers.size == 1
-        transformers.first
+        transformers[0]
       else
         Value::Processor::Pipeline.new(processors: transformers)
       end
@@ -742,7 +745,7 @@ module Foobara
                                                                                      declaration_data: self)
 
       if transformers.size == 1
-        transformers.first
+        transformers[0]
       else
         Value::Processor::Pipeline.new(processors: transformers)
       end
@@ -759,7 +762,7 @@ module Foobara
       )
 
       if transformers.size == 1
-        transformers.first
+        transformers[0]
       else
         Value::Processor::Pipeline.new(processors: transformers)
       end
@@ -790,11 +793,11 @@ module Foobara
       if auth_mapped_method?(method_name)
         auth_mapped_value_for(method_name)
       elsif command.respond_to?(method_name)
-        command.send(method_name, ...)
+        command.__send__(method_name, ...)
       else
-        # :nocov:
+        # simplecov:disable
         super
-        # :nocov:
+        # simplecov:enable
       end
     end
 
@@ -851,9 +854,9 @@ module Foobara
               source = if allowed_rule.block.respond_to?("source") && defined?(MethodSource)
                          begin
                            # This only works when pry is loaded
-                           # :nocov:
+                           # simplecov:disable
                            allowed_rule.block.source
-                           # :nocov:
+                           # simplecov:enable
                          rescue MethodSource::SourceNotFoundError
                            # This path is hit if the way the source code is extracted
                            # doesn't result in valid Ruby, for example, as part of a hash such as:
@@ -900,9 +903,9 @@ module Foobara
       if capture_unknown_error
         self.outcome = Outcome.error(CommandConnector::UnknownError.for(e))
       else
-        # :nocov:
+        # simplecov:disable
         raise
-        # :nocov:
+        # simplecov:enable
       end
     end
 

data/projects/command_connectors/src/transformers/auth_errors_transformer.rb

@@ -26,9 +26,9 @@ module Foobara
         end
 
         def transform(_request)
-          # :nocov:
+          # simplecov:disable
           raise "Not expected to be called"
-          # :nocov:
+          # simplecov:enable
         end
       end
     end

data/projects/command_connectors/src/transformers/entity_to_primary_key_inputs_transformer.rb

@@ -75,9 +75,9 @@ module Foobara
               attributes_type = to_type.target_class.attributes_type
               EntityToPrimaryKeyInputsTransformer.new(to: attributes_type).from_type_declaration
             else
-              # :nocov:
+              # simplecov:disable
               raise "Not sure how to handle #{to_type}"
-              # :nocov:
+              # simplecov:enable
             end
           else
             to_type

data/projects/command_connectors/src/transformers/load_delegated_attributes_entities_pre_commit_transformer.rb

@@ -20,7 +20,7 @@ module Foobara
           case object
           when Entity
             object.class.delegates.each_key do |delegated_attribute_name|
-              object.send(delegated_attribute_name)
+              object.__send__(delegated_attribute_name)
             end
           when Array
             object.each do |element|

data/projects/domain_mapper/src/domain_mapper.rb

@@ -86,7 +86,7 @@ module Foobara
       # TODO: should this be somewhere more general-purpose?
       def args_to_type(*args, **opts, &block)
         if args.size == 1 && opts.empty? && block.nil?
-          object_to_type(args.first)
+          object_to_type(args[0])
         else
           domain.foobara_type_from_declaration(*args, **opts, &block)
         end
@@ -121,9 +121,9 @@ module Foobara
     end
 
     def map
-      # :nocov:
-      raise "subclass responsibility"
-      # :nocov:
+      # simplecov:disable
+      raise NotImplementedError
+      # simplecov:enable
     end
   end
 end

data/projects/domain_mapper/src/domain_mapper_lookups.rb

@@ -8,9 +8,9 @@ module Foobara
         invalid_keys = opts.keys - [:value]
 
         if invalid_keys.any?
-          # :nocov:
+          # simplecov:disable
           raise ArgumentError, "Invalid keys: #{invalid_keys.join(", ")}. expected one of: #{valid_keys.join(", ")}"
-          # :nocov:
+          # simplecov:enable
         end
 
         if opts.key?(:value)
@@ -83,7 +83,7 @@ module Foobara
           end
         end
 
-        value = candidates.first
+        value = candidates[0]
 
         return value if value
 

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

@@ -20,7 +20,7 @@ module Foobara
       end
 
       def reset_all
-        Foobara.raise_if_production!("reset_all")
+        Foobara.raise_if_production!
         install!
       end
     end

data/projects/entities/projects/detached_entity/src/concerns/associations.rb

@@ -59,13 +59,13 @@ module Foobara
                 values
               else
                 if values.size > 1
-                  # :nocov:
+                  # simplecov:disable
                   raise "Multiple records found for #{name} association but only expected 0 or 1."
-                  # :nocov:
+                  # simplecov:enable
                 end
 
                 unless values.empty?
-                  values.first
+                  values[0]
                 end
               end
             end
@@ -73,7 +73,7 @@ module Foobara
 
           def association_for(association_filters)
             if association_filters.size == 1
-              data_path = association_filters.first.to_s
+              data_path = association_filters[0].to_s
 
               if deep_associations.key?(data_path)
                 return data_path
@@ -85,15 +85,15 @@ module Foobara
             end
 
             if result.empty?
-              # :nocov:
+              # simplecov:disable
               raise "Could not find association matching #{association_filters}"
-              # :nocov:
+              # simplecov:enable
             elsif result.size > 1
-              # :nocov:
+              # simplecov:disable
               raise "Multiple associations matched by #{association_filters}"
-              # :nocov:
+              # simplecov:enable
             else
-              result.first
+              result[0]
             end
           end
 
@@ -121,9 +121,9 @@ module Foobara
                 entity_class == filter || entity_class < filter
               end
             else
-              # :nocov:
+              # simplecov:disable
               raise "Not sure how to apply filter #{filter}"
-              # :nocov:
+              # simplecov:enable
             end
           end
 
@@ -168,9 +168,9 @@ module Foobara
 
               if remove_sensitive
                 # TODO: test this code path
-                # :nocov:
+                # simplecov:disable
                 element_types = element_types&.reject(&:sensitive?)
-                # :nocov:
+                # simplecov:enable
               end
 
               element_types&.each&.with_index do |element_type, index|
@@ -196,19 +196,19 @@ module Foobara
               target_class = type.target_class
 
               method = target_class.respond_to?(:foobara_attributes_type) ? :foobara_attributes_type : :attributes_type
-              attributes_type = target_class.send(method)
+              attributes_type = target_class.__send__(method)
 
               if !remove_sensitive || !attributes_type.sensitive?
                 construct_associations(attributes_type, path, result, initial: false)
               end
             elsif type.extends?(BuiltinTypes[:associative_array])
               # not going to bother testing this for now
-              # :nocov:
+              # simplecov:disable
               if contains_associations?(type)
                 raise "Associative array types with associations in them are not currently supported. " \
                       "Use attributes type if you can or set the key_type and/or value_type to duck type"
               end
-              # :nocov:
+              # simplecov:enable
             end
 
             result
@@ -228,9 +228,9 @@ module Foobara
 
               if remove_sensitive
                 # TODO: test this code path
-                # :nocov:
+                # simplecov:disable
                 element_types = element_types&.reject(&:sensitive?)
-                # :nocov:
+                # simplecov:enable
               end
 
               contains_associations?(element_types, false)

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

@@ -20,9 +20,9 @@ module Foobara
           invalid_options = options.keys - ALLOWED_OPTIONS
 
           unless invalid_options.empty?
-            # :nocov:
+            # simplecov:disable
             raise ArgumentError, "Invalid options #{invalid_options} expected only #{ALLOWED_OPTIONS}"
-            # :nocov:
+            # simplecov:enable
           end
 
           if options[:unloaded]

data/projects/entities/projects/detached_entity/src/concerns/primary_key.rb

@@ -11,15 +11,15 @@ module Foobara
         module ClassMethods
           def primary_key(attribute_name)
             if primary_key_attribute
-              # :nocov:
+              # simplecov:disable
               raise "Primary key already set to #{primary_key_attribute}"
-              # :nocov:
+              # simplecov:enable
             end
 
             if attribute_name.nil? || attribute_name.empty?
-              # :nocov:
+              # simplecov:disable
               raise ArgumentError, "Primary key can't be blank"
-              # :nocov:
+              # simplecov:enable
             end
 
             @foobara_primary_key_attribute = attribute_name.to_sym

data/projects/entities/projects/detached_entity/src/concerns/types.rb

@@ -5,9 +5,9 @@ module Foobara
         include Concern
 
         def full_entity_name
-          # :nocov:
+          # simplecov:disable
           self.class.full_entity_name
-          # :nocov:
+          # simplecov:enable
         end
 
         def entity_name

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

@@ -28,18 +28,18 @@ module Foobara
           # TODO: No longer is hit in this test suite but ActiveRecordType needs this class
           # and potentially this snippet of code in order to do the right thing.
           # TODO: test that out and delete this method if possible.
-          # :nocov:
+          # simplecov:disable
           declaration_data = declaration_data.merge(type: :detached_entity)
-          # :nocov:
+          # simplecov:enable
         end
 
         if self.class.model_base_class_requires_conversion?(declaration_data[:model_base_class])
           # TODO: No longer is hit in this test suite but ActiveRecordType needs this class
           # and potentially this snippet of code in order to do the right thing.
           # TODO: test that out and delete this method if possible.
-          # :nocov:
+          # simplecov:disable
           declaration_data = declaration_data.merge(model_base_class: "Foobara::DetachedEntity")
-          # :nocov:
+          # simplecov:enable
         end
 
         # TODO: remove private attributes, add delegated attributes

data/projects/entities/projects/detached_entity/src/extensions/builtin_types/detached_entity/casters/primary_key.rb

@@ -26,7 +26,7 @@ module Foobara
           end
 
           def transform(primary_key)
-            entity_class.send(build_method, primary_key)
+            entity_class.__send__(build_method, primary_key)
           end
 
           def applies_message

data/projects/entities/projects/detached_entity/src/extensions/builtin_types/detached_entity/validators/model_instance_is_valid.rb

@@ -26,9 +26,9 @@ module Foobara
               model_type = type_for_declaration(parent_declaration_data[:type])
               model_type.target_class
             else
-              # :nocov:
+              # simplecov:disable
               raise "Missing :model_class in parent_declaration_data for #{parent_declaration_data}"
-              # :nocov:
+              # simplecov:enable
             end
           end