Merge pull request #102 from inertiajs/fix-partial-rendering-after-deep-merge-changes

Fix partial reload rendering after deep merge changes

Author: bknoles

lib/inertia_rails/renderer.rb

@@ -45,7 +45,7 @@ def layout
     def computed_props
       _props = ::InertiaRails.shared_data(@controller).send(prop_merge_method, @props).select do |key, prop|
         if rendering_partial_component?
-          key.in? partial_keys
+          key.to_sym.in? partial_keys
         else
           !prop.is_a?(InertiaRails::Lazy)
         end

spec/inertia/rendering_spec.rb

@@ -64,7 +64,7 @@
 
   context 'partial rendering' do
     let (:page) {
-      InertiaRails::Renderer.new('TestComponent', controller, request, response, '', props: { sport: 'hockey'}).send(:page)
+      InertiaRails::Renderer.new('TestComponent', controller, request, response, '', props: { sport: 'hockey' }).send(:page)
     }
     let(:headers) {{
       'X-Inertia' => true,
@@ -76,6 +76,7 @@
       before { get props_path, headers: headers }
 
       it { is_expected.to eq page.to_json }
+      it { is_expected.to include('hockey') }
     end
 
     context 'with a non matching partial component header' do
@@ -107,13 +108,16 @@
       }
       let(:headers) {{
         'X-Inertia' => true,
-        'X-Inertia-Partial-Data' => 'sport,level,grit',
+        'X-Inertia-Partial-Data' => 'sport,level',
         'X-Inertia-Partial-Component' => 'TestComponent',
       }}
 
       before { get lazy_props_path, headers: headers }
 
       it { is_expected.to eq page.to_json }
+      it { is_expected.to include('basketball') }
+      it { is_expected.to include('worse') }
+      it { is_expected.not_to include('intense') }
     end
   end
 end