azeroth 0.10.1 → 1.1.0

data/spec/lib/azeroth/decorator/method_builder_spec.rb

@@ -3,21 +3,80 @@
 require 'spec_helper'
 
 describe Azeroth::Decorator::MethodBuilder do
+  subject(:build_reader) do
+    described_class.build_reader(decorator_class, :age, options)
+  end
+
   let(:decorator_class) { Class.new(Azeroth::Decorator) }
   let(:decorator)       { decorator_class.new(object) }
   let(:model)           { build(:dummy_model) }
   let(:object)          { model }
+  let(:options)         { Azeroth::Decorator::Options.new(options_hash) }
+  let(:options_hash)    { {} }
 
   describe '.build_reader' do
     it do
-      expect { described_class.build_reader(decorator_class, :age) }
+      expect { build_reader }
         .to add_method(:age).to(decorator)
     end
 
     it do
-      described_class.build_reader(decorator_class, :age)
+      build_reader
 
       expect(decorator.age).to eq(model.age)
     end
+
+    context 'when passing reader option' do
+      context 'when passing true' do
+        let(:options_hash) { { reader: true } }
+
+        it do
+          expect { build_reader }
+            .to add_method(:age).to(decorator)
+        end
+      end
+
+      context 'when passing false' do
+        let(:options_hash) { { reader: false } }
+
+        it do
+          expect { build_reader }
+            .not_to add_method(:age).to(decorator)
+        end
+      end
+    end
+
+    context 'when passing override option as true' do
+      let(:options_hash) { { override: false } }
+
+      it do
+        expect { build_reader }
+          .to add_method(:age).to(decorator)
+      end
+    end
+
+    context 'when method already existed' do
+      before do
+        decorator_class.define_method(:age) { 1 }
+      end
+
+      context 'when passing override option as true' do
+        let(:options_hash) { { override: true } }
+
+        it do
+          expect { build_reader }
+            .to change_method(:age).on(decorator)
+        end
+      end
+
+      context 'when passing override option as false' do
+        let(:options_hash) { { override: false } }
+
+        it do
+          expect { build_reader }
+            .not_to change_method(:age).on(decorator)
+        end
+      end
+    end
   end
 end

data/spec/lib/azeroth/decorator_spec.rb

@@ -5,14 +5,16 @@ require 'spec_helper'
 describe Azeroth::Decorator do
   subject(:decorator) { DummyModel::Decorator.new(object) }
 
-  let(:model)  { build(:dummy_model) }
-  let(:object) { model }
+  let(:model)    { build(:dummy_model) }
+  let(:object)   { model }
+  let(:instance) { decorator.new(object) }
 
   describe '.expose' do
     subject(:decorator) { Class.new(described_class) }
 
+    let(:options_hash) { {} }
     let(:expected_options) do
-      Azeroth::Decorator::Options.new
+      Azeroth::Decorator::Options.new(options_hash)
     end
 
     it do
@@ -59,6 +61,97 @@ describe Azeroth::Decorator do
           .and raise_error(Sinclair::Exception::InvalidOptions)
       end
     end
+
+    context 'when decorator already has the method' do
+      before do
+        decorator.define_method(:name) do
+          'some name'
+        end
+      end
+
+      context 'when not passing override' do
+        it do
+          expect { decorator.send(:expose, :name) }
+            .to change(decorator, :attributes_map)
+            .from({})
+            .to({ name: expected_options })
+        end
+
+        it do
+          expect { decorator.send(:expose, :name) }
+            .to change_method(:name)
+            .on(instance)
+        end
+      end
+
+      context 'when passing override as true' do
+        let(:options_hash) { { override: true } }
+
+        it do
+          expect { decorator.send(:expose, :name, **options_hash) }
+            .to change(decorator, :attributes_map)
+            .from({})
+            .to({ name: expected_options })
+        end
+
+        it do
+          expect { decorator.send(:expose, :name, **options_hash) }
+            .to change_method(:name)
+            .on(instance)
+        end
+      end
+
+      context 'when passing override as false' do
+        let(:options_hash) { { override: false } }
+
+        it do
+          expect { decorator.send(:expose, :name, **options_hash) }
+            .to change(decorator, :attributes_map)
+            .from({})
+            .to({ name: expected_options })
+        end
+
+        it do
+          expect { decorator.send(:expose, :name, **options_hash) }
+            .not_to change_method(:name)
+            .on(instance)
+        end
+      end
+    end
+
+    context 'when passing reader option' do
+      context 'when option is true' do
+        let(:options_hash) { { reader: true } }
+
+        it do
+          expect { decorator.send(:expose, :name, **options_hash) }
+            .to change(decorator, :attributes_map)
+            .from({})
+            .to({ name: expected_options })
+        end
+
+        it do
+          expect { decorator.send(:expose, :name, **options_hash) }
+            .to add_method(:name).to(decorator)
+        end
+      end
+
+      context 'when option is false' do
+        let(:options_hash) { { reader: false } }
+
+        it do
+          expect { decorator.send(:expose, :name, **options_hash) }
+            .to change(decorator, :attributes_map)
+            .from({})
+            .to({ name: expected_options })
+        end
+
+        it do
+          expect { decorator.send(:expose, :name, **options_hash) }
+            .not_to add_method(:name).to(decorator)
+        end
+      end
+    end
   end
 
   describe '#as_json' do

data/spec/lib/azeroth/params_builder_spec.rb

@@ -3,7 +3,9 @@
 require 'spec_helper'
 
 describe Azeroth::ParamsBuilder do
-  subject(:params_builder) { described_class.new(model, builder) }
+  subject(:params_builder) do
+    described_class.new(model: model, builder: builder)
+  end
 
   let(:model)   { Azeroth::Model.new(:document, options) }
   let(:options) { Azeroth::Options.new }

data/spec/lib/azeroth/request_handler/pagination_spec.rb

@@ -0,0 +1,133 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Azeroth::RequestHandler::Pagination do
+  subject(:pagination) { described_class.new(params, options) }
+
+  let(:options)      { Azeroth::Options.new(options_hash) }
+  let(:options_hash) { {} }
+  let(:params)       { ActionController::Parameters.new(parameters) }
+  let(:parameters)   { {} }
+
+  describe '#offset' do
+    context 'when nothing was defined' do
+      it do
+        expect(pagination.offset).to be_zero
+      end
+    end
+
+    context 'when parameters has page value' do
+      let(:parameters)        { { page: page } }
+      let(:page)              { Random.rand(1..10) }
+      let(:expected_offset)   { (page - 1) * expected_per_page }
+      let(:expected_per_page) { 20 }
+
+      it 'returns value from request using default per_page' do
+        expect(pagination.offset).to eq(expected_offset)
+      end
+    end
+
+    context 'when parameters has per_page value' do
+      let(:parameters)        { { per_page: per_page } }
+      let(:per_page)          { Random.rand(1..10) }
+
+      it do
+        expect(pagination.offset).to be_zero
+      end
+    end
+
+    context 'when parameters has page and per page values' do
+      let(:parameters)        { { page: page, per_page: per_page } }
+      let(:page)              { Random.rand(1..10) }
+      let(:per_page)          { Random.rand(1..10) }
+      let(:expected_offset)   { (page - 1) * expected_per_page }
+      let(:expected_per_page) { per_page }
+
+      it 'returns value from request' do
+        expect(pagination.offset).to eq(expected_offset)
+      end
+    end
+
+    context 'when parameters has page and options per page values' do
+      let(:parameters)        { { page: page } }
+      let(:options_hash)      { { per_page: per_page } }
+      let(:page)              { Random.rand(1..10) }
+      let(:per_page)          { Random.rand(1..10) }
+      let(:expected_offset)   { (page - 1) * expected_per_page }
+      let(:expected_per_page) { per_page }
+
+      it 'returns value from request and options' do
+        expect(pagination.offset).to eq(expected_offset)
+      end
+    end
+
+    context 'when params has page and per_page and options per page values' do
+      let(:parameters)        { { page: page, per_page: per_page } }
+      let(:options_hash)      { { per_page: options_per_page } }
+      let(:page)              { Random.rand(1..10) }
+      let(:options_per_page)  { Random.rand(1..10) }
+      let(:per_page)          { Random.rand(1..10) }
+      let(:expected_offset)   { (page - 1) * expected_per_page }
+      let(:expected_per_page) { per_page }
+
+      it 'returns value from request' do
+        expect(pagination.offset).to eq(expected_offset)
+      end
+    end
+  end
+
+  describe '#limit' do
+    context 'when nothing was defined' do
+      it 'returns default value' do
+        expect(pagination.limit).to eq(20)
+      end
+    end
+
+    context 'when parameters has per_page' do
+      let(:parameters) { { per_page: per_page } }
+      let(:per_page)   { Random.rand(1..10) }
+
+      it 'returns value from request' do
+        expect(pagination.limit).to eq(per_page)
+      end
+    end
+
+    context 'when options has per_page' do
+      let(:options_hash) { { per_page: per_page } }
+      let(:per_page)     { Random.rand(1..10) }
+
+      it 'returns value from options' do
+        expect(pagination.limit).to eq(per_page)
+      end
+    end
+
+    context 'when params and options have per_page' do
+      let(:parameters)       { { per_page: per_page } }
+      let(:per_page)         { Random.rand(1..10) }
+      let(:options_hash)     { { per_page: options_per_page } }
+      let(:options_per_page) { Random.rand(1..10) }
+
+      it 'returns value from request' do
+        expect(pagination.limit).to eq(per_page)
+      end
+    end
+  end
+
+  describe '#current_page' do
+    context 'when nothing was defined' do
+      it 'returns first page' do
+        expect(pagination.current_page).to eq(1)
+      end
+    end
+
+    context 'when parameters has page' do
+      let(:parameters) { { page: page } }
+      let(:page)       { Random.rand(1..10) }
+
+      it 'returns value from request' do
+        expect(pagination.current_page).to eq(page)
+      end
+    end
+  end
+end

data/spec/lib/azeroth/resource_builder_spec.rb

@@ -3,7 +3,9 @@
 require 'spec_helper'
 
 describe Azeroth::ResourceBuilder do
-  subject(:resource_builder) { described_class.new(model, builder) }
+  subject(:resource_builder) do
+    described_class.new(model: model, builder: builder)
+  end
 
   let(:model)   { Azeroth::Model.new(:document, options) }
   let(:options) { Azeroth::Options.new }

data/spec/lib/azeroth/routes_builder_spec.rb

@@ -4,7 +4,9 @@ require 'spec_helper'
 
 describe Azeroth::RoutesBuilder do
   subject(:routes_builder) do
-    described_class.new(model, builder, options)
+    described_class.new(
+      model: model, builder: builder, options: options
+    )
   end
 
   let(:controller) { controller_class.new }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: azeroth
 version: !ruby/object:Gem::Version
-  version: 0.10.1
+  version: 1.1.0
 platform: ruby
 authors:
 - Darthjee
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-08-27 00:00:00.000000000 Z
+date: 2024-05-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.2.x
+        version: 7.0.x
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.2.x
+        version: 7.0.x
 - !ruby/object:Gem::Dependency
   name: darthjee-active_ext
   requirement: !ruby/object:Gem::Requirement
@@ -58,56 +58,56 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.6.7
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.6.7
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
   name: actionpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 5.2.8.1
+        version: 7.0.4.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 5.2.8.1
+        version: 7.0.4.3
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 5.2.8.1
+        version: 7.0.4.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 5.2.8.1
+        version: 7.0.4.3
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.3.14
+        version: 2.4.8
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.3.14
+        version: 2.4.8
 - !ruby/object:Gem::Dependency
   name: factory_bot
   requirement: !ruby/object:Gem::Requirement
@@ -142,14 +142,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.13.6
+        version: 1.13.8
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.13.6
+        version: 1.13.8
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
@@ -184,14 +184,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 5.2.8.1
+        version: 7.0.4.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 5.2.8.1
+        version: 7.0.4.3
 - !ruby/object:Gem::Dependency
   name: rails-controller-testing
   requirement: !ruby/object:Gem::Requirement
@@ -402,6 +402,20 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 0.21.2
+- !ruby/object:Gem::Dependency
+  name: sprockets-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.4.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.4.2
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -486,6 +500,7 @@ files:
 - lib/azeroth.rb
 - lib/azeroth/controller_interface.rb
 - lib/azeroth/decorator.rb
+- lib/azeroth/decorator/class_methods.rb
 - lib/azeroth/decorator/hash_builder.rb
 - lib/azeroth/decorator/key_value_extractor.rb
 - lib/azeroth/decorator/method_builder.rb
@@ -500,6 +515,7 @@ files:
 - lib/azeroth/request_handler/edit.rb
 - lib/azeroth/request_handler/index.rb
 - lib/azeroth/request_handler/new.rb
+- lib/azeroth/request_handler/pagination.rb
 - lib/azeroth/request_handler/show.rb
 - lib/azeroth/request_handler/update.rb
 - lib/azeroth/resource_builder.rb
@@ -564,6 +580,9 @@ files:
 - spec/dummy/app/models/product/decorator_with_factory.rb
 - spec/dummy/app/models/publisher.rb
 - spec/dummy/app/models/user.rb
+- spec/dummy/app/models/website.rb
+- spec/dummy/app/models/website/decorator.rb
+- spec/dummy/app/models/website/with_location.rb
 - spec/dummy/app/views/documents/edit.html
 - spec/dummy/app/views/documents/index.html
 - spec/dummy/app/views/documents/new.html
@@ -634,6 +653,7 @@ files:
 - spec/lib/azeroth/request_handler/edit_spec.rb
 - spec/lib/azeroth/request_handler/index_spec.rb
 - spec/lib/azeroth/request_handler/new_spec.rb
+- spec/lib/azeroth/request_handler/pagination_spec.rb
 - spec/lib/azeroth/request_handler/show_spec.rb
 - spec/lib/azeroth/request_handler/update_spec.rb
 - spec/lib/azeroth/request_handler_spec.rb
@@ -679,7 +699,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.20
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Azeroth
@@ -740,6 +760,9 @@ test_files:
 - spec/dummy/app/models/product/decorator_with_factory.rb
 - spec/dummy/app/models/publisher.rb
 - spec/dummy/app/models/user.rb
+- spec/dummy/app/models/website.rb
+- spec/dummy/app/models/website/decorator.rb
+- spec/dummy/app/models/website/with_location.rb
 - spec/dummy/app/views/documents/edit.html
 - spec/dummy/app/views/documents/index.html
 - spec/dummy/app/views/documents/new.html
@@ -810,6 +833,7 @@ test_files:
 - spec/lib/azeroth/request_handler/edit_spec.rb
 - spec/lib/azeroth/request_handler/index_spec.rb
 - spec/lib/azeroth/request_handler/new_spec.rb
+- spec/lib/azeroth/request_handler/pagination_spec.rb
 - spec/lib/azeroth/request_handler/show_spec.rb
 - spec/lib/azeroth/request_handler/update_spec.rb
 - spec/lib/azeroth/request_handler_spec.rb