comfortable_mexican_sofa 2.0.2 → 2.0.3

data/test/models/page_test.rb

@@ -1,4 +1,4 @@
-require_relative '../test_helper'
+require_relative "../test_helper"
 
 class CmsPageTest < ActiveSupport::TestCase
 
@@ -61,13 +61,13 @@ class CmsPageTest < ActiveSupport::TestCase
 
   def test_validation_of_slug
     page = comfy_cms_pages(:child)
-    page.slug = 'slug.with.d0ts-and_things'
+    page.slug = "slug.with.d0ts-and_things"
     assert page.valid?
 
-    page.slug = 'inva lid'
+    page.slug = "inva lid"
     assert page.invalid?
 
-    page.slug = 'acción'
+    page.slug = "acción"
     assert page.valid?
   end
 
@@ -81,24 +81,26 @@ class CmsPageTest < ActiveSupport::TestCase
 
   def test_label_assignment
     page = @site.pages.new(
-      slug:   'test',
+      slug:   "test",
       parent: @page,
       layout: @layout
     )
     assert page.valid?
-    assert_equal 'Test', page.label
+    assert_equal "Test", page.label
   end
 
   def test_create
     assert_count_difference [Comfy::Cms::Page, Comfy::Cms::Fragment] do
-      page = @site.pages.create!(new_params(
-        parent: @page,
-        fragments_attributes: [
-          { identifier: "content",
-            tag:        "text",
-            content:    "test" }
-        ]
-      ))
+      page = @site.pages.create!(
+        new_params(
+          parent: @page,
+          fragments_attributes: [
+            { identifier: "content",
+              tag:        "text",
+              content:    "test" }
+          ]
+        )
+      )
       assert page.is_published?
       assert_equal 1, page.position
     end
@@ -106,14 +108,16 @@ class CmsPageTest < ActiveSupport::TestCase
 
   def test_create_with_file
     assert_count_difference [Comfy::Cms::Page, Comfy::Cms::Fragment, ActiveStorage::Attachment] do
-      page = @site.pages.create!(new_params(
-        parent: @page,
-        fragments_attributes: [{
-          identifier: "test",
-          tag:        "file",
-          files:      [fixture_file_upload("files/image.jpg", "image/jpeg")]
-        }]
-      ))
+      page = @site.pages.create!(
+        new_params(
+          parent: @page,
+          fragments_attributes: [{
+            identifier: "test",
+            tag:        "file",
+            files:      [fixture_file_upload("files/image.jpg", "image/jpeg")]
+          }]
+        )
+      )
       assert_equal 1, page.fragments.count
       assert page.fragments.first.attachments.attached?
     end
@@ -122,17 +126,19 @@ class CmsPageTest < ActiveSupport::TestCase
   def test_create_with_files
     assert_count_difference [Comfy::Cms::Page, Comfy::Cms::Fragment] do
       assert_count_difference [ActiveStorage::Attachment], 2 do
-        page = @site.pages.create!(new_params(
-          parent: @page,
-          fragments_attributes: [{
-            identifier: "test",
-            tag:        "files",
-            files:      [
-              fixture_file_upload("files/image.jpg", "image/jpeg"),
-              fixture_file_upload("files/document.pdf", "application/pdf")
-            ]
-          }]
-        ))
+        page = @site.pages.create!(
+          new_params(
+            parent: @page,
+            fragments_attributes: [{
+              identifier: "test",
+              tag:        "files",
+              files:      [
+                fixture_file_upload("files/image.jpg", "image/jpeg"),
+                fixture_file_upload("files/document.pdf", "application/pdf")
+              ]
+            }]
+          )
+        )
         assert_equal 1, page.fragments.count
         assert page.fragments.first.attachments.attached?
       end
@@ -143,14 +149,16 @@ class CmsPageTest < ActiveSupport::TestCase
     string = "1981-10-04 12:34:56"
     datetime = DateTime.parse(string)
     assert_count_difference [Comfy::Cms::Page, Comfy::Cms::Fragment] do
-      page = @site.pages.create!(new_params(
-        parent: @page,
-        fragments_attributes: [{
-          identifier: "test",
-          tag:        "date_time",
-          datetime:   string
-        }]
-      ))
+      page = @site.pages.create!(
+        new_params(
+          parent: @page,
+          fragments_attributes: [{
+            identifier: "test",
+            tag:        "date_time",
+            datetime:   string
+          }]
+        )
+      )
       frag = page.fragments.first
       assert_equal datetime, frag.datetime
     end
@@ -158,14 +166,16 @@ class CmsPageTest < ActiveSupport::TestCase
 
   def test_create_with_boolean
     assert_count_difference [Comfy::Cms::Page, Comfy::Cms::Fragment] do
-      page = @site.pages.create!(new_params(
-        parent: @page,
-        fragments_attributes: [{
-          identifier: "test",
-          tag:        "checkbox",
-          boolean:    "1"
-        }]
-      ))
+      page = @site.pages.create!(
+        new_params(
+          parent: @page,
+          fragments_attributes: [{
+            identifier: "test",
+            tag:        "checkbox",
+            boolean:    "1"
+          }]
+        )
+      )
       frag = page.fragments.first
       assert frag.boolean
     end
@@ -174,10 +184,10 @@ class CmsPageTest < ActiveSupport::TestCase
   def test_update
     frag = comfy_cms_fragments(:default)
     assert_count_no_difference [Comfy::Cms::Page, Comfy::Cms::Fragment] do
-      @page.update_attributes!(fragments_attributes: [{
-        identifier: frag.identifier,
-        content:    "updated content"
-      }])
+      @page.update_attributes!(fragments_attributes: [
+        { identifier: frag.identifier,
+          content:    "updated content" }
+      ])
     end
     frag.reload
     assert_equal "updated content", frag.content
@@ -211,12 +221,12 @@ class CmsPageTest < ActiveSupport::TestCase
   def test_update_with_date
     frag = comfy_cms_fragments(:datetime)
     string = "2020-01-01"
-    date    = DateTime.parse(string)
+    date = DateTime.parse(string)
     assert_count_no_difference [Comfy::Cms::Page, Comfy::Cms::Fragment] do
-      @page.update_attributes!(fragments_attributes: [{
-        identifier: frag.identifier,
-        datetime:   string
-      }])
+      @page.update_attributes!(fragments_attributes: [
+        { identifier: frag.identifier,
+          datetime:   string }
+      ])
     end
     frag.reload
     assert_equal date, frag.datetime
@@ -226,10 +236,10 @@ class CmsPageTest < ActiveSupport::TestCase
     frag = comfy_cms_fragments(:boolean)
     assert frag.boolean
     assert_count_no_difference [Comfy::Cms::Page, Comfy::Cms::Fragment] do
-      @page.update_attributes!(fragments_attributes: [{
-        identifier: frag.identifier,
-        boolean:    "0"
-      }])
+      @page.update_attributes!(fragments_attributes: [
+        { identifier: frag.identifier,
+          boolean:    "0" }
+      ])
     end
     frag.reload
     refute frag.boolean
@@ -239,8 +249,8 @@ class CmsPageTest < ActiveSupport::TestCase
     page_a = @page
     page_b = comfy_cms_pages(:child)
     page_c = @site.pages.create!(
-      label:  'Test Page',
-      slug:   'test-page',
+      label:  "Test Page",
+      slug:   "test-page",
       layout: @layout,
       parent: page_a
     )
@@ -251,7 +261,7 @@ class CmsPageTest < ActiveSupport::TestCase
 
     page_c.update_attributes!(parent_id: page_b)
 
-    page_a.reload; page_b.reload; page_c.reload
+    [page_a, page_b, page_c].each(&:reload)
     assert_equal 1, page_a.children_count
     assert_equal 1, page_b.children_count
     assert_equal 0, page_c.children_count
@@ -259,7 +269,7 @@ class CmsPageTest < ActiveSupport::TestCase
 
   def test_initialization_of_full_path
     page = Comfy::Cms::Page.new
-    assert_equal '/', page.full_path
+    assert_equal "/", page.full_path
 
     page = Comfy::Cms::Page.new(new_params)
     assert page.invalid?
@@ -267,67 +277,67 @@ class CmsPageTest < ActiveSupport::TestCase
 
     page = @site.pages.new(new_params(parent: @page))
     assert page.valid?
-    assert_equal '/test-page', page.full_path
+    assert_equal "/test-page", page.full_path
 
     page = @site.pages.new(new_params(parent: comfy_cms_pages(:child)))
     assert page.valid?
-    assert_equal '/child-page/test-page', page.full_path
+    assert_equal "/child-page/test-page", page.full_path
 
     Comfy::Cms::Page.destroy_all
     page = @site.pages.new(new_params)
     assert page.valid?
-    assert_equal '/', page.full_path
+    assert_equal "/", page.full_path
   end
 
   def test_sync_child_pages
     page = comfy_cms_pages(:child)
-    page_1 = @site.pages.create!(new_params(parent: page, slug: 'test-page-1'))
-    page_2 = @site.pages.create!(new_params(parent: page, slug: 'test-page-2'))
-    page_3 = @site.pages.create!(new_params(parent: page_2, slug: 'test-page-3'))
-    page_4 = @site.pages.create!(new_params(parent: page_1, slug: 'test-page-4'))
-
-    assert_equal '/child-page/test-page-1', page_1.full_path
-    assert_equal '/child-page/test-page-2', page_2.full_path
-    assert_equal '/child-page/test-page-2/test-page-3', page_3.full_path
-    assert_equal '/child-page/test-page-1/test-page-4', page_4.full_path
-
-    page.update_attributes!(slug: 'updated-page')
-    assert_equal '/updated-page', page.full_path
-    page_1.reload; page_2.reload; page_3.reload; page_4.reload
-    assert_equal '/updated-page/test-page-1', page_1.full_path
-    assert_equal '/updated-page/test-page-2', page_2.full_path
-    assert_equal '/updated-page/test-page-2/test-page-3', page_3.full_path
-    assert_equal '/updated-page/test-page-1/test-page-4', page_4.full_path
-
-    page_2.update_attributes!(parent: page_1)
-    page_1.reload; page_2.reload; page_3.reload; page_4.reload
-    assert_equal '/updated-page/test-page-1', page_1.full_path
-    assert_equal '/updated-page/test-page-1/test-page-2', page_2.full_path
-    assert_equal '/updated-page/test-page-1/test-page-2/test-page-3', page_3.full_path
-    assert_equal '/updated-page/test-page-1/test-page-4', page_4.full_path
+    page_a = @site.pages.create!(new_params(parent: page, slug: "test-page-1"))
+    page_b = @site.pages.create!(new_params(parent: page, slug: "test-page-2"))
+    page_c = @site.pages.create!(new_params(parent: page_b, slug: "test-page-3"))
+    page_d = @site.pages.create!(new_params(parent: page_a, slug: "test-page-4"))
+
+    assert_equal "/child-page/test-page-1", page_a.full_path
+    assert_equal "/child-page/test-page-2", page_b.full_path
+    assert_equal "/child-page/test-page-2/test-page-3", page_c.full_path
+    assert_equal "/child-page/test-page-1/test-page-4", page_d.full_path
+
+    page.update_attributes!(slug: "updated-page")
+    assert_equal "/updated-page", page.full_path
+    [page_a, page_b, page_c, page_d].each(&:reload)
+    assert_equal "/updated-page/test-page-1", page_a.full_path
+    assert_equal "/updated-page/test-page-2", page_b.full_path
+    assert_equal "/updated-page/test-page-2/test-page-3", page_c.full_path
+    assert_equal "/updated-page/test-page-1/test-page-4", page_d.full_path
+
+    page_b.update_attributes!(parent: page_a)
+    [page_a, page_b, page_c, page_d].each(&:reload)
+    assert_equal "/updated-page/test-page-1", page_a.full_path
+    assert_equal "/updated-page/test-page-1/test-page-2", page_b.full_path
+    assert_equal "/updated-page/test-page-1/test-page-2/test-page-3", page_c.full_path
+    assert_equal "/updated-page/test-page-1/test-page-4", page_d.full_path
   end
 
   def test_children_count_updating
-    page_1 = @page
-    page_2 = comfy_cms_pages(:child)
-    assert_equal 1, page_1.children_count
-    assert_equal 0, page_2.children_count
+    page_a = @page
+    page_b = comfy_cms_pages(:child)
+    assert_equal 1, page_a.children_count
+    assert_equal 0, page_b.children_count
 
-    page_3 = @site.pages.create!(new_params(parent: page_2))
-    page_1.reload; page_2.reload
-    assert_equal 1, page_1.children_count
-    assert_equal 1, page_2.children_count
-    assert_equal 0, page_3.children_count
+    page_c = @site.pages.create!(new_params(parent: page_b))
+    [page_a, page_b].each(&:reload)
+    assert_equal 1, page_a.children_count
+    assert_equal 1, page_b.children_count
+    assert_equal 0, page_c.children_count
 
-    page_3.update_attributes!(:parent => page_1)
-    page_1.reload; page_2.reload
-    assert_equal 2, page_1.children_count
-    assert_equal 0, page_2.children_count
+    page_c.update_attributes!(parent: page_a)
+    [page_a, page_b].each(&:reload)
+    assert_equal 2, page_a.children_count
+    assert_equal 0, page_b.children_count
 
-    page_3.destroy
-    page_1.reload; page_2.reload
-    assert_equal 1, page_1.children_count
-    assert_equal 0, page_2.children_count
+    page_c.destroy
+    [page_a, page_b].each(&:reload)
+    assert_equal 1, page_a.children_count
+    assert_equal 0, page_b.children_count
   end
 
   def test_cascading_destroy
@@ -341,16 +351,16 @@ class CmsPageTest < ActiveSupport::TestCase
   end
 
   def test_options_for_select
-    assert_equal ['Default Page', '. . Child Page'],
-      Comfy::Cms::Page.options_for_select(@site).collect{|t| t.first }
-    assert_equal ['Default Page'],
-      Comfy::Cms::Page.options_for_select(@site, comfy_cms_pages(:child)).collect{|t| t.first }
+    assert_equal ["Default Page", ". . Child Page"],
+      Comfy::Cms::Page.options_for_select(site: @site).collect(&:first)
+    assert_equal ["Default Page"],
+      Comfy::Cms::Page.options_for_select(site: @site, page: comfy_cms_pages(:child)).collect(&:first)
     assert_equal [],
-      Comfy::Cms::Page.options_for_select(@site, @page)
+      Comfy::Cms::Page.options_for_select(site: @site, page: @page)
 
     page = Comfy::Cms::Page.new(new_params(parent: @page))
-    assert_equal ['Default Page', '. . Child Page'],
-      Comfy::Cms::Page.options_for_select(@site, page).collect{|t| t.first }
+    assert_equal ["Default Page", ". . Child Page"],
+      Comfy::Cms::Page.options_for_select(site: @site, page: page).collect(&:first)
   end
 
   def test_fragments_attributes
@@ -440,7 +450,7 @@ class CmsPageTest < ActiveSupport::TestCase
   def test_content_caching
     assert_equal @page.content_cache, @page.render
 
-    @page.update_columns(content_cache: 'Old Content')
+    @page.update_columns(content_cache: "Old Content")
     refute_equal @page.content_cache, @page.render
 
     @page.clear_content_cache!
@@ -448,7 +458,7 @@ class CmsPageTest < ActiveSupport::TestCase
   end
 
   def test_content_cache_clear_on_save
-    old_content = 'Old Content'
+    old_content = "Old Content"
     @page.update_columns(content_cache: old_content)
 
     @page.save!
@@ -467,74 +477,74 @@ class CmsPageTest < ActiveSupport::TestCase
   end
 
   def test_url
-    assert_equal '//test.host/', @page.url
-    assert_equal '//test.host/child-page', comfy_cms_pages(:child).url
+    assert_equal "//test.host/", @page.url
+    assert_equal "//test.host/child-page", comfy_cms_pages(:child).url
 
-    assert_equal '/', @page.url(relative: true)
-    assert_equal '/child-page', comfy_cms_pages(:child).url(relative: true)
+    assert_equal "/", @page.url(relative: true)
+    assert_equal "/child-page", comfy_cms_pages(:child).url(relative: true)
 
-    @site.update_columns(path: '/en/site')
+    @site.update_columns(path: "/en/site")
     @page.reload
     comfy_cms_pages(:child).reload
 
-    assert_equal '//test.host/en/site/', @page.url
-    assert_equal '//test.host/en/site/child-page', comfy_cms_pages(:child).url
+    assert_equal "//test.host/en/site/", @page.url
+    assert_equal "//test.host/en/site/child-page", comfy_cms_pages(:child).url
 
-    assert_equal '/en/site/', @page.url(relative: true)
-    assert_equal '/en/site/child-page', comfy_cms_pages(:child).url(relative: true)
+    assert_equal "/en/site/", @page.url(relative: true)
+    assert_equal "/en/site/child-page", comfy_cms_pages(:child).url(relative: true)
   end
 
   def test_url_with_public_cms_path
-    ComfortableMexicanSofa.config.public_cms_path = '/custom'
-    assert_equal '//test.host/custom/', @page.url
-    assert_equal '//test.host/custom/child-page', comfy_cms_pages(:child).url
+    ComfortableMexicanSofa.config.public_cms_path = "/custom"
+    assert_equal "//test.host/custom/", @page.url
+    assert_equal "//test.host/custom/child-page", comfy_cms_pages(:child).url
 
-    assert_equal '/custom/', @page.url(relative: true)
-    assert_equal '/custom/child-page', comfy_cms_pages(:child).url(relative: true)
+    assert_equal "/custom/", @page.url(relative: true)
+    assert_equal "/custom/child-page", comfy_cms_pages(:child).url(relative: true)
   end
 
   def test_unicode_slug_escaping
     page = comfy_cms_pages(:child)
-    page_1 = @site.pages.create!(new_params(parent: page, slug: 'tést-ünicode-slug'))
-    assert_equal CGI::escape('tést-ünicode-slug'), page_1.slug
-    assert_equal CGI::escape('/child-page/tést-ünicode-slug').gsub('%2F', '/'), page_1.full_path
+    page_a = @site.pages.create!(new_params(parent: page, slug: "tést-ünicode-slug"))
+    assert_equal CGI.escape("tést-ünicode-slug"), page_a.slug
+    assert_equal CGI.escape("/child-page/tést-ünicode-slug").gsub("%2F", "/"), page_a.full_path
   end
 
   def test_unicode_slug_unescaping
     page = comfy_cms_pages(:child)
-    page_1 = @site.pages.create!(new_params(parent: page, slug: 'tést-ünicode-slug'))
-    found_page = @site.pages.where(slug: CGI::escape('tést-ünicode-slug')).first
-    assert_equal 'tést-ünicode-slug', found_page.slug
-    assert_equal '/child-page/tést-ünicode-slug', found_page.full_path
+    @site.pages.create!(new_params(parent: page, slug: "tést-ünicode-slug"))
+    found_page = @site.pages.where(slug: CGI.escape("tést-ünicode-slug")).first
+    assert_equal "tést-ünicode-slug", found_page.slug
+    assert_equal "/child-page/tést-ünicode-slug", found_page.full_path
   end
 
   def test_identifier
-    assert_equal 'index',       @page.identifier
-    assert_equal 'child-page',  comfy_cms_pages(:child).identifier
+    assert_equal "index",       @page.identifier
+    assert_equal "child-page",  comfy_cms_pages(:child).identifier
 
-    @page.update_column(:slug, 'index')
-    assert_equal 'index', comfy_cms_pages(:default).identifier
+    @page.update_column(:slug, "index")
+    assert_equal "index", comfy_cms_pages(:default).identifier
   end
 
   def test_children_count_updating_on_move
-    page_1 = @page
-    page_2 = comfy_cms_pages(:child)
-    page_3 = @site.pages.create!(new_params(parent: page_2))
+    page_a = @page
+    page_b = comfy_cms_pages(:child)
+    page_c = @site.pages.create!(new_params(parent: page_b))
 
-    page_2.reload
+    page_b.reload
 
-    assert_equal 1, page_1.children_count
-    assert_equal 1, page_2.children_count
-    assert_equal 0, page_3.children_count
+    assert_equal 1, page_a.children_count
+    assert_equal 1, page_b.children_count
+    assert_equal 0, page_c.children_count
 
-    page_3.parent_id = page_1.id
-    page_3.save!
+    page_c.parent_id = page_a.id
+    page_c.save!
 
-    page_1.reload; page_2.reload; page_3.reload
+    [page_a, page_b, page_c].each(&:reload)
 
-    assert_equal 2, page_1.children_count
-    assert_equal 0, page_2.children_count
-    assert_equal 0, page_3.children_count
+    assert_equal 2, page_a.children_count
+    assert_equal 0, page_b.children_count
+    assert_equal 0, page_c.children_count
   end
 
   def test_translate
@@ -545,8 +555,8 @@ class CmsPageTest < ActiveSupport::TestCase
     assert @page.readonly?
 
     assert_equal comfy_cms_layouts(:nested), @page.layout
-    assert_equal 'Default Translation', @page.label
-    assert_equal 'Translation Content', @page.content_cache
+    assert_equal "Default Translation", @page.label
+    assert_equal "Translation Content", @page.content_cache
   end
 
   def test_translate_with_unpublished
@@ -561,4 +571,5 @@ class CmsPageTest < ActiveSupport::TestCase
       @page.translate!(:es)
     end
   end
+
 end

data/test/models/site_test.rb

@@ -15,7 +15,7 @@ class CmsSiteTest < ActiveSupport::TestCase
   def test_validation
     site = Comfy::Cms::Site.new
     assert site.invalid?
-    assert_has_errors_on site, [:identifier, :label, :hostname]
+    assert_has_errors_on site, :identifier, :label, :hostname
 
     site = Comfy::Cms::Site.new(identifier: "test", hostname: "http://site.host")
     assert site.invalid?
@@ -140,24 +140,24 @@ class CmsSiteTest < ActiveSupport::TestCase
   end
 
   def test_find_site_with_public_cms_path
-    ComfortableMexicanSofa.config.public_cms_path = '/custom'
-    assert_equal '//test.host/custom', comfy_cms_sites(:default).url
+    ComfortableMexicanSofa.config.public_cms_path = "/custom"
+    assert_equal "//test.host/custom", comfy_cms_sites(:default).url
 
-    site_a = Comfy::Cms::Site.create!(identifier: 'test_a', hostname: 'test2.host', path: 'en')
-    site_b = Comfy::Cms::Site.create!(identifier: 'test_b', hostname: 'test2.host', path: 'fr')
+    site_a = Comfy::Cms::Site.create!(identifier: "test_a", hostname: "test2.host", path: "en")
+    site_b = Comfy::Cms::Site.create!(identifier: "test_b", hostname: "test2.host", path: "fr")
 
-    assert_nil            Comfy::Cms::Site.find_site('test2.host')
-    assert_nil            Comfy::Cms::Site.find_site('test2.host', '/custom/some/path')
-    assert_equal site_a,  Comfy::Cms::Site.find_site('test2.host', '/custom/en')
-    assert_equal site_a,  Comfy::Cms::Site.find_site('test2.host', '/custom/en?a=b')
-    assert_equal site_a,  Comfy::Cms::Site.find_site('test2.host', '/custom/en/some/path?a=b')
+    assert_nil            Comfy::Cms::Site.find_site("test2.host")
+    assert_nil            Comfy::Cms::Site.find_site("test2.host", "/custom/some/path")
+    assert_equal site_a,  Comfy::Cms::Site.find_site("test2.host", "/custom/en")
+    assert_equal site_a,  Comfy::Cms::Site.find_site("test2.host", "/custom/en?a=b")
+    assert_equal site_a,  Comfy::Cms::Site.find_site("test2.host", "/custom/en/some/path?a=b")
 
-    assert_nil            Comfy::Cms::Site.find_site('test2.host', '/custom/english/some/path')
+    assert_nil            Comfy::Cms::Site.find_site("test2.host", "/custom/english/some/path")
 
-    assert_equal site_b,  Comfy::Cms::Site.find_site('test2.host', '/custom/fr')
-    assert_equal site_b,  Comfy::Cms::Site.find_site('test2.host', '/custom/fr?a=b')
-    assert_equal site_b,  Comfy::Cms::Site.find_site('test2.host', '/custom/fr/some/path')
-    assert_equal site_b,  Comfy::Cms::Site.find_site('test2.host', '/custom/fr/some/path?a=b')
+    assert_equal site_b,  Comfy::Cms::Site.find_site("test2.host", "/custom/fr")
+    assert_equal site_b,  Comfy::Cms::Site.find_site("test2.host", "/custom/fr?a=b")
+    assert_equal site_b,  Comfy::Cms::Site.find_site("test2.host", "/custom/fr/some/path")
+    assert_equal site_b,  Comfy::Cms::Site.find_site("test2.host", "/custom/fr/some/path?a=b")
   end
 
   def test_find_site_with_site_alias
@@ -166,7 +166,7 @@ class CmsSiteTest < ActiveSupport::TestCase
 
     ComfortableMexicanSofa.config.hostname_aliases = {
       "test.host"   => "alias_a.host",
-      "test2.host"  => %w(alias_b.host alias_c.host)
+      "test2.host"  => %w[alias_b.host alias_c.host]
     }
 
     assert_equal site_a, Comfy::Cms::Site.find_site("alias_a.host")
@@ -186,4 +186,5 @@ class CmsSiteTest < ActiveSupport::TestCase
 
     assert_equal "/cms/site-path", @site.url(relative: true)
   end
+
 end

data/test/models/snippet_test.rb

@@ -52,4 +52,5 @@ class CmsSnippetTest < ActiveSupport::TestCase
     page.reload
     assert_equal "new snippet content", page.content_cache
   end
+
 end

data/test/models/translation_test.rb

@@ -1,4 +1,4 @@
-require_relative '../test_helper'
+require_relative "../test_helper"
 
 class CmsPageTest < ActiveSupport::TestCase
 
@@ -57,4 +57,5 @@ class CmsPageTest < ActiveSupport::TestCase
     comfy_cms_translations(:default).update_columns(is_published: false)
     assert_equal 0, Comfy::Cms::Translation.published.count
   end
+
 end

data/test/tasks/cms_seeds_test.rb

@@ -12,7 +12,7 @@ class CmsSeedsTaskTest < ActiveSupport::TestCase
   end
 
   def test_import
-    importer = mock()
+    importer = mock
     ComfortableMexicanSofa::Seeds::Importer.expects(:new).with("from_folder", "to_site").returns(importer)
     importer.expects(:import!)
 
@@ -22,7 +22,7 @@ class CmsSeedsTaskTest < ActiveSupport::TestCase
   end
 
   def test_export
-    exporter = mock()
+    exporter = mock
     ComfortableMexicanSofa::Seeds::Exporter.expects(:new).with("from_site", "to_folder").returns(exporter)
     exporter.expects(:export!)
 
@@ -30,4 +30,5 @@ class CmsSeedsTaskTest < ActiveSupport::TestCase
       @rake["comfy:cms_seeds:export"].invoke("from_site", "to_folder")
     end
   end
+
 end