spree_essential_cms 0.3.0.rc1 → 0.4.0

data/test/dummy_hooks/templates/initializers/spree_user_error_fix.rb

@@ -0,0 +1,3 @@
+unless Spree::User.const_defined?(:DestroyWithOrdersError)
+  class Spree::User::DestroyWithOrdersError < StandardError; end
+end

data/test/dummy_hooks/templates/overrides/main_menu.rb

@@ -0,0 +1,6 @@
+# app/overrides/main_menu.rb
+Deface::Override.new(:virtual_path  => "spree/shared/_store_menu",
+                     :name          => "spree_essential_cms_main_menu_items",
+                     :insert_after  => "#home-link[data-hook]",
+                     :partial       => "spree/shared/main_menu_items",
+                     :disabled      => false)

data/test/integration/spree/admin/contents_integration_test.rb

@@ -0,0 +1,116 @@
+require 'test_helper'
+
+class Spree::Admin::ContentsIntegrationTest < SpreeEssentials::IntegrationCase
+
+  setup do
+    Spree::Content.destroy_all
+    @page = Spree::Page.first || Factory.create(:spree_page)
+  end
+  
+  should "have a link to new content" do
+    visit spree.admin_page_contents_path(@page)
+    btn = find(".actions a.button").native
+    assert_match /#{spree.new_admin_page_content_path(@page)}$/, btn.attribute('href')
+    assert_equal "New Content", btn.text
+  end
+  
+  should "get new content" do  
+    visit spree.new_admin_page_content_path(@page)
+    assert has_content?("New Content")
+    within "#new_content" do
+      assert has_field?("Title")
+      assert has_field?("Page")
+      assert has_field?("Body")   
+      assert has_field?("Context")   
+      assert has_field?("Attachment")   
+      assert has_field?("Link URL")
+      assert has_field?("Link Text")
+      assert has_field?("Hide Title")
+    end
+  end
+    
+  should "validate content" do
+    visit spree.new_admin_page_content_path(@page)
+    click_button "Create"
+    within "#errorExplanation" do
+      assert_seen "1 error prohibited this record from being saved:"
+      assert_seen "Title can't be blank"
+    end
+  end
+  
+  should "create some content" do
+    visit spree.new_admin_page_content_path(@page)
+    within "#new_content" do
+      fill_in "Title", :with => "Just some content"
+      select @page.title, :from => "Page"
+      fill_in "Body",  :with => "Just some words in the content..."
+    end
+    click_button "Create"
+    assert_equal spree.admin_page_contents_path(@page), current_path
+    assert_flash :notice, "Content has been successfully created!"
+  end
+  
+  context "existing content" do    
+    setup do
+      @content = Factory.create(:spree_content, :page => @page)
+    end
+    
+    should "edit and update" do
+      visit spree.edit_admin_page_content_path(@page, @content)      
+      within "#edit_content_#{@content.id}" do
+        fill_in "Title", :with => "Just some content"
+        select @page.title, :from => "Page"
+        fill_in "Body",  :with => "Just some words in the content..."
+      end
+      click_button "Update"
+      assert_equal spree.admin_page_contents_path(@page.reload), current_path
+      assert_flash :notice, "Content has been successfully updated!"
+    end
+    
+    should "not delete current attachment unless checkbox is checked" do
+      @content.update_attribute(:attachment, sample_image)
+      visit spree.edit_admin_page_content_path(@page, @content)
+      click_button "Update"
+      assert !@content.reload.attachment_file_name.blank?
+    end
+    
+    should "delete current attachment" do
+      @content.update_attribute(:attachment, sample_image)
+      visit spree.edit_admin_page_content_path(@page, @content)
+      click_link "Optional Fields"
+      check "Delete current attachment"
+      click_button "Update"
+      assert @content.reload.attachment_file_name.blank?
+    end
+    
+    should "get destroyed" do
+      visit spree.admin_page_contents_path(@page)
+      within "table.index" do
+        find("a[href='#']").click
+      end
+      assert find_by_id("popup_ok").click
+    end
+    
+  end
+  
+  context "several contents" do
+  
+    setup do
+      setup_action_controller_behaviour(Spree::Admin::ContentsController)
+      @contents = Array.new(2) {|i| Factory(:spree_content, :title => "Content ##{i + 1}", :page => @page, :position => i) }
+    end
+    
+    should "update positions" do
+      positions = Hash[@contents.map{|i| [i.id, 2 - i.position ]}]
+      visit spree.admin_page_contents_path(@page)
+      assert_seen "Content #1", :within => "tbody tr:first"
+      assert_seen "Content #2", :within => "tbody tr:last"
+      xhr :post, :update_positions, { :page_id => @page.to_param, :positions => positions }
+      visit spree.admin_page_contents_path(@page)      
+      assert_seen "Content #2", :within => "tbody tr:first"
+      assert_seen "Content #1", :within => "tbody tr:last"
+    end
+  
+  end
+  
+end

data/test/integration/spree/admin/page_images_integration_test.rb

@@ -0,0 +1,94 @@
+require 'test_helper'
+
+class Spree::Admin::PageImagesIntegrationTest < SpreeEssentials::IntegrationCase
+
+  setup do
+    Spree::PageImage.destroy_all
+    @page = Spree::Page.first || Factory.create(:spree_page)
+  end
+  
+  should "have a link to new page image" do
+    visit spree.admin_page_images_path(@page)
+    btn = find("#new_image_link").native
+    assert_match /#{spree.new_admin_page_image_path(@page)}$/, btn.attribute('href')
+    assert_equal "New Image", btn.text
+  end
+  
+  should "get new page image" do  
+    visit spree.new_admin_page_image_path(@page)
+    assert_seen "New Image"
+    within "#new_page_image" do
+      assert has_field?("Attachment")
+      assert has_field?("Alt")   
+    end
+  end
+  
+  should "validate page image" do
+    visit spree.new_admin_page_image_path(@page)
+    click_button "Create"
+    within "#errorExplanation" do
+      assert_seen "2 errors prohibited this record from being saved:"
+      assert_seen "Attachment can't be empty"
+      assert_seen "Attachment file name can't be empty"
+    end
+  end
+  
+  should "create page image" do
+    visit spree.admin_page_images_path(@page)
+    click_link "New Image"
+    within "#new_page_image" do
+      attach_file "Attachment", sample_image_path
+      fill_in "Alt", :with => "alt text!"
+    end
+    click_button "Create"
+    assert_equal spree.admin_page_images_path(@page), current_path
+    assert_flash :notice, "Page image has been successfully created!"
+  end
+  
+  context "existing page image" do    
+    setup do
+      @page_image = Factory.create(:spree_page_image, :viewable => @page)
+    end
+    
+    should "edit and update" do
+      visit spree.edit_admin_page_image_path(@page, @page_image)      
+      within "#edit_page_image_#{@page_image.id}" do
+        attach_file "Attachment", sample_image_path("2.jpg")
+        fill_in "Alt", :with => "omg!"
+      end
+      click_button "Update"
+      assert_equal spree.admin_page_images_path(@page), current_path
+      assert_flash :notice, "Page image has been successfully updated!"
+    end
+    
+    should "get destroyed" do
+      visit spree.admin_page_images_path(@page)
+      within "table.index" do
+        find("a[href='#']").click
+      end
+      assert find_by_id("popup_ok").click
+    end
+    
+  end
+  
+  context "several page images" do
+  
+    setup do
+      setup_action_controller_behaviour(Spree::Admin::PageImagesController)
+      @page_images = Array.new(2) {|i| Factory(:spree_page_image, :alt => "Image ##{i + 1}", :viewable => @page, :position => i) }
+    end
+    
+    should "update positions" do
+      positions = Hash[@page_images.map{|i| [i.id, 2 - i.position ]}]
+      visit spree.admin_page_images_path(@page)
+      assert_seen "Image #1", :within => "tbody tr:first"
+      assert_seen "Image #2", :within => "tbody tr:last"
+      xhr :post, :update_positions, { :page_id => @page.to_param, :positions => positions }
+      visit spree.admin_page_images_path(@page)      
+      assert_seen "Image #2", :within => "tbody tr:first"
+      assert_seen "Image #1", :within => "tbody tr:last"
+    end
+  
+  end
+  
+end

data/test/integration/spree/admin/pages_integration_test.rb

@@ -0,0 +1,131 @@
+require 'test_helper'
+
+class Spree::Admin::PagesIntegrationTest < SpreeEssentials::IntegrationCase
+
+  setup do
+    Spree::Page.destroy_all
+    @labels = %(Title, Meta Title, Meta Description, Meta Keywords, Path, Navigation Title).split(', ')
+    @values = %(Just a page, Super Sweet Page Title, #{Faker::Lorem.paragraph}, one keyword, /some-page, Visit Page).split(', ')
+  end
+  
+  should "get index and have a link to new page" do
+    visit spree.admin_pages_path
+    btn = find(".actions a.button").native
+    assert_match /#{spree.new_admin_page_path}$/, btn.attribute('href')
+    assert_equal "New Page", btn.text
+  end
+  
+  should "get new page" do  
+    visit spree.new_admin_page_path
+    assert has_content?("New Page")
+    within "#new_page" do
+      @labels.each do |f|
+        assert has_field?(f)
+      end
+    end
+  end
+    
+  should "validate page" do
+    visit spree.new_admin_page_path
+    click_button "Create"
+    within "#errorExplanation" do
+      assert_seen "2 errors prohibited this record from being saved:"
+      assert_seen "Title can't be blank"
+      assert_seen "Path can't be blank"
+    end
+  end
+  
+  should "create a page" do
+    visit spree.new_admin_page_path
+    within "#new_page" do
+      @labels.each_with_index do |label, index|
+      	fill_in label, :with => @values[index]      
+      end
+    end
+    click_button "Create"
+    @page = Spree::Page.last
+    assert_equal spree.edit_admin_page_content_path(@page, @page.contents.first), current_path
+    assert_flash :notice, %(Page "Just a page" has been successfully created!)
+  end
+  
+  context "an existing page" do    
+    setup do
+      @page = Factory.create(:spree_page)
+    end
+    
+    should "get show" do
+      visit spree.admin_page_path(@page)
+      assert_seen @page.meta_title
+      assert_seen @page.meta_description
+      assert_seen @page.meta_keywords
+      assert has_link?("Edit")
+      within "#sidebar" do
+        assert has_link?("Contents")
+        assert has_link?("Images")
+      end
+    end
+    
+    should "edit and update" do
+      visit spree.edit_admin_page_path(@page)
+      
+      within "#edit_page_#{@page.id}" do
+        @labels.each_with_index do |label, index|
+        	fill_in label, :with => @values[index].reverse      
+        end
+      end
+      click_button "Update"
+      assert_equal spree.admin_page_path(@page.reload), current_path
+      assert_flash :notice, %(Page "egap a tsuJ" has been successfully updated!)
+    end
+    
+    should "get destroyed" do
+      visit spree.admin_pages_path
+      find("a[href='#']").click
+      assert find_by_id("popup_ok").click
+    end
+    
+  end
+  
+  context "The homepage" do
+  
+    setup do
+      @page = Spree::Page.create(:title => "Home", :meta_title => "Welcome to our homepage!", :path => "/")
+    end
+    
+    should "edit and update" do
+      visit spree.edit_admin_page_path(@page)
+      
+      within "#edit_page_#{@page.id}" do
+        @labels.each_with_index do |label, index|
+          next if label == "Path"
+        	fill_in label, :with => @values[index]      
+        end
+      end
+      click_button "Update"
+      assert_equal spree.admin_page_path(@page.reload), current_path
+      assert_flash :notice, %(Page "Just a page" has been successfully updated!)
+    end
+    
+  end
+  
+  context "several pages" do
+  
+    setup do
+      setup_action_controller_behaviour(Spree::Admin::PagesController)
+      @pages = Array.new(2) {|i| Factory(:spree_page, :title => "Page ##{i + 1}", :position => i) }
+    end
+    
+    should "update positions" do
+      positions = Hash[@pages.map{|i| [i.id, 2 - i.position ]}]
+      visit spree.admin_pages_path
+      assert_seen "Page #1", :within => "tbody tr:first"
+      assert_seen "Page #2", :within => "tbody tr:last"
+      xhr :post, :update_positions, { :page_id => @page.to_param, :positions => positions }
+      visit spree.admin_pages_path
+      assert_seen "Page #2", :within => "tbody tr:first"
+      assert_seen "Page #1", :within => "tbody tr:last"
+    end
+  
+  end
+  
+end

data/test/integration/spree/home_integration_test.rb

@@ -0,0 +1,54 @@
+require 'test_helper'
+
+class Spree::HomeIntegrationTest < SpreeEssentials::IntegrationCase
+  
+  setup do
+    Spree::Page.destroy_all
+    @images = Dir[File.expand_path("../../../../lib/tasks/sample", __FILE__) + "/*.jpg"]
+  end
+  
+  should "redirect to products when no homepage is present" do
+    visit "/"
+    assert_equal spree.products_path, current_path
+  end
+  
+  context "an existing homepage" do
+  
+    setup do
+      @home = Spree::Page.create(:title => "Home", :meta_title => "Welcome to our homepage!", :path => "/")
+      @home.contents.first.update_attributes(:body => "This is a test", :context => "main")
+      @home.contents.create(:title => "Some might say...", :body => "This is another test", :context => "intro")
+      @images.each { |image| 
+        image = File.open(image)
+        @home.images.create(:attachment => image, :alt => "Sailing", :viewable => @home)
+        image.close
+      }
+      visit "/"
+    end
+    
+    should "have proper page title" do
+      assert_title "Spree Demo Site - Welcome to our homepage!"      
+    end
+    
+    should "have proper contents" do
+      within ".left .content-main" do
+        assert_seen "Home", :within => "h1.title"
+        assert_seen "This is a test", :within => "p"
+      end
+      within ".intro .content-main" do
+        assert_seen "Some might say...", :within => "h1.title"
+        assert_seen "This is another test", :within => "p"
+      end
+    end
+        
+    should "have a images in slideshow" do
+      within "#content .slideshow" do
+        @home.images.each do |img|
+          assert has_xpath?("//img[@src='#{img.attachment.url(:slide)}']")
+        end
+      end
+    end
+    
+  end
+   
+end

data/test/integration/spree/pages_integration_test.rb

@@ -0,0 +1,122 @@
+require 'test_helper'
+
+class Spree::PagesIntegrationTest < SpreeEssentials::IntegrationCase
+
+  setup do
+    Spree::Page.destroy_all
+    @images = Dir[File.expand_path("../../../../lib/tasks/sample", __FILE__) + "/*.jpg"]
+  end
+
+  context "any other page" do
+
+    setup do
+      @page = Spree::Page.create(:title => "Some Page", :meta_description => "This is the description", :meta_keywords => "just, a, keyword", :path => "/some-page")
+    end
+
+    should "have proper meta tags" do
+      visit @page.path
+      assert_title "Spree Demo Site - Some Page"
+      assert_meta :description, "This is the description"
+      assert_meta :keywords, "just, a, keyword"
+    end
+
+    context "with content that doesn't have an image" do
+
+      setup do
+        @page.contents.first.update_attributes(:body => "OMG it really is a page")
+      end
+
+      should "have proper content" do
+        visit @page.path
+        within ".content-main" do
+          assert_seen "Some Page", :within => "h1.title"
+          assert_seen "OMG it really is a page", :within => "p"
+        end
+      end
+
+      should "hide title is specified" do
+        @page.contents.first.update_attributes(:hide_title => true)
+        visit @page.path
+        within ".content-main" do
+          assert !has_selector?("h1.title")
+          assert_seen "OMG it really is a page", :within => "p"
+        end
+      end
+
+    end
+
+
+    context "with content that has an image" do
+
+      setup do
+        @content = @page.contents.first
+        @content.update_attributes(:body => "OMG it really is a page", :attachment => File.open(@images.first))
+      end
+
+      should "have proper content" do
+        visit @page.path
+        within ".content-left" do
+          assert has_xpath?("//img[@src='#{@content.attachment.url(:medium)}']")
+        end
+        within ".content-right" do
+          assert_seen "Some Page", :within => "h1.title"
+          assert_seen "OMG it really is a page", :within => "p"
+        end
+      end
+
+    end
+
+  end
+
+
+  context "with several existing pages" do
+
+    setup do
+      @home_page    = Factory.create(:spree_page, :title => "Home",  :path => "/", :position => 1)
+      @another_home = Factory.create(:spree_page, :title => "Another Home",  :path => "/home", :position => 2)
+      @about_page   = Factory.create(:spree_page, :title => "About", :path => "/about-us", :position => 3)
+      @nested_page  = Factory.create(:spree_page, :title => "Our Services", :path => "/about-us/services", :position => 4)
+    end
+
+    should "get the homepage" do
+      visit "/"
+      assert_title @home_page.title
+    end
+
+    should "get the page called home" do
+      visit "/home"
+      assert_title @another_home.title
+    end
+
+    should "get the about page" do
+      visit "/about-us"
+      assert_title @about_page.title
+    end
+
+    should "get a nested page" do
+      visit "/about-us/services"
+      assert_title @nested_page.title
+    end
+
+    should "render 404" do
+      visit "/a/page/that/doesnt/exist"
+      assert_seen "Error"
+    end
+
+    should "have a proper main menu" do
+      visit "/"
+      Spree::Page.order(:position).all.each do |page|
+        assert_seen page.nav_title, :within => "#main-nav-bar li:nth-child(#{page.position}) a"
+      end
+    end
+
+    should "only have pages marked visible in main menu" do
+      @about_page.update_attribute(:visible, false)
+      visit "/"
+      within "#main-nav-bar" do
+        assert !has_content?(@about_page.nav_title)
+      end
+    end
+
+  end
+end