Merge pull request #38 from GhostGroup/feature/36-namespace-searching

Allow search to accept namespaces and partial matches

lib/faker/bot/reflectors/search.rb

@@ -16,7 +16,7 @@ def self.call(query)
         end
 
         def initialize(query)
-          @query = query
+          @query = query.downcase
 
           super
         end
@@ -31,14 +31,20 @@ def call
         def search_descendants_matching_query
           faker_descendants.each do |descendant|
             methods = descendant.my_singleton_methods
-            matching = methods.select { |method| query_matches?(method.to_s) }
-            store(descendant, matching)
+            if query_matches_class_name?(descendant.to_s)
+              store(descendant, methods)
+            else
+              store(descendant, methods.select { |method| query_matches_method?(method.to_s) })
+            end
           end
         end
 
-        def query_matches?(method_name)
-          method_name_parts = method_name.split(/_/).reject(&:empty?)
-          query.match(/#{method_name_parts.join('|')}/)
+        def query_matches_method?(method_name)
+          method_name.match(/#{query}/)
+        end
+
+        def query_matches_class_name?(class_name)
+          class_name.downcase.match(/#{query}/)
         end
       end
     end

spec/faker/bot/reflectors/search_spec.rb

@@ -4,25 +4,48 @@
 
 RSpec.describe Faker::Bot::Reflectors::Search do
   describe '#call' do
-    context 'when a match is found' do
+    let(:result) { described_class.new(query).call }
+
+    context 'when passing a full method name' do
+      let(:query) { 'first_name' }
+
       it 'it returns the list of matches' do
-        query = 'firstname'
+        expect(result[Faker::Name]).to include(:first_name)
+        expect(result).to be_a(Hash)
+      end
+    end
 
-        reflector = described_class.new(query)
-        result = reflector.call
+    context 'when passing a partial method name' do
+      let(:query) { 'fir' }
 
+      it 'returns the list of matches' do
         expect(result[Faker::Name]).to include(:first_name)
         expect(result).to be_a(Hash)
       end
     end
 
-    context 'when no match is found' do
-      it 'returns an empty hash' do
-        query = 'foobar'
+    context 'when passing a full class name' do
+      let(:query) { 'Beer' }
 
-        reflector = described_class.new(query)
-        result = reflector.call
+      it 'returns the list of matches' do
+        expect(result).to include(Faker::Beer)
+        expect(result).to be_a(Hash)
+      end
+    end
+
+    context 'when passing a partial class name' do
+      let(:query) { 'bee' }
 
+      it 'returns the list of matches' do
+        expect(result).to include(Faker::Beer)
+        expect(result).to be_a(Hash)
+      end
+    end
+
+    context 'when no match is found' do
+      let(:query) { 'firstname'}
+
+      it 'returns an empty hash' do
         expect(result).to be_empty
         expect(result).to be_a(Hash)
       end