Merge branch 'explore-topics/case-insensitive-name' into 'master'

Explore topics: find topic by case insensitive name for detail page

See merge request gitlab-org/gitlab!84064

app/controllers/explore/projects_controller.rb

@@ -110,7 +110,7 @@ class Explore::ProjectsController < Explore::ApplicationController
   end
 
   def load_topic
-    @topic = Projects::Topic.find_by_name(params[:topic_name])
+    @topic = Projects::Topic.find_by_name_case_insensitive(params[:topic_name])
   end
 
   # rubocop: disable CodeReuse/ActiveRecord

app/models/projects/topic.rb

@@ -23,6 +23,10 @@ module Projects
     end
 
     class << self
+      def find_by_name_case_insensitive(name)
+        find_by('LOWER(name) = ?', name.downcase)
+      end
+
       def search(query)
         fuzzy_search(query, [:name])
       end

spec/controllers/explore/projects_controller_spec.rb

@@ -112,6 +112,13 @@ RSpec.describe Explore::ProjectsController do
           expect(response).to have_gitlab_http_status(:ok)
           expect(response).to render_template('topic')
         end
+
+        it 'finds topic by case insensitive name' do
+          get :topic, params: { topic_name: 'TOPIC1' }
+
+          expect(response).to have_gitlab_http_status(:ok)
+          expect(response).to render_template('topic')
+        end
       end
     end
   end

spec/models/projects/topic_spec.rb

@@ -56,6 +56,14 @@ RSpec.describe Projects::Topic do
     end
   end
 
+  describe '#find_by_name_case_insensitive' do
+    it 'returns topic with case insensitive name' do
+      %w(topic TOPIC Topic).each do |name|
+        expect(described_class.find_by_name_case_insensitive(name)).to eq(topic)
+      end
+    end
+  end
+
   describe '#search' do
     it 'returns topics with a matching name' do
       expect(described_class.search(topic.name)).to eq([topic])