refinerycms-blog 2.0.5 → 2.1.0

data/spec/{requests → features}/refinery/blog/admin/comments_spec.rb

@@ -4,11 +4,11 @@ module Refinery
   module Blog
     module Admin
       describe Comment do
-        login_refinery_user
+        refinery_login_with :refinery_user
 
         describe "#index" do
           context "when has no new unapproved comments" do
-            before(:each) do
+            before do
               subject.class.delete_all
               visit refinery.blog_admin_comments_path
             end
@@ -21,7 +21,7 @@ module Refinery
           end
           context "when has new unapproved comments" do
             let!(:blog_comment) { FactoryGirl.create(:blog_comment) }
-            before(:each) { visit refinery.blog_admin_comments_path }
+            before { visit refinery.blog_admin_comments_path }
 
             it "should list comments" do
               page.should have_content(blog_comment.body)
@@ -44,7 +44,7 @@ module Refinery
 
         describe "#approved" do
           context "when has no approved comments" do
-            before(:each) do
+            before do
               subject.class.delete_all
               visit refinery.approved_blog_admin_comments_path
             end
@@ -57,7 +57,7 @@ module Refinery
             let!(:blog_comment) do
               FactoryGirl.create(:blog_comment, :state => 'approved')
             end
-            before(:each) { visit refinery.approved_blog_admin_comments_path }
+            before { visit refinery.approved_blog_admin_comments_path }
 
             it "should list comments" do
               page.should have_content(blog_comment.body)
@@ -74,7 +74,7 @@ module Refinery
 
         describe "#rejected" do
           context "when has no rejected comments" do
-            before(:each) do
+            before do
               subject.class.delete_all
               visit refinery.rejected_blog_admin_comments_path
             end
@@ -87,7 +87,7 @@ module Refinery
             let!(:blog_comment) do
               FactoryGirl.create(:blog_comment, :state => 'rejected')
             end
-            before(:each) { visit refinery.rejected_blog_admin_comments_path }
+            before { visit refinery.rejected_blog_admin_comments_path }
 
             it "should list comments" do
               page.should have_content(blog_comment.body)
@@ -104,7 +104,7 @@ module Refinery
 
         describe "#show" do
           let!(:blog_comment) { FactoryGirl.create(:blog_comment) }
-          before(:each) { visit refinery.blog_admin_comment_path(blog_comment) }
+          before { visit refinery.blog_admin_comment_path(blog_comment) }
           it "should display the comment" do
             page.should have_content(blog_comment.body)
             page.should have_content(blog_comment.name)
@@ -118,4 +118,4 @@ module Refinery
       end
     end
   end
-end
+end

data/spec/{requests → features}/refinery/blog/admin/menu_spec.rb

@@ -1,8 +1,8 @@
 require 'spec_helper'
 
 describe "Blog menu entry" do
-  login_refinery_user
-
+  refinery_login_with :refinery_user
+  
   it "is highlighted when managing the blog" do
     visit refinery.admin_root_path
 

data/spec/features/refinery/blog/admin/posts_spec.rb

@@ -0,0 +1,338 @@
+# encoding: utf-8
+require "spec_helper"
+
+module Refinery
+  module Blog
+    module Admin
+      describe Post do
+        refinery_login_with :refinery_user
+
+        let!(:blog_category) do
+          Globalize.with_locale(:en) { FactoryGirl.create(:blog_category) }
+        end
+
+        context "when no blog posts" do
+          before { subject.class.destroy_all }
+
+          describe "blog post listing" do
+            before { visit refinery.blog_admin_posts_path }
+
+            it "invites to create new post" do
+              page.should have_content("There are no Blog Posts yet. Click \"Create new post\" to add your first blog post.")
+            end
+          end
+
+          describe "new blog post form" do
+            before do
+              visit refinery.blog_admin_posts_path
+              click_link "Create new post"
+            end
+
+            it "should have Tags" do
+              page.should have_content("Tags")
+            end
+
+            it "should have category title", :js => true do
+              click_link "toggle_advanced_options"
+              page.should have_content(blog_category.title)
+            end
+
+            describe "create blog post", :js => true do
+              before do
+                fill_in "post_title", :with => "This is my blog post"
+                # this is a dirty hack but textarea that needs to be filled is
+                # hidden and capybara refuses to fill in elements it can't see
+                page.evaluate_script("WYMeditor.INSTANCES[0].html('<p>And I love it</p>')")
+                click_link "toggle_advanced_options"
+                check blog_category.title
+                click_button "Save"
+              end
+
+              it "should succeed" do
+                page.should have_content("was successfully added.")
+              end
+
+              it "should be the only blog post" do
+                subject.class.count.should eq(1)
+              end
+
+              it "should belong to me" do
+                subject.class.first.author.should eq(::Refinery::User.last)
+              end
+
+              it "should save categories" do
+                subject.class.last.categories.count.should eq(1)
+                subject.class.last.categories.first.title.should eq(blog_category.title)
+              end
+            end
+
+            describe "create blog post with tags", :js => true do
+              before do
+                @tag_list = "chicago, bikes, beers, babes"
+                fill_in "Title", :with => "This is a tagged blog post"
+                # this is a dirty hack but textarea that needs to be filled is
+                # hidden and capybara refuses to fill in elements it can't see
+                page.evaluate_script("WYMeditor.INSTANCES[0].html('<p>And I also love it</p>')")
+                fill_in "Tags", :with => @tag_list
+                click_button "Save"
+              end
+
+              it "should succeed" do
+                page.should have_content("was successfully added.")
+              end
+
+              it "should be the only blog post" do
+                subject.class.count.should eq(1)
+              end
+
+              it "should have the specified tags" do
+                subject.class.last.tag_list.sort.should eq(@tag_list.split(', ').sort)
+              end
+            end
+          end
+        end
+
+        context "when has blog posts" do
+          let!(:blog_post) do
+            Globalize.with_locale(:en) { FactoryGirl.create(:blog_post) }
+          end
+
+          describe "blog post listing" do
+            before { visit refinery.blog_admin_posts_path }
+
+            describe "edit blog post" do
+              it "should succeed" do
+                page.should have_content(blog_post.title)
+
+                click_link("Edit this blog post")
+                current_path.should == refinery.edit_blog_admin_post_path(blog_post)
+
+                fill_in "post_title", :with => "hax0r"
+                click_button "Save"
+
+                page.should_not have_content(blog_post.title)
+                page.should have_content("'hax0r' was successfully updated.")
+              end
+            end
+
+            describe "deleting blog post" do
+              it "should succeed" do
+                page.should have_content(blog_post.title)
+
+                click_link "Remove this blog post forever"
+
+                page.should have_content("'#{blog_post.title}' was successfully removed.")
+              end
+            end
+
+            describe "view live" do
+              it "redirects to blog post in the frontend" do
+                click_link "View this blog post live"
+
+                current_path.should == refinery.blog_post_path(blog_post)
+                page.should have_content(blog_post.title)
+              end
+            end
+          end
+
+          context "when uncategorized post" do
+            it "shows up in the list" do
+              visit refinery.uncategorized_blog_admin_posts_path
+              page.should have_content(blog_post.title)
+            end
+          end
+
+          context "when categorized post" do
+            it "won't show up in the list" do
+              blog_post.categories << blog_category
+              blog_post.save!
+
+              visit refinery.uncategorized_blog_admin_posts_path
+              page.should_not have_content(blog_post.title)
+            end
+          end
+        end
+
+        context "with multiple users" do
+          let!(:other_guy) { FactoryGirl.create(:refinery_user, :username => "Other Guy") }
+
+          describe "create blog post with alternate author", :js => true do
+            before do
+              visit refinery.blog_admin_posts_path
+              click_link "Create new post"
+
+              fill_in "post_title", :with => "This is some other guy's blog post"
+              # this is a dpage_titleirty hack but textarea that needs to be filled is
+              # hidden and capybara refuses to fill in elements it can't see
+              page.evaluate_script("WYMeditor.INSTANCES[0].html('<p>I totally did not write it.</p>')")
+
+              click_link "toggle_advanced_options"
+
+              select other_guy.username, :from => "Author"
+
+              click_button "Save"
+            end
+
+            it "should succeed" do
+              page.should have_content("was successfully added.")
+            end
+
+            it "belongs to another user" do
+              subject.class.last.author.should eq(other_guy)
+            end
+          end
+        end
+
+        context "with translations" do
+          before do
+            Globalize.locale = :en
+            Refinery::I18n.stub(:frontend_locales).and_return([:en, :ru])
+            blog_page = FactoryGirl.create(:page, :link_url => "/blog", :title => "Blog")
+            Globalize.with_locale(:ru) do
+              blog_page.title = 'блог'
+              blog_page.save
+            end
+            visit refinery.blog_admin_posts_path
+          end
+
+          describe "add a blog post with title for default locale" do
+            before do
+              click_link "Create new post"
+              fill_in "Title", :with => "Post"
+              fill_in "post_body", :with => "One post in my blog"
+              click_button "Save"
+              @p = Refinery::Blog::Post.find_by_title("Post")
+            end
+
+            it "succeeds" do
+              page.should have_content("'Post' was successfully added.")
+              Refinery::Blog::Post.count.should eq(1)
+            end
+
+            it "shows locale flag for post" do
+
+              within "#post_#{@p.id}" do
+                page.should have_css("img[src='/assets/refinery/icons/flags/en.png']")
+              end
+            end
+
+            it "shows up in blog page for default locale" do
+              visit refinery.blog_root_path
+              page.should have_selector("#post_#{@p.id}")
+            end
+
+            it "does not show up in blog page for secondary locale" do
+              visit refinery.blog_root_path(:locale => :ru)
+              page.should_not have_selector("#post_#{@p.id}")
+            end
+
+          end
+
+          describe "add a blog post with title only for secondary locale" do
+
+            let(:ru_page_title) { 'Новости' }
+
+            before do
+              click_link "Create new post"
+              within "#switch_locale_picker" do
+                click_link "Ru"
+              end
+              fill_in "Title", :with => ru_page_title
+              fill_in "post_body", :with => "One post in my blog"
+              click_button "Save"
+              @p = Refinery::Blog::Post.find_by_title("Новости")
+            end
+
+            it "succeeds" do
+              page.should have_content("was successfully added.")
+              Refinery::Blog::Post.count.should eq(1)
+            end
+
+            it "shows title in secondary locale" do
+              within "#post_#{@p.id}" do
+                page.should have_content(ru_page_title)
+              end
+            end
+
+            it "shows locale flag for post" do
+              within "#post_#{@p.id}" do
+                page.should have_css("img[src='/assets/refinery/icons/flags/ru.png']")
+              end
+            end
+
+            it "does not show locale flag for primary locale" do
+              within "#post_#{@p.id}" do
+                page.should_not have_css("img[src='/assets/refinery/icons/flags/en.png']")
+              end
+            end
+
+            it "does not show up in blog page for default locale" do
+              visit refinery.blog_root_path
+              page.should_not have_selector("#post_#{@p.id}")
+            end
+
+            it "shows up in blog page for secondary locale" do
+              visit refinery.blog_root_path(:locale => :ru)
+              page.should have_selector("#post_#{@p.id}")
+            end
+
+          end
+
+          context "with a blog post in both locales" do
+
+            let!(:blog_post) do
+              _blog_post = Globalize.with_locale(:en) { FactoryGirl.create(:blog_post, :title => 'First Post') }
+              Globalize.with_locale(:ru) do
+                _blog_post.title = 'Домашняя страница'
+                _blog_post.save
+              end
+              _blog_post
+            end
+
+            before do
+              visit refinery.blog_admin_posts_path
+            end
+
+            it "shows both locale flags for post" do
+              within "#post_#{blog_post.id}" do
+                page.should have_css("img[src='/assets/refinery/icons/flags/en.png']")
+                page.should have_css("img[src='/assets/refinery/icons/flags/ru.png']")
+              end
+            end
+
+            describe "edit the post in english" do
+              it "succeeds" do
+
+                within "#post_#{blog_post.id}" do
+                  click_link("En")
+                end
+                current_path.should == refinery.edit_blog_admin_post_path(blog_post)
+                fill_in "Title", :with => "New Post Title"
+                click_button "Save"
+
+                page.should_not have_content(blog_post.title)
+                page.should have_content("'New Post Title' was successfully updated.")
+              end
+            end
+
+            describe "edit the post in secondary locale" do
+              it "succeeds" do
+                within "#post_#{blog_post.id}" do
+                  click_link("Ru")
+                end
+
+                fill_in "Title", :with => "Нов"
+                click_button "Save"
+
+                page.should_not have_content(blog_post.title)
+                page.should have_content("'Нов' was successfully updated.")
+              end
+            end
+
+          end
+        end
+
+      end
+    end
+  end
+end

data/spec/features/refinery/blog/categories_spec.rb

@@ -0,0 +1,28 @@
+require "spec_helper"
+
+module Refinery
+  describe "BlogCategories" do
+    refinery_login_with :refinery_user
+
+    context "has one category and post" do
+      before do
+        post = Globalize.with_locale(:en) do
+          FactoryGirl.create(:blog_post, :title => "Refinery CMS blog post")
+        end
+        @category = Globalize.with_locale(:en) do
+          FactoryGirl.create(:blog_category, :title => "Video Games")
+        end
+        post.categories << @category
+        post.save!
+      end
+
+      describe "show categories blog posts" do
+        it "should displays categories blog posts" do
+          visit refinery.blog_category_path(@category)
+          page.should have_content("Refinery CMS blog post")
+          page.should have_content("Video Games")
+        end
+      end
+    end
+  end
+end

data/spec/{requests → features}/refinery/blog/posts_spec.rb

@@ -2,47 +2,65 @@ require "spec_helper"
 
 module Refinery
   describe "Blog::Posts" do
-    login_refinery_user
-  
-    context "when has blog posts" do    
-      let!(:blog_post) { FactoryGirl.create(:blog_post, :title => "Refinery CMS blog post") }
-    
+    refinery_login_with :refinery_user
+
+    context "when has blog posts" do
+      let!(:blog_post) do
+        Globalize.with_locale(:en) { FactoryGirl.create(:blog_post, :title => "Refinery CMS blog post") }
+      end
+
       it "should display blog post" do
         visit refinery.blog_post_path(blog_post)
-        
+
         page.should have_content(blog_post.title)
       end
-    
-      it "should display the blog rss feed" do
-        get refinery.blog_rss_feed_path
-        
-        response.should be_success
-        response.content_type.should eq("application/rss+xml")
+
+      describe "visit blog" do
+
+        before do
+          FactoryGirl.create(:page, :link_url => "/")
+          FactoryGirl.create(:page, :link_url => "/blog", :title => "Blog")
+        end
+
+        it "shows blog link in menu" do
+          visit "/"
+          within "#menu" do
+            page.should have_content("Blog")
+            page.should have_selector("a[href='/blog']")
+          end
+        end
+
+        it "shows blog posts" do
+          visit refinery.blog_root_path
+          page.should have_content blog_post.title
+        end
+
       end
+
     end
-      
-    describe "list tagged posts" do    
-      context "when has tagged blog posts" do      
-        before(:each) do
+
+    describe "list tagged posts" do
+      context "when has tagged blog posts" do
+        before do
           @tag_name = "chicago"
           @post = FactoryGirl.create(:blog_post,
                                           :title => "I Love my city",
                                           :tag_list => @tag_name)
-          @tag = ::Refinery::Blog::Post.tag_counts_on(:tags).first          
+          @tag = ::Refinery::Blog::Post.tag_counts_on(:tags).first
         end
         it "should have one tagged post" do
           visit refinery.blog_tagged_posts_path(@tag.id, @tag_name.parameterize)
-          
+
           page.should have_content(@tag_name)
           page.should have_content(@post.title)
         end
       end
     end
-    
+
     describe "#show" do
       context "when has no comments" do
         let(:blog_post) { FactoryGirl.create(:blog_post) }
-        
+
         it "should display the blog post" do
           visit refinery.blog_post_path(blog_post)
           page.should have_content(blog_post.title)
@@ -51,35 +69,35 @@ module Refinery
       end
       context "when has approved comments" do
         let(:approved_comment) { FactoryGirl.create(:approved_comment) }
-        
+
         it "should display the comments" do
           visit refinery.blog_post_path(approved_comment.post)
-          
+
           page.should have_content(approved_comment.body)
           page.should have_content("Posted by #{approved_comment.name}")
         end
       end
       context "when has rejected comments" do
         let(:rejected_comment) { FactoryGirl.create(:rejected_comment) }
-        
-        it "should not display the comments" do          
+
+        it "should not display the comments" do
           visit refinery.blog_post_path(rejected_comment.post)
-          
+
           page.should_not have_content(rejected_comment.body)
         end
       end
       context "when has new comments" do
         let(:blog_comment) { FactoryGirl.create(:blog_comment) }
-        
+
         it "should not display the comments" do
           visit refinery.blog_post_path(blog_comment.post)
-          
+
           page.should_not have_content(blog_comment.body)
         end
       end
 
       context "when posting comments" do
-        let(:blog_post) { Factory(:blog_post) }
+        let(:blog_post) { FactoryGirl.create(:blog_post) }
         let(:name) { "pete" }
         let(:email) { "pete@mcawesome.com" }
         let(:body) { "Witty comment." }
@@ -122,16 +140,11 @@ module Refinery
       end
 
       context "post recent" do
-        let(:blog_post) { FactoryGirl.create(:blog_post) }
-        let(:blog_post2) { FactoryGirl.create(:blog_post) }
-
-        before do
-          visit refinery.blog_post_path(blog_post2)
-          visit refinery.blog_post_path(blog_post)
-        end
+        let!(:blog_post) { FactoryGirl.create(:blog_post, :published_at => Time.now - 5.minutes) }
+        let!(:blog_post2) { FactoryGirl.create(:blog_post, :published_at => Time.now - 2.minutes) }
 
         it "should be the most recent" do
-          Refinery::Blog::Post.recent(2).first.should eq(blog_post2)
+          Refinery::Blog::Post.recent(2).first.id.should eq(blog_post2.id)
         end
       end
 
@@ -139,20 +152,27 @@ module Refinery
 
     describe "#show draft preview" do
       let(:blog_post) { FactoryGirl.create(:blog_post_draft) }
-      context "when logged in as admin" do        
+
+      context "when logged in as admin" do
         it "should display the draft notification" do
           visit refinery.blog_post_path(blog_post)
-          
+
           page.should have_content('This page is NOT live for public viewing.')
         end
       end
+
       context "when not logged in as an admin" do
-        before(:each) { visit refinery.destroy_refinery_user_session_path }
-        
+        before do
+          # TODO: remove Refinery::Pages::Engine.load_seed dependency.
+          # It's here to temporary fix the issue with 404 after visiting logout path.
+          Refinery::Pages::Engine.load_seed
+          visit refinery.logout_path
+        end
+
         it "should not display the blog post" do
           visit refinery.blog_post_path(blog_post)
-          
-          page.should have_content("The page you were looking for doesn't exist (404)")
+
+          page.should have_content("The page you requested was not found.")
         end
       end
     end