made changes to allow changing layouts to test turbolinks

cameronnorman · cameronnorman · commit 9fa2a8f96596 · 2020-02-23T16:05:42.000+01:00
diff --git a/app/helpers/matestack/ui/core/application_helper.rb b/app/helpers/matestack/ui/core/application_helper.rb
@@ -26,6 +26,8 @@ def responder_for(page_class, options = {})
           end
           if params[:only_page]
             render html: render_page(page_class, true)
+          elsif params[:layout]
+            render html: render_page(page_class), layout: params[:layout]
           else
             render html: render_page(page_class), layout: true
           end
diff --git a/spec/dummy/app/assets/javascripts/application_with_turbolinks.js b/spec/dummy/app/assets/javascripts/application_with_turbolinks.js
@@ -0,0 +1,25 @@
+// This is a manifest file that'll be compiled into application.js, which will include all the files
+// listed below.
+//
+// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
+// or any plugin's vendor/assets/javascripts directory can be referenced here using a relative path.
+//
+// It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
+// compiled file. JavaScript code in this file should be added after the last require_* statement.
+//
+// Read Sprockets README (https://github.com/rails/sprockets#sprockets-directives) for details
+// about supported directives.
+//
+//= require rails-ujs
+//= require activestorage
+//= require turbolinks
+//= require matestack-ui-core
+//= require_tree .
+
+//= require demo/component
+
+App.cable.subscriptions.create("MatestackUiCoreChannel", {
+  received(data) {
+    MatestackUiCore.matestackEventHub.$emit('MatestackUiCoreChannel', data)
+  }
+});
diff --git a/spec/dummy/app/controllers/example_controller.rb b/spec/dummy/app/controllers/example_controller.rb
@@ -5,4 +5,8 @@ def page
     responder_for(ExamplePage)
   end
 
+  def turbolinks
+    responder_for(ExamplePage, {options: 'turbolinks'})
+  end
+
 end
diff --git a/spec/dummy/app/views/layouts/turbolinks.html.erb b/spec/dummy/app/views/layouts/turbolinks.html.erb
@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <title>Dummy</title>
+    <%= csrf_meta_tags %>
+    <%= csp_meta_tag %>
+
+    <%= stylesheet_link_tag    'application', media: 'all' %>
+    <%= javascript_include_tag 'application_with_turbolinks' %>
+  </head>
+
+  <body>
+    <div id="matestack_ui">
+      <%= yield %>
+    </div>
+  </body>
+</html>
diff --git a/spec/dummy/config/routes.rb b/spec/dummy/config/routes.rb
@@ -18,6 +18,7 @@
   end
 
   get '/example', to: 'example#page'
+  get '/example_turbolinks', to: 'example#turbolinks'
 
   scope :my_app do
     get 'my_first_page', to: 'my_app#my_first_page'
diff --git a/spec/usage/base/turbolinks_spec.rb b/spec/usage/base/turbolinks_spec.rb
@@ -0,0 +1,35 @@
+describe "Turbolinks", type: :feature, js: true do
+  describe "when turbolinks is enabled" do
+    it "enables the main Vue component on turbolinks:ready" do
+      class ExamplePage < Matestack::Ui::Page
+        def response
+          components {
+            heading text: "Hello world"
+          }
+        end
+      end
+
+      visit "/example_turbolinks"
+
+      static_output = page.html
+      expect(static_output).to include("<h1></h1>")
+    end
+  end
+
+  describe "when turbolinks is not enabled" do
+    it "enables the main Vue component on page load" do
+      class ExamplePage < Matestack::Ui::Page
+        def response
+          components {
+            heading text: XSS::EVIL_SCRIPT
+          }
+        end
+      end
+
+      visit "/example"
+
+      static_output = page.html
+      expect(static_output).to include("<h1></h1>")
+    end
+  end
+end
