Fix current path detection for locale URL parameter

app/components/tab_navigation_component.rb

@@ -8,8 +8,10 @@ def initialize(label:, routes:, **tag_options)
   end
 
   def is_current_path?(path)
-    request.path == URI.parse(path).path
-  rescue URI::InvalidURIError
+    recognized_path = Rails.application.routes.recognize_path(path)
+    request[:controller] == recognized_path[:controller] &&
+      request[:action] == recognized_path[:action]
+  rescue ActionController::RoutingError
     false
   end
 end

spec/components/tab_navigation_component_spec.rb

@@ -24,8 +24,17 @@
   end
 
   context 'with link for current request' do
-    before do
-      allow(vc_test_request).to receive(:path).and_return('/first')
+    let(:request_path) { '/first' }
+
+    around do |example|
+      Rails.application.routes.draw do
+        get '(:example_param)/first' => 'application#first'
+        get '(:example_param)/second' => 'application#second'
+      end
+
+      with_request_url(request_path) { example.run }
+
+      Rails.application.reload_routes!
     end
 
     it 'renders current link as highlighted' do
@@ -50,8 +59,22 @@
     context 'with routes including query parameters' do
       let(:routes) do
         [
-          { path: '/first?foo=bar', text: 'First' },
-          { path: '/second?foo=bar', text: 'Second' },
+          { path: '/first?example_param=example_param_value', text: 'First' },
+          { path: '/second?example_param=example_param_value', text: 'Second' },
+        ]
+      end
+
+      it 'renders current link as highlighted' do
+        expect(rendered).to have_link('First') { |link| is_current_link?(link) }
+        expect(rendered).to have_link('Second') { |link| !is_current_link?(link) }
+      end
+    end
+
+    context 'with equivalent routes based on param' do
+      let(:routes) do
+        [
+          { path: '/example_param_value/first', text: 'First' },
+          { path: '/example_param_value/second', text: 'Second' },
         ]
       end