storytime 0.0.4 → 1.0.0

data/spec/features/dashboard/posts_spec.rb

@@ -1,5 +1,5 @@
 require 'spec_helper'
-
+require 'pry'
 describe "In the dashboard, Posts" do
   before{ login_admin }
 
@@ -27,7 +27,7 @@ describe "In the dashboard, Posts" do
     fill_in "post_excerpt", with: "It was a dark and stormy night..."
     fill_in "post_draft_content", with: "It was a dark and stormy night..."
     find("#featured_media_id").set media.id
-    click_button "Create Blog post"
+    click_button "Save Draft"
     
     page.should have_content(I18n.t('flash.posts.create.success'))
     Storytime::BlogPost.count.should == 1
@@ -41,6 +41,50 @@ describe "In the dashboard, Posts" do
     post.featured_media.should == media
   end
 
+  it "saves a post when previewing a new post", js: true do
+    Storytime::BlogPost.count.should == 0
+
+    visit url_for([:new, :dashboard, :post, only_path: true])
+    fill_in "post_title", with: "Snow Crash"
+    fill_in "post_excerpt", with: "The Deliverator belongs to an elite order, a hallowed sub-category."
+    find("#post_draft_content", visible: false).set "The Deliverator belongs to an elite order, a hallowed sub-category."
+    click_button "Preview"
+    
+    page.should have_content(I18n.t('flash.posts.create.success'))
+    Storytime::BlogPost.count.should == 1
+
+    # Check to see if popup is opened?
+    # Check to see if popup is directed to correct post.
+
+    post = Storytime::BlogPost.last
+    post.title.should == "Snow Crash"
+    post.draft_content.should == "The Deliverator belongs to an elite order, a hallowed sub-category."
+    post.user.should == current_user
+    post.should_not be_published
+    post.type.should == "Storytime::BlogPost"
+  end
+
+  it "autosaves a post when editing", js: true do
+    post = FactoryGirl.create(:post, published_at: nil, title: "A Scandal in Bohemia",
+                              draft_content: "To Sherlock Holmes she was always the woman.")
+    original_creator = post.user
+    Storytime::BlogPost.count.should == 1
+
+    post.autosave.should == nil
+
+    visit url_for([:edit, :dashboard, post, only_path: true])
+
+    page.execute_script "Storytime.instance.editor.autosavePostForm()"
+
+    visit url_for([:edit, :dashboard, post, only_path: true])
+
+    page.should have_content("View the autosave.")
+
+    post.reload
+    expect(post.autosave).not_to be_nil
+    expect(post.autosave.content).to eq("To Sherlock Holmes she was always the woman.")
+  end
+
   it "updates a post" do
     post = FactoryGirl.create(:post, published_at: nil)
     original_creator = post.user
@@ -49,7 +93,7 @@ describe "In the dashboard, Posts" do
     visit url_for([:edit, :dashboard, post, only_path: true])
     fill_in "post_title", with: "The Story"
     fill_in "post_draft_content", with: "It was a dark and stormy night..."
-    click_button "Update Blog"
+    click_button "Save Draft"
     
     page.should have_content(I18n.t('flash.posts.update.success'))
     Storytime::BlogPost.count.should == 1

data/spec/features/dashboard/sites_spec.rb

@@ -36,7 +36,8 @@ describe "In the dashboard, Sites" do
 
     it "new redirects to edit if a site already exists" do
       visit new_dashboard_site_path
-      page.should have_content("Editing site")
+      page.should have_content("Site Settings")
+      page.should have_content("Permissions")
     end
   end
 end

data/spec/features/dashboard/snippets_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+
+describe "In the dashboard, Snippets" do
+  before do 
+    login_admin
+  end
+
+  it "lists snippets" do
+    3.times{ FactoryGirl.create(:snippet) }
+    visit url_for([:dashboard, Storytime::Snippet])
+    
+    Storytime::Snippet.all.each do |s|
+      expect(page).to have_link(s.name, href: url_for([:edit, :dashboard, s, only_path: true]))
+      expect(page).to_not have_content(s.content)
+    end
+  end
+  
+  it "creates a snippet" do
+    expect(Storytime::Snippet.count).to eq(0)
+
+    visit url_for([:new, :dashboard, :snippet, only_path: true])
+    fill_in "snippet_name", with: "jumbotron-text"
+    fill_in "snippet_content", with: "Hooray Writing!"
+    
+    click_button "Create Snippet"
+    
+    expect(page).to have_content(I18n.t('flash.snippets.create.success'))
+    expect(Storytime::Snippet.count).to eq(1)
+
+    snippet = Storytime::Snippet.last
+    expect(snippet.name).to eq("jumbotron-text")
+    expect(snippet.content).to eq("Hooray Writing!")
+  end
+
+  it "updates a snippet" do
+    snippet = FactoryGirl.create(:snippet)
+    expect(Storytime::Snippet.count).to eq(1)
+    
+    visit url_for([:edit, :dashboard, snippet])
+    fill_in "snippet_name", with: "new-name"
+    fill_in "snippet_content", with: "It was a dark and stormy night..."
+    click_button "Update Snippet"
+    
+    expect(page).to have_content(I18n.t('flash.snippets.update.success'))
+    expect(Storytime::Snippet.count).to eq(1)
+
+    snippet = Storytime::Snippet.last
+    expect(snippet.name).to eq("new-name")
+    expect(snippet.content).to eq("It was a dark and stormy night...")
+  end
+
+  it "deletes a snippet", js: true do
+    3.times{|i| FactoryGirl.create(:snippet) }
+    visit url_for([:dashboard, Storytime::Snippet, only_path: true])
+    s1 = Storytime::Snippet.first
+    s2 = Storytime::Snippet.last
+    click_link("delete_snippet_#{s1.id}")
+
+    expect(page).to_not have_content(s1.name)
+    expect(page).to have_content(s2.name)
+
+    expect{ s1.reload }.to raise_error
+  end
+  
+end

data/spec/lib/storytime_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe Storytime do
+  describe ".snippet" do
+    context "when the snippet doesn't exist" do
+      it "returns a blank string" do
+        expect(Storytime.snippet("missing")).to be_blank
+      end
+    end
+
+    context "when the snippet exists" do
+      let(:snippet){ FactoryGirl.create(:snippet) }
+      let(:returned_content){ Storytime.snippet(snippet.name) }
+      it "returns the content of a snippet with the passed name" do
+        expect(returned_content).to eq(snippet.content)
+      end
+
+      it "returns an html safe string" do
+        expect(returned_content).to be_html_safe
+      end
+    end
+  end
+end

data/spec/models/post_spec.rb

@@ -1,20 +1,49 @@
 require 'spec_helper'
 
 describe Storytime::Post do
-  it "Sets the page slug on create" do
+  it "sets the page slug on create" do
     post = FactoryGirl.create(:post)
     post.slug.should == post.title.parameterize
   end
 
+  it "sets slug to user inputted value" do
+    post = FactoryGirl.create(:post)
+
+    post.slug = "random slug here"
+    post.save
+
+    post.slug.should == "random-slug-here"
+  end
+
+  it "does not allow the same slug" do
+    post_1 = FactoryGirl.create(:post)
+    post_2 = FactoryGirl.create(:post)
+
+    post_2.slug = post_1.slug
+    post_2.save
+
+    post_2.slug.should_not == post_1.slug
+    post_2.slug.should include(post_1.slug)
+  end
+
+  it "does not allow a blank slug" do
+    post = FactoryGirl.create(:post)
+    post.slug = ""
+    post.save
+
+    post.slug.should_not == ""
+    post.slug.should == post.title.parameterize
+  end
+
   it "creates tags from tag_list attribute" do
     post = FactoryGirl.create(:post)
-    post.tag_list = "tag1, tag2"
+    post.tag_list = ["tag1", "tag2"]
     post.tags.count.should == 2
   end
 
   it "scopes posts by tag" do
-    post_1 = FactoryGirl.create(:post, tag_list: "tag1, tag2")
-    post_2 = FactoryGirl.create(:post, tag_list: "tag1")
+    post_1 = FactoryGirl.create(:post, tag_list: ["tag1", "tag2"])
+    post_2 = FactoryGirl.create(:post, tag_list: ["tag1"])
 
     Storytime::Post.tagged_with("tag1").should include(post_1)
     Storytime::Post.tagged_with("tag1").should include(post_2)
@@ -23,8 +52,8 @@ describe Storytime::Post do
   end
 
   it "counts tags across posts" do
-    post_1 = FactoryGirl.create(:post, tag_list: "tag1, tag2")
-    post_2 = FactoryGirl.create(:post, tag_list: "tag1")
+    post_1 = FactoryGirl.create(:post, tag_list: ["tag1", "tag2"])
+    post_2 = FactoryGirl.create(:post, tag_list: ["tag1"])
 
     Storytime::Post.tag_counts.find_by(name: "tag1").count.should == 2
     Storytime::Post.tag_counts.find_by(name: "tag2").count.should == 1

data/spec/models/tagging_spec.rb

@@ -1,28 +1,29 @@
 require 'spec_helper'
 
 describe Storytime::Tagging do
-  it "removes unused tags after deleting a post" do
-    post_1 = FactoryGirl.create(:post, tag_list: "tag1, tag2")
-    post_2 = FactoryGirl.create(:post, tag_list: "tag1")
+  it "unused tags remain after deleting a post" do
+    post_1 = FactoryGirl.create(:post, tag_list: ["tag1", "tag2"])
+    post_2 = FactoryGirl.create(:post, tag_list: ["tag1"])
     Storytime::Tag.all.count.should == 2
     post_1.destroy
-    Storytime::Tag.all.count.should == 1
+    Storytime::Tag.all.count.should == 2
     post_2.destroy
-    Storytime::Tag.all.count.should == 0
+    Storytime::Tag.all.count.should == 2
   end
 
-  it "removes unused tags after updating tag list" do
-    post_1 = FactoryGirl.create(:post, tag_list: "tag1, tag2")
+  it "does not remove unused tags after updating tag list" do
+    post_1 = FactoryGirl.create(:post, tag_list: ["tag1", "tag2"])
+    Storytime::Tag.all.count.should == 2
+    post_1.update(tag_list: ["tag1"])
     Storytime::Tag.all.count.should == 2
-    post_1.update(tag_list: "tag1")
-    Storytime::Tag.all.count.should == 1
+    post_1.tags.count.should == 1
   end
 
   it "does not removes tags used by other posts after updating tag list" do
-    post_1 = FactoryGirl.create(:post, tag_list: "tag1, tag2")
-    post_2 = FactoryGirl.create(:post, tag_list: "tag2")
+    post_1 = FactoryGirl.create(:post, tag_list: ["tag1", "tag2"])
+    post_2 = FactoryGirl.create(:post, tag_list: ["tag2"])
     Storytime::Tag.all.count.should == 2
-    post_1.update(tag_list: "tag1")
+    post_1.update(tag_list: ["tag1"])
     Storytime::Tag.all.count.should == 2
   end
 end

data/spec/requests/routings_spec.rb

@@ -35,7 +35,7 @@ describe "Post path" do
     FactoryGirl.create(:site, post_slug_style: :day_and_name)
     post = FactoryGirl.create(:post)
     date = post.created_at.to_date
-    expect(url_for([post, only_path: true])).to  eq("/#{date.year}/#{date.month}/#{date.day}/#{post.slug}")
+    expect(url_for([post, only_path: true])).to  eq("/#{date.year}/#{date.strftime('%m')}/#{date.strftime('%d')}/#{post.slug}")
 
     get url_for([post, only_path: true])
     
@@ -48,7 +48,7 @@ describe "Post path" do
     FactoryGirl.create(:site, post_slug_style: :month_and_name)
     post = FactoryGirl.create(:post)
     date = post.created_at.to_date
-    expect(url_for([post, only_path: true])).to  eq("/#{date.year}/#{date.month}/#{post.slug}")
+    expect(url_for([post, only_path: true])).to  eq("/#{date.year}/#{date.strftime('%m')}/#{post.slug}")
 
     get url_for([post, only_path: true])
     
@@ -57,15 +57,15 @@ describe "Post path" do
     expect(request.params[:id]).to eq(post.slug)
   end
 
-  it "uses /post-slug when site#post_slug_style is post_name" do
-    FactoryGirl.create(:site, post_slug_style: :post_name)
+  it "uses /post-id when site#post_slug_style is post_id" do
+    FactoryGirl.create(:site, post_slug_style: :post_id)
     post = FactoryGirl.create(:post)
-    expect(url_for([post, only_path: true])).to  eq("/#{post.slug}")
+    expect(url_for([post, only_path: true])).to eq("/posts/#{post.id}")
 
     get url_for([post, only_path: true])
     
     expect(request.params[:controller]).to eq("storytime/posts")
     expect(request.params[:action]).to eq("show")
-    expect(request.params[:id]).to eq(post.slug)
+    expect(request.params[:id]).to eq(post.id.to_s)
   end
 end