feat(count): add deactivate count support (#547)

Author: nicolasalexandre9

app/controllers/forest_liana/resources_controller.rb

@@ -8,9 +8,13 @@ class ResourcesController < ForestLiana::ApplicationController
     rescue_from ActiveRecord::RecordNotFound, :with => :record_not_found
 
     if Rails::VERSION::MAJOR < 4
-      before_filter :find_resource
+      before_filter :find_resource, except: :count
     else
-      before_action :find_resource
+      before_action :find_resource, except: :count
+    end
+
+    def deactivate_count_response
+      render serializer: nil, json: { meta: { count: 'deactivated '} }
     end
 
     def index
@@ -54,6 +58,7 @@ def index
     end
 
     def count
+      find_resource
       begin
         checker = ForestLiana::PermissionsChecker.new(
           @resource,

spec/dummy/config/routes.rb

@@ -1,6 +1,7 @@
 Rails.application.routes.draw do
   namespace :forest do
     post '/actions/test' => 'islands#test'
+    get '/Owner/count' , to: 'owners#count'
   end
 
   mount ForestLiana::Engine => "/forest"

spec/dummy/lib/forest_liana/controllers/owners_controller.rb

@@ -0,0 +1,10 @@
+class Forest::OwnersController < ForestLiana::ResourcesController
+  def count
+    if (params[:search])
+      deactivate_count_response
+    else
+      params[:collection] = 'Owner'
+      super
+    end
+  end
+end

spec/requests/count_spec.rb

@@ -0,0 +1,79 @@
+require 'rails_helper'
+
+describe 'Requesting Owner', :type => :request  do
+  before(:each) do
+    1.upto(10) do |i|
+     Owner.create(name: "Owner #{i}")
+    end
+  end
+
+  after(:each) do
+    Owner.destroy_all
+  end
+
+  before(:each) do
+    allow(ForestLiana::IpWhitelist).to receive(:retrieve) { true }
+    allow(ForestLiana::IpWhitelist).to receive(:is_ip_whitelist_retrieved) { true }
+    allow(ForestLiana::IpWhitelist).to receive(:is_ip_valid) { true }
+
+    allow_any_instance_of(ForestLiana::PermissionsChecker).to receive(:is_authorized?) { true }
+
+    allow(ForestLiana::ScopeManager).to receive(:fetch_scopes).and_return({})
+  end
+
+  token = JWT.encode({
+                       id: 38,
+                       email: '[email protected]',
+                       first_name: 'Michael',
+                       last_name: 'Kelso',
+                       team: 'Operations',
+                       rendering_id: 16,
+                       exp: Time.now.to_i + 2.weeks.to_i
+                     }, ForestLiana.auth_secret, 'HS256')
+
+  headers = {
+    'Accept' => 'application/json',
+    'Content-Type' => 'application/json',
+    'Authorization' => "Bearer #{token}"
+  }
+  describe 'count' do
+    params = {
+      fields: { 'Owner' => 'id,name' },
+      page: { 'number' => '1', 'size' => '10' },
+      searchExtended: '0',
+      sort: '-id',
+      timezone: 'Europe/Paris'
+    }
+
+    it 'should respond 200' do
+      get '/forest/Owner/count', params: params, headers: headers
+      expect(response.status).to eq(200)
+    end
+
+    it 'should equal to 10' do
+      get '/forest/Owner/count', params: params, headers: headers
+      expect(response.status).to eq(200)
+    end
+  end
+
+  describe 'deactivate_count_response' do
+    params = {
+      fields: { 'Product' => 'id,name' },
+      page: { 'number' => '1', 'size' => '10' },
+      search: 'foo',
+      searchExtended: '0',
+      sort: '-id',
+      timezone: 'Europe/Paris'
+    }
+
+    it 'should respond 200' do
+      get '/forest/Owner/count', params: params, headers: headers
+      expect(response.status).to eq(200)
+    end
+
+    it 'should equal to deactivated response' do
+      get '/forest/Owner/count', params: params, headers: headers
+      expect(response.body).to eq('{"meta":{"count":"deactivated "}}')
+    end
+  end
+end