Fix specs

6 files changed, 287 insertions, 267 deletions

diff --git a/spec/features/refinery/blog/admin/categories_spec.rb b/spec/features/refinery/blog/admin/categories_spec.rb
index 7a8df8c..b1c8fd5 100644
--- a/spec/features/refinery/blog/admin/categories_spec.rb
+++ b/spec/features/refinery/blog/admin/categories_spec.rb
@@ -1,119 +1,121 @@
 # encoding: utf-8
 require 'spec_helper'
 
-describe "Categories admin", type: :feature do
-  refinery_login_with :refinery_user
+module Refinery
+  module Blog
+    module Admin
+      describe Category, type: :feature do
+        refinery_login_with_devise :authentication_devise_refinery_superuser
 
-  let(:title) { "lol" }
+        let(:title) { "lol" }
 
-  it "can create categories" do
-    visit refinery.admin_root_path
+        it "can create categories" do
+          visit refinery.admin_root_path
 
-    within("nav#menu") { click_link "Blog" }
-    within("nav.multilist") { click_link "Create new category" }
+          within("nav#menu") { click_link "Blog" }
+          within("nav.multilist") { click_link "Create new category" }
 
-    fill_in "Title", :with => title
-    click_button "Save"
+          fill_in "Title", :with => title
+          click_button "Save"
 
-    category = Refinery::Blog::Category.first
-    expect(category.title).to eq(title)
-  end
-
-  context "with translations" do
-    before do
-      allow(Refinery::I18n).to receive(:frontend_locales).and_return([:en, :ru])
-      blog_page = Globalize.with_locale(:en) { FactoryGirl.create(:page, :link_url => "/blog", :title => "Blog") }
-      Globalize.with_locale(:ru) do
-        blog_page.title = 'блог'
-        blog_page.save
-      end
-    end
-
-    describe "add a category with title for default locale" do
-      before do
-        Globalize.locale = :en
-        visit refinery.blog_admin_posts_path
-        click_link "Create new category"
-        fill_in "Title", :with => "Testing Category"
-        expect { click_button "Save" }.to change(Refinery::Blog::Category, :count).by(1)
-        @c = Refinery::Blog::Category.by_title("Testing Category")
-      end
-
-      it "suceeds" do
-        expect(page).to have_content("'#{@c.title}' was successfully added.")
-        expect(Refinery::Blog::Category.count).to eq(1)
-      end
-
-      it "shows locale for category" do
-        click_link "Manage"
-        within "#category_#{@c.id}" do
-          expect(page).to have_css(".locale_icon.en")
-        end
-      end
-
-      it "shows up in blog page for default locale" do
-        visit refinery.blog_root_path
-        within "#categories" do
-          expect(page).to have_selector('li')
-        end
-      end
-
-      it "does not show up in blog page for secondary locale" do
-        visit refinery.blog_root_path(:locale => :ru)
-        expect(page).not_to have_selector('#categories')
-      end
-
-    end
-
-    describe "add a category with title for secondary locale" do
-
-      let(:ru_category_title) { 'категория' }
-
-      before do
-        visit refinery.blog_admin_posts_path
-        click_link "Create new category"
-        within "#switch_locale_picker" do
-          click_link "RU"
-        end
-        fill_in "Title", :with => ru_category_title
-        expect { click_button "Save" }.to change(Refinery::Blog::Category, :count).by(1)
-        @c = Refinery::Blog::Category.by_title(ru_category_title)
-      end
-
-      it "suceeds" do
-        expect(page).to have_content("'#{@c.title}' was successfully added.")
-        expect(Refinery::Blog::Category.count).to eq(1)
-      end
-
-      it "shows locale for category" do
-        click_link "Manage"
-        within "#category_#{@c.id}" do
-          expect(page).to have_css(".locale_icon.ru")
-        end
-      end
-
-      it "does not show locale for primary locale" do
-        click_link "Manage"
-        within "#category_#{@c.id}" do
-          expect(page).not_to have_css(".locale_icon.en")
+          category = Refinery::Blog::Category.first
+          expect(category.title).to eq(title)
         end
-      end
-
-      it "does not shows up in blog page for default locale" do
-        visit refinery.blog_root_path
-        expect(page).not_to have_selector('#categories')
-      end
 
-      it "shows up in blog page for secondary locale" do
-        visit refinery.blog_root_path(:locale => :ru)
-        within "#categories" do
-          expect(page).to have_selector('li')
+        context "with translations" do
+          before do
+            allow(Refinery::I18n).to receive(:frontend_locales).and_return([:en, :ru])
+            blog_page = Globalize.with_locale(:en) { FactoryGirl.create(:page, :link_url => "/blog", :title => "Blog") }
+            Globalize.with_locale(:ru) do
+              blog_page.title = 'блог'
+              blog_page.save
+            end
+          end
+
+          describe "add a category with title for default locale" do
+            before do
+              Globalize.locale = :en
+              visit refinery.blog_admin_posts_path
+              click_link "Create new category"
+              fill_in "Title", :with => "Testing Category"
+              expect { click_button "Save" }.to change(Refinery::Blog::Category, :count).by(1)
+              @c = Refinery::Blog::Category.by_title("Testing Category")
+            end
+
+            it "suceeds" do
+              expect(page).to have_content("'#{@c.title}' was successfully added.")
+              expect(Refinery::Blog::Category.count).to eq(1)
+            end
+
+            it "shows locale for category" do
+              click_link "Manage"
+              within "#category_#{@c.id}" do
+                expect(page).to have_css(".locale_icon.en")
+              end
+            end
+
+            it "shows up in blog page for default locale" do
+              visit refinery.blog_root_path
+              within "#categories" do
+                expect(page).to have_selector('li')
+              end
+            end
+
+            it "does not show up in blog page for secondary locale" do
+              visit refinery.blog_root_path(:locale => :ru)
+              expect(page).not_to have_selector('#categories')
+            end
+
+          end
+
+          describe "add a category with title for secondary locale" do
+
+            let(:ru_category_title) { 'категория' }
+
+            before do
+              visit refinery.blog_admin_posts_path
+              click_link "Create new category"
+              within "#switch_locale_picker" do
+                click_link "RU"
+              end
+              fill_in "Title", :with => ru_category_title
+              expect { click_button "Save" }.to change(Refinery::Blog::Category, :count).by(1)
+              @c = Refinery::Blog::Category.by_title(ru_category_title)
+            end
+
+            it "suceeds" do
+              expect(page).to have_content("'#{@c.title}' was successfully added.")
+              expect(Refinery::Blog::Category.count).to eq(1)
+            end
+
+            it "shows locale for category" do
+              click_link "Manage"
+              within "#category_#{@c.id}" do
+                expect(page).to have_css(".locale_icon.ru")
+              end
+            end
+
+            it "does not show locale for primary locale" do
+              click_link "Manage"
+              within "#category_#{@c.id}" do
+                expect(page).not_to have_css(".locale_icon.en")
+              end
+            end
+
+            it "does not shows up in blog page for default locale" do
+              visit refinery.blog_root_path
+              expect(page).not_to have_selector('#categories')
+            end
+
+            it "shows up in blog page for secondary locale" do
+              visit refinery.blog_root_path(:locale => :ru)
+              within "#categories" do
+                expect(page).to have_selector('li')
+              end
+            end
+          end
         end
       end
-
-
     end
-
-
   end
 end
diff --git a/spec/features/refinery/blog/admin/comments_spec.rb b/spec/features/refinery/blog/admin/comments_spec.rb
index 097210d..1c6d67d 100644
--- a/spec/features/refinery/blog/admin/comments_spec.rb
+++ b/spec/features/refinery/blog/admin/comments_spec.rb
@@ -4,7 +4,7 @@ module Refinery
   module Blog
     module Admin
       describe Comment, type: :feature do
-        refinery_login_with :refinery_user
+        refinery_login_with_devise :authentication_devise_refinery_superuser
 
         describe "#index" do
           context "when has no new unapproved comments" do
diff --git a/spec/features/refinery/blog/admin/menu_spec.rb b/spec/features/refinery/blog/admin/menu_spec.rb
index 748d5c3..64f6461 100644
--- a/spec/features/refinery/blog/admin/menu_spec.rb
+++ b/spec/features/refinery/blog/admin/menu_spec.rb
@@ -1,13 +1,19 @@
 require 'spec_helper'
 
-describe "Blog menu entry", type: :feature do
-  refinery_login_with :refinery_user
+module Refinery
+  module Blog
+    module Admin
+      describe Menu, type: :feature do
+        refinery_login_with_devise :authentication_devise_refinery_superuser
 
-  it "is highlighted when managing the blog" do
-    visit refinery.admin_root_path
+        it "is highlighted when managing the blog" do
+          visit refinery.admin_root_path
 
-    within("#menu") { click_link "Blog" }
+          within("#menu") { click_link "Blog" }
 
-    expect(page).to have_css("a.active", :text => "Blog")
+          expect(page).to have_css("a.active", :text => "Blog")
+        end
+      end
+    end
   end
-end
+end
\ No newline at end of file
diff --git a/spec/features/refinery/blog/admin/posts_spec.rb b/spec/features/refinery/blog/admin/posts_spec.rb
index 0efbb1d..ffb4b15 100644
--- a/spec/features/refinery/blog/admin/posts_spec.rb
+++ b/spec/features/refinery/blog/admin/posts_spec.rb
@@ -5,7 +5,7 @@ module Refinery
   module Blog
     module Admin
       describe Post, type: :feature do
-        refinery_login_with :refinery_user
+        refinery_login_with_devise :authentication_devise_refinery_superuser
 
         let!(:blog_category) do
           Globalize.with_locale(:en) { FactoryGirl.create(:blog_category) }
@@ -58,7 +58,7 @@ module Refinery
               end
 
               it "should belong to me" do
-                expect(subject.class.first.author).to eq(::Refinery::User.last)
+                expect(subject.class.first.author).to eq(::Refinery::Blog.user_class.last)
               end
 
               it "should save categories" do
@@ -153,7 +153,7 @@ module Refinery
         end
 
         context "with multiple users" do
-          let!(:other_guy) { FactoryGirl.create(:refinery_user, :username => "Other Guy") }
+          let!(:other_guy) { FactoryGirl.create(:authentication_devise_refinery_user, :username => "Other Guy") }
 
           describe "create blog post with alternate author" do
             before do
@@ -210,13 +210,20 @@ module Refinery
             end
 
             it "shows up in blog page for default locale" do
-              visit refinery.blog_root_path
+              visit refinery.blog_admin_root_path
               expect(page).to 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)
-              expect(page).not_to have_selector("#post_#{@p.id}")
+            it "does show locale for default locale" do
+              within "#post_#{@p.id}" do
+                expect(page).to have_css(".locale_icon.en")
+              end
+            end
+
+            it "does not show locale for secondary locale" do
+              within "#post_#{@p.id}" do
+                expect(page).not_to have_css(".locale_icon.ru")
+              end
             end
 
           end
@@ -259,13 +266,14 @@ module Refinery
               end
             end
 
-            it "does not show up in blog page for default locale" do
-              visit refinery.blog_root_path
-              expect(page).not_to have_selector("#post_#{@p.id}")
+            it "does show locale for secondary locale" do
+              within "#post_#{@p.id}" do
+                expect(page).to have_css(".locale_icon.ru")
+              end
             end
 
             it "shows up in blog page for secondary locale" do
-              visit refinery.blog_root_path(:locale => :ru)
+              visit refinery.blog_admin_root_path(:locale => :ru)
               expect(page).to have_selector("#post_#{@p.id}")
             end
 
diff --git a/spec/features/refinery/blog/categories_spec.rb b/spec/features/refinery/blog/categories_spec.rb
index b778254..1ffe29f 100644
--- a/spec/features/refinery/blog/categories_spec.rb
+++ b/spec/features/refinery/blog/categories_spec.rb
@@ -1,28 +1,30 @@
 require "spec_helper"
 
 module Refinery
-  describe "BlogCategories", type: :feature do
-    refinery_login_with :refinery_user
+  module Blog
+    describe Category, type: :feature do
+      refinery_login_with_devise :authentication_devise_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")
+      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
-        @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)
-          expect(page).to have_content("Refinery CMS blog post")
-          expect(page).to have_content("Video Games")
+        describe "show categories blog posts" do
+          it "should displays categories blog posts" do
+            visit refinery.blog_category_path(@category)
+            expect(page).to have_content("Refinery CMS blog post")
+            expect(page).to have_content("Video Games")
+          end
         end
       end
     end
   end
-end
+end
\ No newline at end of file
diff --git a/spec/features/refinery/blog/posts_spec.rb b/spec/features/refinery/blog/posts_spec.rb
index 3382398..816e11c 100644
--- a/spec/features/refinery/blog/posts_spec.rb
+++ b/spec/features/refinery/blog/posts_spec.rb
@@ -1,180 +1,182 @@
 require "spec_helper"
 
 module Refinery
-  describe "Blog::Posts", type: :feature do
-    refinery_login_with :refinery_user
+  module Blog
+    describe Post, type: :feature do
+      refinery_login_with_devise :authentication_devise_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
+      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)
+        it "should display blog post" do
+          visit refinery.blog_post_path(blog_post)
 
-        expect(page).to have_content(blog_post.title)
-      end
+          expect(page).to have_content(blog_post.title)
+        end
 
-      describe "visit blog" do
+        describe "visit blog" do
 
-        before do
-          FactoryGirl.create(:page, :link_url => "/")
-          FactoryGirl.create(:page, :link_url => "/blog", :title => "Blog")
-        end
+          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
-            expect(page).to have_content("Blog")
-            expect(page).to have_selector("a[href='/blog']")
+          it "shows blog link in menu" do
+            visit "/"
+            within "#menu" do
+              expect(page).to have_content("Blog")
+              expect(page).to have_selector("a[href='/blog']")
+            end
+          end
+
+          it "shows blog posts" do
+            visit refinery.blog_root_path
+            expect(page).to have_content blog_post.title
           end
-        end
 
-        it "shows blog posts" do
-          visit refinery.blog_root_path
-          expect(page).to have_content blog_post.title
         end
 
       end
 
-    end
-
-    describe "list tagged posts" do
-      context "when has tagged blog posts" do
-        let!(:tag_name) { "chicago" }
-        let!(:post) {
-          FactoryGirl.create(:blog_post,
-            :title => "I Love my city",
-            :tag_list => tag_name
-          )
-        }
-        let!(:tag) { ::Refinery::Blog::Post.tag_counts_on(:tags).first }
-
-        it "should have one tagged post" do
-          visit refinery.blog_tagged_posts_path(tag.id, tag_name.parameterize)
-
-          expect(page).to have_content(tag_name)
-          expect(page).to have_content(post.title)
+      describe "list tagged posts" do
+        context "when has tagged blog posts" do
+          let!(:tag_name) { "chicago" }
+          let!(:post) {
+            FactoryGirl.create(:blog_post,
+              :title => "I Love my city",
+              :tag_list => tag_name
+            )
+          }
+          let!(:tag) { ::Refinery::Blog::Post.tag_counts_on(:tags).first }
+
+          it "should have one tagged post" do
+            visit refinery.blog_tagged_posts_path(tag.id, tag_name.parameterize)
+
+            expect(page).to have_content(tag_name)
+            expect(page).to have_content(post.title)
+          end
         end
       end
-    end
 
-    describe "#show" do
-      context "when has no comments" do
-        let!(:blog_post) { FactoryGirl.create(:blog_post) }
+      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)
-          expect(page).to have_content(blog_post.title)
-          expect(page).to have_content(blog_post.body)
+          it "should display the blog post" do
+            visit refinery.blog_post_path(blog_post)
+            expect(page).to have_content(blog_post.title)
+            expect(page).to have_content(blog_post.body)
+          end
         end
-      end
-      context "when has approved comments" do
-        let!(:approved_comment) { FactoryGirl.create(:approved_comment) }
+        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)
+          it "should display the comments" do
+            visit refinery.blog_post_path(approved_comment.post)
 
-          expect(page).to have_content(approved_comment.body)
-          expect(page).to have_content("Posted by #{approved_comment.name}")
+            expect(page).to have_content(approved_comment.body)
+            expect(page).to have_content("Posted by #{approved_comment.name}")
+          end
         end
-      end
-      context "when has rejected comments" do
-        let!(:rejected_comment) { FactoryGirl.create(:rejected_comment) }
+        context "when has rejected comments" do
+          let!(:rejected_comment) { FactoryGirl.create(:rejected_comment) }
 
-        it "should not display the comments" do
-          visit refinery.blog_post_path(rejected_comment.post)
+          it "should not display the comments" do
+            visit refinery.blog_post_path(rejected_comment.post)
 
-          expect(page).not_to have_content(rejected_comment.body)
+            expect(page).not_to have_content(rejected_comment.body)
+          end
         end
-      end
-      context "when has new comments" do
-        let!(:blog_comment) { FactoryGirl.create(:blog_comment) }
+        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)
+          it "should not display the comments" do
+            visit refinery.blog_post_path(blog_comment.post)
 
-          expect(page).not_to have_content(blog_comment.body)
+            expect(page).not_to have_content(blog_comment.body)
+          end
         end
-      end
 
-      context "when posting comments" do
-        let!(:blog_post) { FactoryGirl.create(:blog_post) }
-        let(:name) { "pete" }
-        let(:email) { "pete@mcawesome.com" }
-        let(:body) { "Witty comment." }
+        context "when posting comments" do
+          let!(:blog_post) { FactoryGirl.create(:blog_post) }
+          let(:name) { "pete" }
+          let(:email) { "pete@mcawesome.com" }
+          let(:body) { "Witty comment." }
 
-        before do
-          visit refinery.blog_post_path(blog_post)
+          before do
+            visit refinery.blog_post_path(blog_post)
 
-          fill_in "Name", :with => name
-          fill_in "Email", :with => email
-          fill_in "Message", :with => body
-          click_button "Send comment"
-        end
+            fill_in "Name", :with => name
+            fill_in "Email", :with => email
+            fill_in "Message", :with => body
+            click_button "Send comment"
+          end
 
-        it "creates the comment" do
-          comment = blog_post.reload.comments.last
+          it "creates the comment" do
+            comment = blog_post.reload.comments.last
 
-          expect(comment.name).to eq(name)
-          expect(comment.email).to eq(email)
-          expect(comment.body).to eq(body)
+            expect(comment.name).to eq(name)
+            expect(comment.email).to eq(email)
+            expect(comment.body).to eq(body)
+          end
         end
-      end
 
-      context "post popular" do
-        let!(:blog_post) { FactoryGirl.create(:blog_post) }
-        let!(:blog_post2) { FactoryGirl.create(:blog_post) }
+        context "post popular" do
+          let!(:blog_post) { FactoryGirl.create(:blog_post) }
+          let!(:blog_post2) { FactoryGirl.create(:blog_post) }
 
-        before do
-          visit refinery.blog_post_path(blog_post)
-        end
+          before do
+            visit refinery.blog_post_path(blog_post)
+          end
 
-        it "should increment access count" do
-          expect(blog_post.reload.access_count).to eq(1)
-          visit refinery.blog_post_path(blog_post)
-          expect(blog_post.reload.access_count).to eq(2)
-        end
+          it "should increment access count" do
+            expect(blog_post.reload.access_count).to eq(1)
+            visit refinery.blog_post_path(blog_post)
+            expect(blog_post.reload.access_count).to eq(2)
+          end
 
-        it "should be most popular" do
-          expect(Refinery::Blog::Post.popular(2).first).to eq(blog_post)
+          it "should be most popular" do
+            expect(Refinery::Blog::Post.popular(2).first).to eq(blog_post)
+          end
         end
-      end
 
-      context "post recent" do
-        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) }
+        context "post recent" do
+          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
-          expect(Refinery::Blog::Post.recent(2).first.id).to eq(blog_post2.id)
+          it "should be the most recent" do
+            expect(Refinery::Blog::Post.recent(2).first.id).to eq(blog_post2.id)
+          end
         end
-      end
 
-    end
+      end
 
-    describe "#show draft preview" do
-      let!(:blog_post) { FactoryGirl.create(:blog_post_draft) }
+      describe "#show draft preview" do
+        let!(:blog_post) { FactoryGirl.create(:blog_post_draft) }
 
-      context "when logged in as admin" do
-        it "should display the draft notification" do
-          visit refinery.blog_post_path(blog_post)
+        context "when logged in as admin" do
+          it "should display the draft notification" do
+            visit refinery.blog_post_path(blog_post)
 
-          expect(page).to have_content('This page is NOT live for public viewing.')
+            expect(page).to have_content('This page is NOT live for public viewing.')
+          end
         end
-      end
 
-      context "when not logged in as an admin" do
-        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
+        context "when not logged in as an admin" do
+          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)
+          it "should not display the blog post" do
+            visit refinery.blog_post_path(blog_post)
 
-          expect(page).to have_content("The page you requested was not found.")
+            expect(page).to have_content("The page you requested was not found.")
+          end
         end
       end
     end