comfortable_mexican_sofa 2.0.2 → 2.0.3

data/test/controllers/comfy/admin/cms/layouts_controller_test.rb

@@ -1,4 +1,4 @@
-require_relative '../../../../test_helper'
+require_relative "../../../../test_helper"
 
 class Comfy::Admin::Cms::LayoutsControllerTest < ActionDispatch::IntegrationTest
 
@@ -21,7 +21,7 @@ class Comfy::Admin::Cms::LayoutsControllerTest < ActionDispatch::IntegrationTest
     r :get, new_comfy_admin_cms_site_layout_path(site_id: site)
     assert_response :success
     assert assigns(:layout)
-    assert_equal '{{ cms:wysiwyg content }}', assigns(:layout).content
+    assert_equal "{{ cms:wysiwyg content }}", assigns(:layout).content
     assert_template :new
     assert_select "form[action='/admin/sites/#{site.id}/layouts']"
   end
@@ -29,8 +29,8 @@ class Comfy::Admin::Cms::LayoutsControllerTest < ActionDispatch::IntegrationTest
   def test_get_new_with_parent
     site    = comfy_cms_sites(:default)
     layout  = comfy_cms_layouts(:default)
-    layout.update_column(:app_layout, 'application')
-    r :get, new_comfy_admin_cms_site_layout_path(site_id: site), params: {parent_id: layout.id}
+    layout.update_column(:app_layout, "application")
+    r :get, new_comfy_admin_cms_site_layout_path(site_id: site), params: { parent_id: layout.id }
     assert_response :success
     assert_equal layout.app_layout, assigns(:layout).app_layout
   end
@@ -45,76 +45,76 @@ class Comfy::Admin::Cms::LayoutsControllerTest < ActionDispatch::IntegrationTest
   end
 
   def test_get_edit_failure
-    r :get, edit_comfy_admin_cms_site_layout_path(site_id: comfy_cms_sites(:default), id: 'invalid')
+    r :get, edit_comfy_admin_cms_site_layout_path(site_id: comfy_cms_sites(:default), id: "invalid")
     assert_response :redirect
     assert_redirected_to action: :index
-    assert_equal 'Layout not found', flash[:danger]
+    assert_equal "Layout not found", flash[:danger]
   end
 
   def test_creation
-    assert_difference 'Comfy::Cms::Layout.count' do
-      r :post, comfy_admin_cms_site_layouts_path(site_id: comfy_cms_sites(:default)), params: {layout: {
-        label:      'Test Layout',
-        identifier: 'test',
-        content:    'Test {{cms:page:content}}'
-      }}
+    assert_difference "Comfy::Cms::Layout.count" do
+      r :post, comfy_admin_cms_site_layouts_path(site_id: comfy_cms_sites(:default)), params: { layout: {
+        label:      "Test Layout",
+        identifier: "test",
+        content:    "Test {{cms:page:content}}"
+      } }
       assert_response :redirect
       layout = Comfy::Cms::Layout.last
       assert_equal comfy_cms_sites(:default), layout.site
       assert_redirected_to action: :edit, site_id: layout.site, id: layout
-      assert_equal 'Layout created', flash[:success]
+      assert_equal "Layout created", flash[:success]
     end
   end
 
   def test_creation_failure
-    assert_no_difference 'Comfy::Cms::Layout.count' do
-      r :post, comfy_admin_cms_site_layouts_path(site_id: comfy_cms_sites(:default)), params: {layout: {}}
+    assert_no_difference "Comfy::Cms::Layout.count" do
+      r :post, comfy_admin_cms_site_layouts_path(site_id: comfy_cms_sites(:default)), params: { layout: {} }
       assert_response :success
       assert_template :new
-      assert_equal 'Failed to create layout', flash[:danger]
+      assert_equal "Failed to create layout", flash[:danger]
     end
   end
 
   def test_update
     layout = comfy_cms_layouts(:default)
-    r :put, comfy_admin_cms_site_layout_path(site_id: comfy_cms_sites(:default), id: layout), params: {layout: {
-      label:    'New Label',
-      content:  'New {{cms:page:content}}'
-    }}
+    r :put, comfy_admin_cms_site_layout_path(site_id: comfy_cms_sites(:default), id: layout), params: { layout: {
+      label:    "New Label",
+      content:  "New {{cms:page:content}}"
+    } }
     assert_response :redirect
     assert_redirected_to action: :edit, site_id: layout.site, id: layout
-    assert_equal 'Layout updated', flash[:success]
+    assert_equal "Layout updated", flash[:success]
     layout.reload
-    assert_equal 'New Label', layout.label
-    assert_equal 'New {{cms:page:content}}', layout.content
+    assert_equal "New Label", layout.label
+    assert_equal "New {{cms:page:content}}", layout.content
   end
 
   def test_update_failure
     layout = comfy_cms_layouts(:default)
-    r :put, comfy_admin_cms_site_layout_path(site_id: comfy_cms_sites(:default), id: layout), params: {layout: {
-      identifier: ''
-    }}
+    r :put, comfy_admin_cms_site_layout_path(site_id: comfy_cms_sites(:default), id: layout), params: { layout: {
+      identifier: ""
+    } }
     assert_response :success
     assert_template :edit
     layout.reload
-    assert_not_equal '', layout.identifier
-    assert_equal 'Failed to update layout', flash[:danger]
+    assert_not_equal "", layout.identifier
+    assert_equal "Failed to update layout", flash[:danger]
   end
 
   def test_destroy
-    assert_difference 'Comfy::Cms::Layout.count', -1 do
+    assert_difference "Comfy::Cms::Layout.count", -1 do
       r :delete, comfy_admin_cms_site_layout_path(site_id: comfy_cms_sites(:default), id: comfy_cms_layouts(:default))
       assert_response :redirect
       assert_redirected_to action: :index
-      assert_equal 'Layout deleted', flash[:success]
+      assert_equal "Layout deleted", flash[:success]
     end
   end
 
   def test_reorder
     layout_one = comfy_cms_layouts(:default)
     layout_two = comfy_cms_sites(:default).layouts.create!(
-      label:      'test',
-      identifier: 'test'
+      label:      "test",
+      identifier: "test"
     )
     assert_equal 0, layout_one.position
     assert_equal 1, layout_two.position
@@ -129,4 +129,5 @@ class Comfy::Admin::Cms::LayoutsControllerTest < ActionDispatch::IntegrationTest
     assert_equal 1, layout_one.position
     assert_equal 0, layout_two.position
   end
+
 end

data/test/controllers/comfy/admin/cms/pages_controller_test.rb

@@ -1,4 +1,4 @@
-require_relative '../../../../test_helper'
+require_relative "../../../../test_helper"
 
 class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
 
@@ -24,12 +24,12 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
 
   def test_get_index_with_category
     category = @site.categories.create!(
-      label:            'Test Category',
-      categorized_type: 'Comfy::Cms::Page'
+      label:            "Test Category",
+      categorized_type: "Comfy::Cms::Page"
     )
     category.categorizations.create!(categorized: comfy_cms_pages(:child))
 
-    r :get, comfy_admin_cms_site_pages_path(site_id: @site), params: {categories: category.label}
+    r :get, comfy_admin_cms_site_pages_path(site_id: @site), params: { categories: category.label }
     assert_response :success
     assert assigns(:pages)
     assert_equal 1, assigns(:pages).count
@@ -37,7 +37,7 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
   end
 
   def test_get_index_with_category_invalid
-    r :get, comfy_admin_cms_site_pages_path(site_id: @site), params: {categories: "invalid"}
+    r :get, comfy_admin_cms_site_pages_path(site_id: @site), params: { categories: "invalid" }
     assert_response :success
     assert assigns(:pages)
     assert_equal 0, assigns(:pages).count
@@ -45,8 +45,8 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
 
   def test_get_index_with_toggle
     @site.pages.create!(
-      label:  'test',
-      slug:   'test',
+      label:  "test",
+      slug:   "test",
       parent: comfy_cms_pages(:child),
       layout: comfy_cms_layouts(:default)
     )
@@ -55,13 +55,13 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
   end
 
   def test_get_links_with_redactor
-    r :get, comfy_admin_cms_site_pages_path(site_id: @site), params: {source: 'redactor'}
+    r :get, comfy_admin_cms_site_pages_path(site_id: @site), params: { source: "redactor" }
     assert_response :success
 
     assert_equal [
-      {'name' => 'Select page...',  'url' => false},
-      {'name' => 'Default Page',    'url' => '/'},
-      {'name' => '. . Child Page',  'url' => '/child-page'}
+      { "name" => "Select page...",  "url" => false },
+      { "name" => "Default Page",    "url" => "/" },
+      { "name" => ". . Child Page",  "url" => "/child-page" }
     ], JSON.parse(response.body)
   end
 
@@ -220,11 +220,12 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
   end
 
   def test_get_new_with_localized_names
-    I18n.backend.store_translations(:en, {comfy: {cms: {content: {
-      tag:        {localized_a: "Localized Fragment"},
-      namespace:  {localized_a: "Localized Namespace"}
-    }}}})
-    @layout.update_column(:content,
+    I18n.backend.store_translations(:en, comfy: { cms: { content:
+      { tag:        { localized_a: "Localized Fragment" },
+        namespace:  { localized_a: "Localized Namespace" } }
+    } })
+    @layout.update_column(
+      :content,
       "{{cms:text localized_a, namespace: localized_a}}{{cms:text b, namespace: b}}"
     )
     r :get, new_comfy_admin_cms_site_page_path(site_id: @site)
@@ -232,15 +233,14 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
 
     assert_select "a[data-toggle='tab'][href='#ns-localized_a']", "Localized Namespace"
     assert_select "label", "Localized Fragment"
-
   ensure
-    I18n.backend.store_translations(:en, {comfy: {cms: {content: {
-      tag: nil, namespace:  nil
-    }}}})
+    I18n.backend.store_translations(:en, comfy: { cms: { content:
+      { tag: nil, namespace: nil }
+    } })
   end
 
   def test_get_new_as_child_page
-    r :get, new_comfy_admin_cms_site_page_path(site_id: @site), params: {parent_id: @page}
+    r :get, new_comfy_admin_cms_site_page_path(site_id: @site), params: { parent_id: @page }
     assert_response :success
     assert assigns(:page)
     assert_equal comfy_cms_pages(:default), assigns(:page).parent
@@ -257,10 +257,10 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
   end
 
   def test_get_edit_failure
-    r :get, edit_comfy_admin_cms_site_page_path(site_id: @site, id: 'not_found')
+    r :get, edit_comfy_admin_cms_site_page_path(site_id: @site, id: "not_found")
     assert_response :redirect
     assert_redirected_to action: :index
-    assert_equal 'Page not found', flash[:danger]
+    assert_equal "Page not found", flash[:danger]
   end
 
   def test_get_edit_with_blank_layout
@@ -283,34 +283,34 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
   end
 
   def test_creation
-    assert_difference 'Comfy::Cms::Page.count' do
-      assert_difference 'Comfy::Cms::Fragment.count', 2 do
+    assert_difference "Comfy::Cms::Page.count" do
+      assert_difference "Comfy::Cms::Fragment.count", 2 do
         r :post, comfy_admin_cms_site_pages_path(site_id: @site), params: {
           page: {
-            label:              'Test Page',
-            slug:               'test-page',
+            label:              "Test Page",
+            slug:               "test-page",
             parent_id:          @page.id,
             layout_id:          @layout.id,
             fragments_attributes: [
-              { identifier: 'default_page_text',
-                content:    'content content' },
-              { identifier: 'default_field_text',
-                content:    'title content' }
+              { identifier: "default_page_text",
+                content:    "content content" },
+              { identifier: "default_field_text",
+                content:    "title content" }
             ]
           },
-          commit: 'Create Page'
+          commit: "Create Page"
         }
         assert_response :redirect
         page = Comfy::Cms::Page.last
         assert_equal @site, page.site
         assert_redirected_to action: :edit, id: page
-        assert_equal 'Page created', flash[:success]
+        assert_equal "Page created", flash[:success]
       end
     end
   end
 
   def test_creation_with_files
-    assert_difference 'Comfy::Cms::Page.count' do
+    assert_difference "Comfy::Cms::Page.count" do
       assert_difference "Comfy::Cms::Fragment.count", 3 do
         assert_difference "ActiveStorage::Attachment.count", 3 do
           r :post, comfy_admin_cms_site_pages_path(site_id: @site), params: {
@@ -334,94 +334,94 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
                 { identifier: "unpopulated",
                   tag:        "file",
                   content:    nil
-                },
+                }
               ]
             },
-            commit: 'Create Page'
+            commit: "Create Page"
           }
           assert_response :redirect
           page = Comfy::Cms::Page.last
           assert_equal @site, page.site
           assert_redirected_to action: :edit, id: page
-          assert_equal 'Page created', flash[:success]
+          assert_equal "Page created", flash[:success]
         end
       end
     end
   end
 
   def test_creation_failure
-    assert_no_difference ['Comfy::Cms::Page.count', 'Comfy::Cms::Fragment.count'] do
-      r :post, comfy_admin_cms_site_pages_path(site_id: @site), params: {page: {
+    assert_no_difference ["Comfy::Cms::Page.count", "Comfy::Cms::Fragment.count"] do
+      r :post, comfy_admin_cms_site_pages_path(site_id: @site), params: { page: {
         layout_id: @layout.id,
         fragments_attributes: [
-          { identifier: 'content',
-            content:    'content content' },
-          { identifier: 'title',
-            content:    'title content' }
+          { identifier: "content",
+            content:    "content content" },
+          { identifier: "title",
+            content:    "title content" }
         ]
-      }}
+      } }
       assert_response :success
       page = assigns(:page)
 
       assert_equal 2, page.fragments.size
-      assert_equal ['content content', 'title content'], page.fragments.collect{|b| b.content}
+      assert_equal ["content content", "title content"], page.fragments.collect(&:content)
       assert_template :new
-      assert_equal 'Failed to create page', flash[:danger]
+      assert_equal "Failed to create page", flash[:danger]
     end
   end
 
   def test_update
-    assert_no_difference 'Comfy::Cms::Fragment.count' do
-      r :put, comfy_admin_cms_site_page_path(site_id: @site, id: @page), params: {page: {
-        label: 'Updated Label'
-      }}
+    assert_no_difference "Comfy::Cms::Fragment.count" do
+      r :put, comfy_admin_cms_site_page_path(site_id: @site, id: @page), params: { page: {
+        label: "Updated Label"
+      } }
       @page.reload
       assert_response :redirect
       assert_redirected_to action: :edit, id: @page
-      assert_equal 'Page updated', flash[:success]
-      assert_equal 'Updated Label', @page.label
+      assert_equal "Page updated", flash[:success]
+      assert_equal "Updated Label", @page.label
     end
   end
 
   def test_update_with_layout_change
-    assert_difference 'Comfy::Cms::Fragment.count' do
-      r :put, comfy_admin_cms_site_page_path(site_id: @site, id: @page), params: {page: {
-        label:      'Updated Label',
+    assert_difference "Comfy::Cms::Fragment.count" do
+      r :put, comfy_admin_cms_site_page_path(site_id: @site, id: @page), params: { page: {
+        label:      "Updated Label",
         layout_id:  comfy_cms_layouts(:nested).id,
         fragments_attributes: [
-          { identifier: 'content',
-            content:    'new_page_text_content' },
-          { identifier: 'header',
-            content:    'new_page_string_content' }
+          { identifier: "content",
+            content:    "new_page_text_content" },
+          { identifier: "header",
+            content:    "new_page_string_content" }
         ]
-      }}
+      } }
       @page.reload
       assert_response :redirect
       assert_redirected_to action: :edit, id: @page
-      assert_equal 'Page updated', flash[:success]
-      assert_equal 'Updated Label', @page.label
-      identifiers = @page.fragments.collect {|b| b.identifier}
-      assert_equal ["boolean", "content", "datetime", "file", "header"], identifiers.sort
+      assert_equal "Page updated", flash[:success]
+      assert_equal "Updated Label", @page.label
+      identifiers = @page.fragments.collect(&:identifier)
+      assert_equal %w[boolean content datetime file header], identifiers.sort
     end
   end
 
   def test_update_failure
-    r :put, comfy_admin_cms_site_page_path(site_id: @site, id: @page), params: {page: {
+    r :put, comfy_admin_cms_site_page_path(site_id: @site, id: @page), params: { page: {
       label: ""
-    }}
+    } }
     assert_response :success
     assert_template :edit
     assert assigns(:page)
-    assert_equal 'Failed to update page', flash[:danger]
+    assert_equal "Failed to update page", flash[:danger]
   end
 
   def test_destroy
-    assert_difference 'Comfy::Cms::Page.count', -2 do
-      assert_difference 'Comfy::Cms::Fragment.count', -4 do
+    assert_difference "Comfy::Cms::Page.count", -2 do
+      assert_difference "Comfy::Cms::Fragment.count", -4 do
         r :delete, comfy_admin_cms_site_page_path(site_id: @site, id: @page)
         assert_response :redirect
         assert_redirected_to action: :index
-        assert_equal 'Page deleted', flash[:success]
+        assert_equal "Page deleted", flash[:success]
       end
     end
   end
@@ -455,23 +455,23 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
   end
 
   def test_creation_preview
-    assert_no_difference 'Comfy::Cms::Page.count' do
+    assert_no_difference "Comfy::Cms::Page.count" do
       r :post, comfy_admin_cms_site_pages_path(site_id: @site), params: {
-        preview: 'Preview',
+        preview: "Preview",
         page: {
-          label:     'Test Page',
-          slug:      'test-page',
+          label:     "Test Page",
+          slug:      "test-page",
           parent_id: @page.id,
           layout_id: @layout.id,
           fragments_attributes: [
-            { identifier: 'content',
-              content:    'preview content' }
+            { identifier: "content",
+              content:    "preview content" }
           ]
         }
       }
       assert_response :success
-      assert_match /preview content/, response.body
-      assert_equal 'text/html', response.content_type
+      assert_match %r{preview content}, response.body
+      assert_equal "text/html", response.content_type
 
       assert_equal @site, assigns(:cms_site)
       assert_equal @layout, assigns(:cms_layout)
@@ -481,21 +481,21 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
   end
 
   def test_update_preview
-    assert_no_difference 'Comfy::Cms::Page.count' do
+    assert_no_difference "Comfy::Cms::Page.count" do
       r :put, comfy_admin_cms_site_page_path(site_id: @site, id: @page), params: {
-        preview: 'Preview',
+        preview: "Preview",
         page: {
-        label: 'Updated Label',
-        fragments_attributes: [
-          { identifier: 'content',
-            content:    'preview content' }
+          label: "Updated Label",
+          fragments_attributes: [
+            { identifier: "content",
+              content:    "preview content" }
           ]
         }
       }
       assert_response :success
-      assert_match /preview content/, response.body
+      assert_match %r{preview content}, response.body
       @page.reload
-      assert_not_equal 'Updated Label', @page.label
+      assert_not_equal "Updated Label", @page.label
 
       assert_equal @page.site,   assigns(:cms_site)
       assert_equal @page.layout, assigns(:cms_layout)
@@ -504,20 +504,20 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
   end
 
   def test_preview_language
-    @site.update_column(:locale, 'de')
+    @site.update_column(:locale, "de")
 
     assert_equal :en, I18n.locale
 
     r :post, comfy_admin_cms_site_pages_path(site_id: @site), params: {
-      preview: 'Preview',
+      preview: "Preview",
       page: {
-        label:     'Test Page',
-        slug:      'test-page',
+        label:     "Test Page",
+        slug:      "test-page",
         parent_id: @page.id,
         layout_id: @layout.id,
         fragments_attributes: [
-          { identifier: 'content',
-            content:    'preview content' }
+          { identifier: "content",
+            content:    "preview content" }
         ]
       }
     }
@@ -531,7 +531,7 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
     r :get, new_comfy_admin_cms_site_page_path(site_id: @site)
     assert_response :redirect
     assert_redirected_to new_comfy_admin_cms_site_layout_path(@site)
-    assert_equal 'No Layouts found. Please create one.', flash[:danger]
+    assert_equal "No Layouts found. Please create one.", flash[:danger]
   end
 
   def test_get_edit_with_no_layout
@@ -539,15 +539,15 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
     r :get, edit_comfy_admin_cms_site_page_path(site_id: @site, id: @page)
     assert_response :redirect
     assert_redirected_to new_comfy_admin_cms_site_layout_path(@site)
-    assert_equal 'No Layouts found. Please create one.', flash[:danger]
+    assert_equal "No Layouts found. Please create one.", flash[:danger]
   end
 
   def test_get_toggle_branch
-    r :get, toggle_branch_comfy_admin_cms_site_page_path(site_id: @site, id: @page), xhr: true, params: {format: :js}
+    r :get, toggle_branch_comfy_admin_cms_site_page_path(site_id: @site, id: @page), xhr: true, params: { format: :js }
     assert_response :success
     assert_equal [@page.id.to_s], session[:cms_page_tree]
 
-    r :get, toggle_branch_comfy_admin_cms_site_page_path(site_id: @site, id: @page), xhr: true, params: {format: :js}
+    r :get, toggle_branch_comfy_admin_cms_site_page_path(site_id: @site, id: @page), xhr: true, params: { format: :js }
     assert_response :success
     assert_equal [], session[:cms_page_tree]
   end
@@ -557,8 +557,8 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
     page_two = @site.pages.create!(
       parent: @page,
       layout: @layout,
-      label:  'test',
-      slug:   'test'
+      label:  "test",
+      slug:   "test"
     )
     assert_equal 0, page_one.position
     assert_equal 1, page_two.position
@@ -573,4 +573,5 @@ class Comfy::Admin::Cms::PagesControllerTest < ActionDispatch::IntegrationTest
     assert_equal 1, page_one.position
     assert_equal 0, page_two.position
   end
+
 end

data/test/controllers/comfy/admin/cms/revisions/layout_controller_test.rb

@@ -1,4 +1,4 @@
-require_relative '../../../../../test_helper'
+require_relative "../../../../../test_helper"
 
 class Comfy::Admin::Cms::Revisions::LayoutControllerTest < ActionDispatch::IntegrationTest
 
@@ -34,7 +34,7 @@ class Comfy::Admin::Cms::Revisions::LayoutControllerTest < ActionDispatch::Integ
     r :get, comfy_admin_cms_site_layout_revision_path(@site, "invalid", @revision)
     assert_response :redirect
     assert_redirected_to comfy_admin_cms_site_layouts_path(@site)
-    assert_equal 'Record Not Found', flash[:danger]
+    assert_equal "Record Not Found", flash[:danger]
   end
 
   def test_get_show_failure
@@ -42,21 +42,21 @@ class Comfy::Admin::Cms::Revisions::LayoutControllerTest < ActionDispatch::Integ
     assert_response :redirect
     assert assigns(:record)
     assert_redirected_to edit_comfy_admin_cms_site_layout_path(@site, assigns(:record))
-    assert_equal 'Revision Not Found', flash[:danger]
+    assert_equal "Revision Not Found", flash[:danger]
   end
 
-
   def test_revert
-    assert_difference -> {@layout.revisions.count} do
+    assert_difference -> { @layout.revisions.count } do
       r :patch, revert_comfy_admin_cms_site_layout_revision_path(@site, @layout, @revision)
       assert_response :redirect
       assert_redirected_to edit_comfy_admin_cms_site_layout_path(@site, @layout)
-      assert_equal 'Content Reverted', flash[:success]
+      assert_equal "Content Reverted", flash[:success]
 
       @layout.reload
-      assert_equal 'revision {{cms:fragment content}}', @layout.content
-      assert_equal 'revision css', @layout.css
-      assert_equal 'revision js', @layout.js
+      assert_equal "revision {{cms:fragment content}}", @layout.content
+      assert_equal "revision css", @layout.css
+      assert_equal "revision js", @layout.js
     end
   end
+
 end

data/test/controllers/comfy/admin/cms/revisions/page_controller_test.rb

@@ -1,4 +1,4 @@
-require_relative '../../../../../test_helper'
+require_relative "../../../../../test_helper"
 
 class Comfy::Admin::Cms::Revisions::PageControllerTest < ActionDispatch::IntegrationTest
 
@@ -34,24 +34,23 @@ class Comfy::Admin::Cms::Revisions::PageControllerTest < ActionDispatch::Integra
     r :get, comfy_admin_cms_site_page_revision_path(@site, "invalid", @revision)
     assert_response :redirect
     assert_redirected_to comfy_admin_cms_site_pages_path(@site)
-    assert_equal 'Record Not Found', flash[:danger]
+    assert_equal "Record Not Found", flash[:danger]
   end
 
-
   def test_get_show_failure
     r :get, comfy_admin_cms_site_page_revision_path(@site, @page, "invalid")
     assert_response :redirect
     assert assigns(:record)
     assert_redirected_to edit_comfy_admin_cms_site_page_path(@site, assigns(:record))
-    assert_equal 'Revision Not Found', flash[:danger]
+    assert_equal "Revision Not Found", flash[:danger]
   end
 
   def test_revert
-    assert_difference -> {@page.revisions.count} do
+    assert_difference -> { @page.revisions.count } do
       r :patch, revert_comfy_admin_cms_site_page_revision_path(@site, @page, @revision)
       assert_response :redirect
       assert_redirected_to edit_comfy_admin_cms_site_page_path(@site, @page)
-      assert_equal 'Content Reverted', flash[:success]
+      assert_equal "Content Reverted", flash[:success]
 
       @page.reload
 
@@ -84,4 +83,5 @@ class Comfy::Admin::Cms::Revisions::PageControllerTest < ActionDispatch::Integra
       ], @page.fragments_attributes
     end
   end
+
 end