Improve build path for ajax request

To prevent error in build path for ajax request when we have namespaced classes
solves #29 issue

Author: Ivanov-Anton

lib/activeadmin/searchable_select/select_input_extension.rb

@@ -45,11 +45,21 @@ def collection_from_options
 
       def ajax_url
         return unless options[:ajax]
-        template.polymorphic_path([template.active_admin_namespace.route_prefix, ajax_resource_class],
+        template.polymorphic_path([active_admin_prefix, resource_route_name || ajax_resource_class],
                                   action: option_collection.collection_action_name,
                                   **ajax_params)
       end
 
+      def active_admin_prefix
+        template.active_admin_namespace.route_prefix
+      end
+
+      def resource_route_name
+        active_admin_namespace = template.active_admin_namespace
+        str = active_admin_namespace.resource_for(ajax_resource_class)&.route_collection_path
+        str&.split('/')&.last
+      end
+
       def all_options_collection
         option_collection_scope.all.map do |record|
           option_for_record(record)

spec/features/end_to_end_spec.rb

@@ -5,60 +5,100 @@
 require 'support/active_admin_helpers'
 
 RSpec.describe 'end to end', type: :feature, js: true do
-  before(:each) do
-    ActiveAdminHelpers.setup do
-      ActiveAdmin.register(Category) do
-        searchable_select_options(scope: Category, text_attribute: :name)
-      end
+  context 'class name without namespaces' do
+    before(:each) do
+      ActiveAdminHelpers.setup do
+        ActiveAdmin.register(Category) do
+          searchable_select_options(scope: Category, text_attribute: :name)
+        end
 
-      ActiveAdmin.register(Post) do
-        filter(:category, as: :searchable_select, ajax: true)
+        ActiveAdmin.register(Post) do
+          filter(:category, as: :searchable_select, ajax: true)
 
-        form do |f|
-          f.input(:category, as: :searchable_select, ajax: true)
+          form do |f|
+            f.input(:category, as: :searchable_select, ajax: true)
+          end
         end
-      end
 
-      ActiveAdmin.setup {}
+        ActiveAdmin.setup {}
+      end
     end
-  end
 
-  describe 'index page with searchable select filter' do
-    it 'loads filter input options' do
-      Category.create(name: 'Music')
-      Category.create(name: 'Travel')
+    describe 'index page with searchable select filter' do
+      it 'loads filter input options' do
+        Category.create(name: 'Music')
+        Category.create(name: 'Travel')
 
-      visit '/admin/posts'
+        visit '/admin/posts'
 
-      expand_select_box
-      wait_for_ajax
+        expand_select_box
+        wait_for_ajax
 
-      expect(select_box_items).to eq(%w(Music Travel))
-    end
+        expect(select_box_items).to eq(%w(Music Travel))
+      end
 
-    it 'allows filtering options by term' do
-      Category.create(name: 'Music')
-      Category.create(name: 'Travel')
+      it 'allows filtering options by term' do
+        Category.create(name: 'Music')
+        Category.create(name: 'Travel')
 
-      visit '/admin/posts'
+        visit '/admin/posts'
 
-      expand_select_box
-      enter_search_term('T')
-      wait_for_ajax
+        expand_select_box
+        enter_search_term('T')
+        wait_for_ajax
 
-      expect(select_box_items).to eq(%w(Travel))
+        expect(select_box_items).to eq(%w(Travel))
+      end
+
+      it 'loads more items when scrolling down' do
+        15.times { |i| Category.create(name: "Category #{i}") }
+        visit '/admin/posts'
+
+        expand_select_box
+        wait_for_ajax
+        scroll_select_box_list
+        wait_for_ajax
+
+        expect(select_box_items.size).to eq(15)
+      end
     end
+  end
 
-    it 'loads more items when scrolling down' do
-      15.times { |i| Category.create(name: "Category #{i}") }
-      visit '/admin/posts'
+  context 'class name with namespace' do
+    before(:each) do
+      ActiveAdminHelpers.setup do
+        ActiveAdmin.register RGB::Color, as: 'color' do
+          searchable_select_options scope: RGB::Color, text_attribute: :code
+        end
+
+        ActiveAdmin.register Internal::TagName, as: 'Tag Name' do
+          filter :color, as: :searchable_select, ajax: { resource: 'RGB::Color' }
+        end
+
+        ActiveAdmin::SearchableSelect.inline_ajax_options = true
+        ActiveAdmin.setup {}
+      end
+    end
 
-      expand_select_box
-      wait_for_ajax
-      scroll_select_box_list
-      wait_for_ajax
+    describe 'index page with searchable select filter' do
+      let!(:orange) { RGB::Color.create(code: '#eac112', description: 'Orange') }
+      let(:filters_options) { %w(Any #19bf25 #eac112) }
 
-      expect(select_box_items.size).to eq(15)
+      let!(:other_records) do
+        RGB::Color.create(code: '#19bf25', description: 'Green')
+        Internal::TagName.create(name: 'Important Guest', color: orange)
+      end
+
+      before { visit '/admin/tag_names' }
+
+      it 'should have all created colors in filter' do
+        expect(page).to have_select :Color, options: filters_options
+      end
+
+      it 'should have selected option' do
+        select orange.code, from: :Color
+        expect(page).to have_select :Color, selected: orange.code
+      end
     end
   end
 

spec/internal/db/schema.rb

@@ -20,6 +20,17 @@
     t.boolean :published
   end
 
+  create_table(:rgb_colors, force: true) do |t|
+    t.string :code
+    t.text :description
+  end
+
+  create_table(:internal_tag_names, force: true) do |t|
+    t.string :name
+    t.text :description
+    t.integer :color_id
+  end
+
   create_table(:users, force: true) do |t|
     t.string :name
   end

spec/support/models.rb

@@ -13,6 +13,20 @@ class Post < ActiveRecord::Base
   scope(:published, -> { where(published: true) })
 end
 
+module RGB
+  class Color < ActiveRecord::Base
+    self.table_name = :rgb_colors
+    has_many :tags, class_name: 'Internal::TagName'
+  end
+end
+
+module Internal
+  class TagName < ActiveRecord::Base
+    self.table_name = :internal_tag_names
+    belongs_to :color, class_name: 'RGB::Color', foreign_key: :color_id
+  end
+end
+
 RSpec.configure do |config|
   config.after do
     DatabaseCleaner.strategy = :truncation