Improve skip message for React Router test and keep shared examples

- Keeps the `it_behaves_like "streamed component tests"` to maintain all tests
- Overrides only the one test that doesn't apply with a clear skip message
- The skip message now explains WHY the test doesn't apply to React Router
- Fixes RuboCop violations by replacing click_link/click_button with click_on

The "renders without JavaScript" test is kept but skipped because React Router
requires JavaScript for client-side routing to nested routes. This documents
the reason while keeping the test structure intact.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: justin808

react_on_rails_pro/spec/dummy/Gemfile.lock

@@ -22,6 +22,7 @@ PATH
   specs:
     react_on_rails_pro (16.2.0.beta.4)
       addressable
+      async (>= 2.6)
       connection_pool
       execjs (~> 2.9)
       httpx (~> 1.5)
@@ -107,6 +108,12 @@ GEM
       public_suffix (>= 2.0.2, < 7.0)
     amazing_print (1.6.0)
     ast (2.4.2)
+    async (2.27.4)
+      console (~> 1.29)
+      fiber-annotation
+      io-event (~> 1.11)
+      metrics (~> 0.12)
+      traces (~> 0.15)
     base64 (0.2.0)
     benchmark (0.4.0)
     bigdecimal (3.1.9)
@@ -131,6 +138,10 @@ GEM
     coderay (1.1.3)
     concurrent-ruby (1.3.5)
     connection_pool (2.5.0)
+    console (1.33.0)
+      fiber-annotation
+      fiber-local (~> 1.1)
+      json
     coveralls (0.8.23)
       json (>= 1.8, < 3)
       simplecov (~> 0.16.1)
@@ -165,6 +176,9 @@ GEM
     ffi (1.17.0-x86_64-darwin)
     ffi (1.17.0-x86_64-linux-gnu)
     ffi (1.17.0-x86_64-linux-musl)
+    fiber-annotation (0.2.0)
+    fiber-local (1.1.0)
+      fiber-storage
     fiber-storage (1.0.0)
     generator_spec (0.10.0)
       activesupport (>= 3.0.0)
@@ -184,6 +198,7 @@ GEM
     i18n (1.14.7)
       concurrent-ruby (~> 1.0)
     io-console (0.8.0)
+    io-event (1.12.1)
     irb (1.15.1)
       pp (>= 0.6.0)
       rdoc (>= 4.0.0)
@@ -216,6 +231,7 @@ GEM
     marcel (1.0.4)
     matrix (0.4.2)
     method_source (1.1.0)
+    metrics (0.14.0)
     mini_mime (1.1.5)
     mini_portile2 (2.8.8)
     minitest (5.25.4)
@@ -447,6 +463,7 @@ GEM
     tins (1.33.0)
       bigdecimal
       sync
+    traces (0.18.1)
     turbolinks (5.2.1)
       turbolinks-source (~> 5.2)
     turbolinks-source (5.2.0)

react_on_rails_pro/spec/dummy/spec/system/integration_spec.rb

@@ -428,29 +428,14 @@ def change_text_expect_dom_selector(dom_selector, expect_no_change: false)
 end
 
 describe "React Router Sixth Page", :js do
-  subject { page }
-
-  it "renders the component" do
-    visit "/server_router/streaming-server-component"
-    expect(page).to have_text "Header for AsyncComponentsTreeForTesting"
-    expect(page).to have_text "Footer for AsyncComponentsTreeForTesting"
-  end
-
-  it "hydrates the component" do
-    visit "/server_router/streaming-server-component"
-    expect(page.html).to match(/client-bundle[^\"]*.js/)
-    change_text_expect_dom_selector("#ServerComponentRouter-react-component-0")
-  end
+  it_behaves_like "streamed component tests", "/server_router/streaming-server-component",
+                  "#ServerComponentRouter-react-component-0"
 
-  it "doesn't hydrate status component if packs are not loaded" do
-    # visit waits for the page to load, so we ensure that the page is loaded before checking the hydration status
-    visit "/server_router/streaming-server-component?skip_js_packs=true"
-    expect(page).to have_text "HydrationStatus: Streaming server render"
-    expect(page).to have_no_text "HydrationStatus: Hydrated"
-    expect(page).to have_no_text "HydrationStatus: Page loaded"
+  # Override: This test is not applicable for React Router because client-side routing
+  # requires JavaScript to navigate to nested routes like /server_router/streaming-server-component.
+  # The test is kept but skipped to document why it doesn't apply to this scenario.
+  it "renders the page completely on server and displays content on client even without JavaScript",
+     skip: "React Router requires JavaScript for client-side routing to nested routes" do
+    # This test is overridden to skip it with explanation
   end
-
-  # NOTE: The "renders the page completely on server" test is not applicable for React Router
-  # because client-side routing requires JavaScript to navigate to nested routes.
-  # This test only makes sense for direct server-rendered pages like /stream_async_components_for_testing
 end