praxis 2.0.pre.11 → 2.0.pre.16

data/spec/praxis/response_definition_spec.rb

@@ -8,7 +8,8 @@ describe Praxis::ResponseDefinition do
     Proc.new do
       status 200
       description 'test description'
-      headers({ "X-Header" => "value", "Content-Type" => "application/some-type" })
+      header( "X-Header", "value", description: 'Very nais header')
+      header( "Content-Type", "application/some-type" )
     end
   end
 
@@ -17,7 +18,7 @@ describe Praxis::ResponseDefinition do
   its(:parts) { should be(nil) }
   let(:response_status) { 200 }
   let(:response_content_type) { "application/some-type" }
-  let(:response_headers) { { "X-Header" => "value", "Content-Type" => response_content_type} }
+  let(:response_headers) { { "X-Header" => "value", "Content-Type" => response_content_type, "Location" => '/somewhere/over/the/rainbow'} }
 
   let(:response) { instance_double("Praxis::Response", status: response_status , headers: response_headers, content_type: response_content_type ) }
 
@@ -105,29 +106,6 @@ describe Praxis::ResponseDefinition do
     end
   end
 
-  context '#headers' do
-    it 'accepts a Hash' do
-      response_definition.headers Hash["X-Header" => "value", "Content-Type" => "application/some-type"]
-      expect(response_definition.headers).to be_a(Hash)
-    end
-
-    it 'accepts an Array' do
-      response_definition.headers ["X-Header: value", "Content-Type: application/some-type"]
-      expect(response_definition.headers).to be_a(Hash)
-      expect(response_definition.headers.keys).to include("X-Header: value", "Content-Type: application/some-type")
-    end
-
-    it 'accepts a String' do
-      response_definition.headers "X-Header: value"
-      expect(response_definition.headers).to be_a(Hash)
-      expect(response_definition.headers.keys).to include("X-Header: value")
-    end
-
-    it 'should return an error when headers are not a Hash, Array or String object' do
-      expect{ response_definition.headers Object.new }. to raise_error(Praxis::Exceptions::InvalidConfiguration)
-    end
-  end
-
   context '#parts' do
     context 'with a :like argument (and no block)' do
       before do
@@ -242,7 +220,6 @@ describe Praxis::ResponseDefinition do
 
       it "calls all the validation sub-functions" do
         expect(response_definition).to receive(:validate_status!).once
-        expect(response_definition).to receive(:validate_location!).once
         expect(response_definition).to receive(:validate_headers!).once
         expect(response_definition).to receive(:validate_content_type!).once
         response_definition.validate(response)
@@ -272,112 +249,39 @@ describe Praxis::ResponseDefinition do
 
       end
 
-      describe "#validate_location!" do
-        let(:block) { proc { status 200 } }
-
-        context 'checking location mismatches' do
-          before { response_definition.location(location) }
-
-          context 'for Regexp' do
-            let(:location) { /no_match/ }
-
-            it 'should raise an error' do
-              expect {
-                response_definition.validate_location!(response)
-              }.to raise_error(Praxis::Exceptions::Validation)
-            end
-          end
-
-          context 'for String' do
-            let(:location) { "no_match" }
-            it 'should raise error' do
-              expect {
-                response_definition.validate_location!(response)
-              }.to raise_error(Praxis::Exceptions::Validation)
-            end
+      describe "#validate_headers!" do
+        context 'when there are missing headers' do
+          it 'should raise error' do
+            response_definition.header('X-Unknown', 'test')
+            expect {
+              response_definition.validate_headers!(response)
+            }.to raise_error(Praxis::Exceptions::Validation)
           end
-
         end
-      end
-
-      describe "#validate_headers!" do
-        before { response_definition.headers(headers) }
-        context 'checking headers are set' do
-          context 'when there are missing headers' do
-            let (:headers) { { 'X-some' => 'test' } }
-            it 'should raise error' do
-              expect {
-                response_definition.validate_headers!(response)
-              }.to raise_error(Praxis::Exceptions::Validation)
-            end
+        context 'when headers with same names are returned' do
+          it 'a simply required header should not raise error just by being there' do
+            response_definition.header('X-Header', nil)
+            expect {
+              response_definition.validate_headers!(response)
+            }.to_not raise_error
           end
-
-          context "when headers specs are name strings" do
-            context "and is missing" do
-              let (:headers) { [ "X-Just-Key" ] }
-              it 'should raise error' do
-                expect {
-                  response_definition.validate_headers!(response)
-                }.to raise_error(Praxis::Exceptions::Validation)
-              end
-            end
-
-            context "and is not missing" do
-              let (:headers) { [ "X-Header" ] }
-              it 'should not raise error' do
-                expect {
-                  response_definition.validate_headers!(response)
-                }.not_to raise_error
-              end
-            end
+          it 'an exact string header should not raise error if it fully matches' do
+            response_definition.header('X-Header', 'value')
+            expect {
+              response_definition.validate_headers!(response)
+            }.to_not raise_error
           end
-
-          context "when header specs are hashes" do
-            context "and is missing" do
-              let (:headers) {
-                [ { "X-Just-Key" => "notfoodbar" } ]
-              }
-              it 'should raise error' do
-                expect {
-                  response_definition.validate_headers!(response)
-                }.to raise_error(Praxis::Exceptions::Validation)
-              end
-            end
-
-            context "and is not missing" do
-              let (:headers) {
-                [ { "X-Header" => "value" } ]
-              }
-              it 'should not raise error' do
-                expect {
-                  response_definition.validate_headers!(response)
-                }.not_to raise_error
-              end
-            end
+          it 'a regexp header should not raise error if it matches the regexp' do
+            response_definition.header('X-Header', /value/)
+            expect {
+              response_definition.validate_headers!(response)
+            }.to_not raise_error
           end
-
-          context "when header specs are of mixed type " do
-            context "and is missing" do
-              let (:headers) {
-                [ { "X-Header" => "value" }, "not-gonna-find-me" ]
-              }
-              it 'should raise error' do
-                expect {
-                  response_definition.validate_headers!(response)
-                }.to raise_error(Praxis::Exceptions::Validation)
-              end
-            end
-
-            context "and is not missing" do
-              let (:headers) {
-                [ { "X-Header" => "value" }, "Content-Type" ]
-              }
-              it 'should not raise error' do
-                expect {
-                  response_definition.validate_headers!(response)
-                }.not_to raise_error
-              end
-            end
+          it 'a regexp header should raise error if it does not match the regexp' do
+            response_definition.header('X-Header', /anotherthing/)
+            expect {
+              response_definition.validate_headers!(response)
+            }.to raise_error(Praxis::Exceptions::Validation)
           end
         end
       end
@@ -478,7 +382,10 @@ describe Praxis::ResponseDefinition do
       if parts || parts_block
         parts ? response.parts(nil, **parts, &parts_block) : response.parts(nil, &parts_block) 
       end
-      response.headers(headers) if headers
+
+      headers&.each do |(name, value)|
+        response.header(name, value)
+      end
     end
 
     context 'for a definition with a media type' do
@@ -520,8 +427,9 @@ describe Praxis::ResponseDefinition do
       its([:location]){ should == {value: location.inspect ,type: :regexp} }
 
       it 'should have a header defined with value and type keys' do
-        expect( output[:headers] ).to have(1).keys
+        expect( output[:headers] ).to have(2).keys
         expect( output[:headers]['Header1'] ).to eq({value: 'Value1' ,type: :string })
+        expect( output[:headers]['Location'] ).to eq({value: "/\\/my\\/url\\//" ,type: :regexp })        
       end
     end
 

data/tasks/thor/templates/generator/example_app/app/v1/concerns/href.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+module V1
+  module Resources
+    module Concerns
+      module Href
+        extend ActiveSupport::Concern
+
+        # Base module where the href concern will grab constants from
+        included do
+          def self.base_module
+            ::V1
+          end
+        end
+
+        module ClassMethods
+          def endpoint_path_template
+            # memoize a templated path for an endpoint, like
+            # /im/contacts/%{id}
+            return @endpoint_path_template if @endpoint_path_template
+
+            path = self.base_module.const_get(:Endpoints).const_get(model.name.split(':').last.pluralize).canonical_path.route.path
+            @endpoint_path_template = path.names.inject(path.to_s) { |p, name| p.sub(':' + name, "%{#{name}}") }
+          end
+        end
+
+        def href
+          format(self.class.endpoint_path_template, id: id)
+        end
+      end
+    end
+  end
+end

data/tasks/thor/templates/generator/example_app/app/v1/resources/base.rb

@@ -1,8 +1,12 @@
 # frozen_string_literal: true
 
+require_relative '../concerns/href'
+
 module V1
   module Resources
     class Base < Praxis::Mapper::Resource
+      include Resources::Concerns::Href
+
       # Base for all V1 resources.
       # Resources withing a single version should have resource mappings separate from other versions
       # and the Mapper::Resource will appropriately maintain different model_maps for each Base classes

data/tasks/thor/templates/generator/example_app/config/environment.rb

@@ -4,7 +4,7 @@ Praxis::Application.configure do |application|
   # Configure the Mapper plugin (if we want to use all the filtering/field_selection extensions)
   application.bootloader.use Praxis::Plugins::MapperPlugin
   # Configure the Pagination plugin (if we want to use all the pagination/ordering extensions)
-  application.bootloader.use Praxis::Plugins::PaginationPlugin, {
+  application.bootloader.use Praxis::Plugins::PaginationPlugin, **{
     # max_items: 500,  # Unlimited by default,
     # default_page_size: 100,
     # paging_default_mode: {by: :id},

data/tasks/thor/templates/generator/scaffold/implementation/resources/item.rb

@@ -18,7 +18,7 @@ module <%= version_module %>
       <%- if action_enabled?(:create) -%>
       def self.create(payload)
         # Assuming the API field names directly map the the model attributes. Massage if appropriate.
-        self.new(model.create(*payload.to_h))
+        self.new(model.create(**payload.to_h))
       end
       <%- end -%>
 
@@ -27,7 +27,7 @@ module <%= version_module %>
         record = model.find_by(id: id)
         return nil unless record
         # Assuming the API field names directly map the the model attributes. Massage if appropriate.
-        record.update(*payload.to_h)
+        record.update(**payload.to_h)
         self.new(record)
       end
       <%- end -%>

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: praxis
 version: !ruby/object:Gem::Version
-  version: 2.0.pre.11
+  version: 2.0.pre.16
 platform: ruby
 authors:
 - Josep M. Blanquer
 - Dane Jensen
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-02-05 00:00:00.000000000 Z
+date: 2021-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -367,7 +367,7 @@ dependencies:
     - - ">"
       - !ruby/object:Gem::Version
         version: '4'
-description:
+description: 
 email:
 - blanquer@gmail.com
 - dane.jensen@gmail.com
@@ -446,6 +446,7 @@ files:
 - lib/praxis/extensions/attribute_filtering/active_record_patches/6_1_plus.rb
 - lib/praxis/extensions/attribute_filtering/filter_tree_node.rb
 - lib/praxis/extensions/attribute_filtering/filtering_params.rb
+- lib/praxis/extensions/attribute_filtering/filters_parser.rb
 - lib/praxis/extensions/attribute_filtering/sequel_filter_query_builder.rb
 - lib/praxis/extensions/field_expansion.rb
 - lib/praxis/extensions/field_selection.rb
@@ -540,6 +541,7 @@ files:
 - spec/praxis/extensions/attribute_filtering/active_record_filter_query_builder_spec.rb
 - spec/praxis/extensions/attribute_filtering/filter_tree_node_spec.rb
 - spec/praxis/extensions/attribute_filtering/filtering_params_spec.rb
+- spec/praxis/extensions/attribute_filtering/filters_parser_spec.rb
 - spec/praxis/extensions/field_expansion_spec.rb
 - spec/praxis/extensions/field_selection/active_record_query_selector_spec.rb
 - spec/praxis/extensions/field_selection/field_selector_spec.rb
@@ -645,6 +647,7 @@ files:
 - tasks/thor/templates/generator/example_app/Rakefile
 - tasks/thor/templates/generator/example_app/app/models/user.rb
 - tasks/thor/templates/generator/example_app/app/v1/concerns/controller_base.rb
+- tasks/thor/templates/generator/example_app/app/v1/concerns/href.rb
 - tasks/thor/templates/generator/example_app/app/v1/controllers/users.rb
 - tasks/thor/templates/generator/example_app/app/v1/resources/base.rb
 - tasks/thor/templates/generator/example_app/app/v1/resources/user.rb
@@ -669,7 +672,7 @@ homepage: https://github.com/praxis/praxis
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -685,7 +688,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.1
 requirements: []
 rubygems_version: 3.1.2
-signing_key:
+signing_key: 
 specification_version: 4
 summary: Building APIs the way you want it.
 test_files: []