Add tests for command options.

--date, --layout, --type options were mostly untested

Author: courajs

spec/draft_spec.rb

@@ -40,6 +40,18 @@
     expect(drafts_dir).to exist
   end
 
+  it 'creates the draft with a specified extension' do
+    html_path = drafts_dir.join 'a-test-post.html'
+    expect(html_path).not_to exist
+    capture_stdout { described_class.process(args, 'type' => 'html') }
+    expect(html_path).to exist
+  end
+
+  it 'creates a new draft with the specified layout' do
+    capture_stdout { described_class.process(args, 'layout' => 'other-layout') }
+    expect(File.read(path)).to match(/layout: other-layout/)
+  end
+
   context 'when the draft already exists' do
     let(:name) { 'An existing draft' }
     let(:path) { drafts_dir.join('an-existing-draft.md') }

spec/page_spec.rb

@@ -19,6 +19,18 @@
     expect(path).to exist
   end
 
+  it 'creates a new page with the specified type' do
+    html_path = Pathname.new(source_dir).join 'a-test-page.html'
+    FileUtils.rm html_path if File.exist? html_path
+    capture_stdout { described_class.process(args, 'type' => 'html') }
+    expect(html_path).to exist
+  end
+
+  it 'creates a new page with the specified layout' do
+    capture_stdout { described_class.process(args, 'layout' => 'other-layout') }
+    expect(File.read(path)).to match(/layout: other-layout/)
+  end
+
   it 'should write a helpful message when successful' do
     output = capture_stdout { described_class.process(args) }
     expect(output).to eql("New page created at #{filename}.\n")

spec/post_spec.rb

@@ -2,7 +2,8 @@
   let(:name) { 'A test post' }
   let(:args) { [name] }
   let(:posts_dir) { Pathname.new source_dir('_posts') }
-  let(:filename) { "#{Time.now.strftime('%Y-%m-%d')}-a-test-post.md" }
+  let(:datestamp) { Time.now.strftime('%Y-%m-%d') }
+  let(:filename) { "#{datestamp}-a-test-post.md" }
   let(:path) { posts_dir.join(filename) }
 
   before(:all) do
@@ -31,6 +32,18 @@
     expect(path).to exist
   end
 
+  it 'creates the post with a specified extension' do
+    html_path = posts_dir.join "#{datestamp}-a-test-post.html"
+    expect(html_path).not_to exist
+    capture_stdout { described_class.process(args, 'type' => 'html') }
+    expect(html_path).to exist
+  end
+
+  it 'creates a new post with the specified layout' do
+    capture_stdout { described_class.process(args, 'layout' => 'other-layout') }
+    expect(File.read(path)).to match(/layout: other-layout/)
+  end
+
   it 'should write a helpful message when successful' do
     output = capture_stdout { described_class.process(args) }
     expect(output).to eql("New post created at _posts/#{filename}.\n")

spec/publish_spec.rb

@@ -1,8 +1,9 @@
 RSpec.describe(Jekyll::Commands::Publish) do
   let(:drafts_dir) { source_dir('_drafts') }
   let(:posts_dir)  { source_dir('_posts') }
-  let(:draft_to_publish) { 'a-test-post.markdown' }
-  let(:post_filename) { "#{Time.now.strftime('%Y-%m-%d')}-#{draft_to_publish}" }
+  let(:draft_to_publish) { 'a-test-post.md' }
+  let(:datestamp) { Time.now.strftime('%Y-%m-%d') }
+  let(:post_filename) { "#{datestamp}-#{draft_to_publish}" }
   let(:args) { ["_drafts/#{draft_to_publish}"] }
 
   let(:draft_path) { Pathname.new(File.join(drafts_dir, draft_to_publish)) }
@@ -38,6 +39,12 @@
     output = capture_stdout { described_class.process(args) }
     expect(output).to eql("Draft _drafts/#{draft_to_publish} was published to _posts/#{post_filename}\n")
   end
+  
+  it 'publishes a draft on the specified date' do
+    path = Pathname.new(posts_dir).join "2012-03-04-a-test-post.md"
+    capture_stdout { described_class.process(args, {"date" => '2012-3-4'}) }
+    expect(path).to exist
+  end
 
   it 'creates the posts folder if necessary' do
     FileUtils.rm_r posts_dir if File.directory? posts_dir