blogit 0.8.0 → 1.0.0.rc1

data/spec/factories.rb

@@ -6,9 +6,13 @@ FactoryGirl.define do
   factory :post, class: Blogit::Post do
     title "Tis is a blog post title"
     body "This is the body of the blog post - you'll see it's a lot bigger than the title"
+    state "draft"
     association :blogger, :factory => :user
+    factory :published_post do
+      state "published"
+    end
   end
-  
+
   factory :comment, class: Blogit::Comment do
     name "Gavin"
     email "gavin@gavinmorrice.com"

data/spec/helpers/blogit/posts_helper_spec.rb

@@ -56,7 +56,7 @@ describe Blogit::PostsHelper do
       Post.delete_all
     end
 
-
+    # TODO: Clean up this horrible spec
     it "should create an ul tag tree with years, monthes and articles" do
       july_2012_1 = FactoryGirl.create(:post, title: "Great Post 2", created_at: Time.new(2012,7,14))
       july_2012_2 = FactoryGirl.create(:post, title: "Great Post 3", created_at: Time.new(2012,7,28))
@@ -74,13 +74,13 @@ describe Blogit::PostsHelper do
                                   "<ul class=\"#{month_css}\">",
                                     "<li><a data-blogit-click-to-toggle-children>September</a>",
                                       "<ul class=\"#{post_css}\">",
-                                        "<li>#{sept_2012.title}</li>",
+                                        "<li>#{link_to(sept_2012.title, blogit.post_path(sept_2012))}</li>",
                                       "</ul>",
                                     "</li>",
                                     "<li><a data-blogit-click-to-toggle-children>July</a>",
                                       "<ul class=\"#{post_css}\">",
-                                        "<li>#{july_2012_2.title}</li>",
-                                        "<li>#{july_2012_1.title}</li>",
+                                        "<li>#{link_to(july_2012_2.title, blogit.post_path(july_2012_2))}</li>",
+                                        "<li>#{link_to(july_2012_1.title, blogit.post_path(july_2012_1))}</li>",
                                       "</ul>",
                                     "</li>",
                                   "</ul>",
@@ -89,7 +89,7 @@ describe Blogit::PostsHelper do
                                   "<ul class=\"#{month_css}\">",
                                     "<li><a data-blogit-click-to-toggle-children>December</a>",
                                       "<ul class=\"#{post_css}\">",
-                                        "<li>#{dec_2011.title}</li>",
+                                        "<li>#{link_to(dec_2011.title, blogit.post_path(dec_2011))}</li>",
                                       "</ul>",
                                     "</li>",
                                   "</ul>",

data/spec/lib/blogs_spec.rb

@@ -10,12 +10,6 @@ describe Blogit::Blogs do
     it "should be a class macro to AR Base" do
       ActiveRecord::Base.methods.should include(:blogs)
     end
-    
-    
-    it "should build a hm assosciation on the model it's called in" do
-      lambda { User.new.blog_posts }.should_not raise_exception(NoMethodError)
-      User.new.blog_posts.should be_an(Array)
-    end
         
   end
 

data/spec/lib/configuration_spec.rb

@@ -92,7 +92,7 @@ describe Blogit::Configuration do
         gh_blockcode: true
       }
   end
-  
+
   it "should set the Pingr mode to :test unless Rails env is production" do
     Pingr.mode.should eql(:test)
   end
@@ -109,4 +109,20 @@ describe Blogit::Configuration do
     blog_configuration.rss_feed_description.should eql "#{Rails.application.engine_name.titleize} Blog Posts"
   end
 
+  it "should set layout to nil by default" do
+    blog_configuration.layout.should be_nil
+  end
+
+  it "should allow layout to be set to a string" do
+    lambda { blog_configuration.layout = "custom" }.should_not raise_error
+  end
+
+  it 'should set active_states default options' do
+    blog_configuration.active_states.should eql([:published])
+  end
+
+  it 'should set hidden_states default options' do
+    blog_configuration.hidden_states.should eql([:draft, :archive])
+  end
+
 end

data/spec/models/blogit/post_spec.rb

@@ -1,5 +1,5 @@
 require "spec_helper"
-
+  
 describe Blogit::Post do
 
   context "should not be valid" do
@@ -24,10 +24,6 @@ describe Blogit::Post do
         @blog_post.errors[:title].should_not be_blank
       end
 
-      it "is blank" do
-        # before and after block cover this
-      end
-
       it "is less than 10 characters" do
         @blog_post.title = "a" * 9
       end
@@ -58,6 +54,21 @@ describe Blogit::Post do
 
     end
 
+    context "if state" do
+
+      before(:each) { @blog_post = Blogit::Post.new(state: nil) }
+
+      it "is nil" do
+        @blog_post.should_not be_valid
+        @blog_post.should have(1).error_on(:state)
+      end
+
+    end
+
+  end
+
+  it "sets the first value of Blogit::configuration.hidden_states as default" do
+    Blogit::Post.new.state.should eql(Blogit::configuration.hidden_states[0].to_s)
   end
 
   context "with Blogit.configuration.comments == active_record" do
@@ -68,7 +79,7 @@ describe Blogit::Post do
       end
       User.blogs
       @blog_post = Blogit::Post.new
-      lambda { @blog_post.comments }.should_not raise_exception(RuntimeError)
+      lambda { @blog_post.comments }.should_not raise_exception
     end
 
   end
@@ -128,13 +139,16 @@ describe Blogit::Post do
         Blogit::Post.for_index.count.should eql(3)
       end
 
-
     end
 
-
+    describe :active do
+      it 'should include only posts in active states blogit.config.active_states' do
+        published_post = create(:published_post)
+        Blogit::Post.active.should include (published_post)
+      end
+    end
   end
 
-
   describe "with Blogit.configuration.comments != active_record" do
 
     it "should not allow comments" do
@@ -169,4 +183,10 @@ describe Blogit::Post do
     end
   end
 
+  describe 'AVAILABLE_STATUS'  do
+    it "returns all the statues in Blogit::configuration" do
+      Blogit::Post::AVAILABLE_STATUS.should  == (Blogit.configuration.hidden_states + Blogit.configuration.active_states)
+    end
+    
+  end
 end

data/spec/spec_helper.rb

@@ -1,11 +1,12 @@
 # This file is copied to spec/ when you run 'rails generate rspec:install'
-ENV["RAILS_ENV"] ||= 'test'
+ENV['RAILS_ENV'] ||= 'test'
 
-require File.expand_path("../dummy/config/environment", __FILE__)
+require File.expand_path('../dummy/config/environment', __FILE__)
 require 'rspec/rails'
-require "factory_girl"
-require "factories"
-require "support/authentication"
+require 'factory_girl'
+require 'factories'
+require 'support/authentication'
+require 'pry'
 
 include Blogit