mr_poole 0.1.0 → 0.2.0

data/spec/cli_spec.rb

@@ -1,328 +1,288 @@
 require 'spec_helper'
-
 require 'mr_poole'
-require 'fileutils'
-require 'stringio'
 
 module MrPoole
   describe CLI do
 
     context 'should determine jekyll dir correctly' do
-
       it 'should exit with no _posts directory' do
         olddir, tmpdir = make_no_jekyll_dir
-
-        argv = []
-        output = capture_stdout do
-          begin
-            cli = CLI.new(argv)
-          rescue SystemExit => e
-            e.should be_instance_of(SystemExit)
-          end
-        end
-
+        expect {capture_stdout { cli = CLI.new([]) }}.to raise_error(SystemExit)
         clean_tmp_files(tmpdir, olddir)
       end
 
       it 'should not exit with _posts directory' do
         olddir, tmpdir = make_jekyll_dir
-
-        argv = []
-        lambda { cli = CLI.new(argv) }.should_not raise_error
-
+        expect { cli = CLI.new([]) }.not_to raise_error
         clean_tmp_files(tmpdir, olddir)
       end
-
-    end   # end context determine jekyll dir
+    end   # context determine jekyll dir
 
     describe "action 'post'" do
-
-      before :each do
-        @olddir, @tmpdir = make_jekyll_dir
-      end
-
-      after :each do
-        clean_tmp_files(@tmpdir, @olddir)
-      end
+      before(:each) { @olddir, @tmpdir = make_jekyll_dir }
+      after(:each) { clean_tmp_files(@tmpdir, @olddir) }
 
       context 'error handling' do
-
         it 'should fail with no arguments' do
           argv = ['post']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.to raise_error(SystemExit)
+          expect { poole_no_stdout(argv).call }.to raise_error(SystemExit)
         end
 
         it 'should fail with no title (with slug)' do
           argv = ['post', '-s', 'post_slug']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.to raise_error(SystemExit)
+          expect { poole_no_stdout(argv).call }.to raise_error(SystemExit)
         end
 
         it 'should not fail with a title (no switch)' do
           argv = ['post', 'Here is a title']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.not_to raise_error
+          expect { poole_no_stdout(argv).call }.not_to raise_error
         end
 
         it 'should not fail with a title (long switch)' do
           argv = ['post', '--title', 'Here is a title']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.not_to raise_error
+          expect { poole_no_stdout(argv).call }.not_to raise_error
         end
 
         it 'should not fail with a title (short switch)' do
           argv = ['post', '-t', 'Here is a title']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.not_to raise_error
+          expect { poole_no_stdout(argv).call }.not_to raise_error
         end
-
       end   # context error handling
 
       context 'exit message' do
-
         it 'should exit with a usage message' do
           argv = ['post']
+          output = aborted_poole_output(argv).call
+          expect(output).to match(/Usage:\s+poole post/)
+        end
+      end   # context exit message
 
-          output = capture_stdout do
-            begin
-              poole_with_args(argv).call
-            rescue SystemExit => e
-              # this will fail, but we want the exit message
-            end
-          end
+      context 'custom layout' do
+        let(:layout_path) { write_custom_layout }
 
-          output.should match(/Usage:\s+poole post/)
+        it 'should use custom layout with --layout' do
+          argv = ['post', '--layout', layout_path, '--title', 'title']
+          poole_no_stdout(argv).call
+          new_file = Dir.glob("_posts/*.md").first
+          content = File.open(new_file, 'r').read
+          expect(content).to match(/tags: testing/)
         end
 
-      end   # context exit message
+        it 'should exit with bad layout path' do
+          argv = ['post', '--layout', 'bogus_path.md', 'title']
+          expect { poole_no_stdout(argv).call }.to raise_error(SystemExit)
+        end
+      end   # context custom layout
 
-    end   # end describe post
+      context 'default_layout in config file' do
+        before(:each) { write_config_file_custom_layout }
 
-    describe "action 'draft'" do
+        it 'overrides default layout' do
+          argv = ['post', 'post title']
+          poole_no_stdout(argv).call
+          new_file = Dir.glob("_posts/*.md").first
+          content = File.open(new_file, 'r').read
+          expect(content).to match(/tags: from_config/)
+        end
 
-      before :each do
-        @olddir, @tmpdir = make_jekyll_dir
-      end
+        it 'is overridden by command-line layout switch' do
+          custom_path = write_custom_layout
 
-      after :each do
-        clean_tmp_files(@tmpdir, @olddir)
-      end
+          argv = ['post', '--layout', custom_path, '--title', 'title']
+          poole_no_stdout(argv).call
+          new_file = Dir.glob("_posts/*.md").first
+          content = File.open(new_file, 'r').read
+          expect(content).not_to match(/tags: from_config/)
+          expect(content).to match(/tags: testing/)
+        end
+      end   # default layout in config
 
-      context 'error handling' do
+    end   # end describe post
 
+    describe "action 'draft'" do
+      before(:each) { @olddir, @tmpdir = make_jekyll_dir }
+      after(:each) { clean_tmp_files(@tmpdir, @olddir) }
+
+      context 'error handling' do
         it 'should fail with no arguments' do
           argv = ['draft']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.to raise_error(SystemExit)
+          expect { poole_no_stdout(argv).call }.to raise_error(SystemExit)
         end
 
         it 'should fail with no title (with slug)' do
           argv = ['draft', '-s', 'draft_slug']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.to raise_error(SystemExit)
+          expect { poole_no_stdout(argv).call }.to raise_error(SystemExit)
         end
 
         it 'should not fail with a title (no switch)' do
           argv = ['draft', 'Here is a title']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.not_to raise_error
+          expect { poole_no_stdout(argv).call }.not_to raise_error
         end
 
         it 'should not fail with a title (long switch)' do
           argv = ['draft', '--title', 'Here is a title']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.not_to raise_error
+          expect { poole_no_stdout(argv).call }.not_to raise_error
         end
 
         it 'should not fail with a title (short switch)' do
           argv = ['draft', '-t', 'Here is a title']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.not_to raise_error
+          expect { poole_no_stdout(argv).call }.not_to raise_error
         end
-
       end   # context error handling
 
       context 'exit message' do
-
         it 'should exit with a usage message' do
           argv = ['draft']
+          output = aborted_poole_output(argv).call
+          expect(output).to match(/Usage:\s+poole draft/)
+        end
+      end   # context exit message
 
-          output = capture_stdout do
-            begin
-              poole_with_args(argv).call
-            rescue SystemExit
-              # this will fail, but we want the exit message
-            end
-          end
+      context 'custom layout' do
+        let(:layout_path) { write_custom_layout }
 
-          output.should match(/Usage:\s+poole draft/)
+        it 'should use custom layout with --layout' do
+          argv = ['draft', '--layout', layout_path, '--title', 'title']
+          poole_no_stdout(argv).call
+          content = File.open(layout_path, 'r').read
+          expect(content).to match(/tags: testing/)
         end
 
-      end   # context exit message
+        it 'should exit with bad layout path' do
+          argv = ['draft', '--layout', 'bogus_path.md', 'title']
+          expect { poole_no_stdout(argv).call }.to raise_error(SystemExit)
+        end
+      end   # end custom layout
 
-    end   # end describe draft
+      context 'default_layout in config file' do
+        before(:each) { write_config_file_custom_layout }
 
-    describe "action 'publish'" do
-      before :each do
-        @olddir, @tmpdir = make_jekyll_dir
-        @c = Commands.new
-        @d_path = @c.draft('test_draft')
-      end
+        it 'overrides default layout' do
+          argv = ['draft', 'post title']
+          poole_no_stdout(argv).call
+          new_file = Dir.glob("_drafts/*.md").first
+          content = File.open(new_file, 'r').read
+          expect(content).to match(/tags: from_config/)
+        end
 
-      after :each do
-        clean_tmp_files(@tmpdir, @olddir)
-      end
+        it 'is overridden by command-line layout switch' do
+          custom_path = write_custom_layout
 
-      context 'error handling' do
+          argv = ['draft', '--layout', custom_path, '--title', 'title']
+          poole_no_stdout(argv).call
+          new_file = Dir.glob("_drafts/*.md").first
+          content = File.open(new_file, 'r').read
+          expect(content).not_to match(/tags: from_config/)
+          expect(content).to match(/tags: testing/)
+        end
+      end   # default layout in config
+
+    end   # describe draft
+
+    describe "action 'publish'" do
+      let(:d_path) { Commands.new.draft({title: 'test_draft'}) }
+      before(:each) { @olddir, @tmpdir = make_jekyll_dir }
+      after(:each) { clean_tmp_files(@tmpdir, @olddir) }
 
+      context 'error handling' do
         it 'should fail with no arguments' do
           argv = ['publish']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.to raise_error(SystemExit)
+          expect { poole_no_stdout(argv).call }.to raise_error(SystemExit)
         end
 
         it 'should fail with a bad path' do
           argv = ['publish', '_drafts/does_not_exist.md']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.to raise_error(SystemExit)
+          expect { poole_no_stdout(argv).call }.to raise_error(SystemExit)
         end
 
         it 'should not fail with a good path' do
-          argv = ['publish', @d_path]
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.not_to raise_error
+          argv = ['publish', d_path]
+          expect { poole_no_stdout(argv).call }.not_to raise_error
         end
-
       end   # context error handling
 
       context 'exit message' do
-
         it 'should exit with usage with no arguments' do
           argv = ['publish']
-
-          output = capture_stdout do
-            begin
-              poole_with_args(argv).call
-            rescue SystemExit
-            end
-          end
-
-          output.should match(/Usage:\s+poole publish/)
+          output = aborted_poole_output(argv).call
+          expect(output).to match(/Usage:\s+poole publish/)
         end
 
         it 'should exit with a description of bad path' do
           argv = ['publish', '_drafts/does_not_exist.md']
-
-          output = capture_stdout do
-            begin
-              poole_with_args(argv).call
-            rescue SystemExit
-            end
-          end
-
-          output.should match(/Error:\s+could not open/)
+          output = aborted_poole_output(argv).call
+          expect(output).to match(/Error:\s+could not open/)
         end
       end   # context exit message
 
     end
 
     describe "action 'unpublish'" do
-      before :each do
-        @olddir, @tmpdir = make_jekyll_dir
-        @c = Commands.new
-        @p_path = @c.post('test_post')
-      end
-
-      after :each do
-        clean_tmp_files(@tmpdir, @olddir)
-      end
+      let(:p_path) { Commands.new.post({title: 'test_post'}) }
+      before(:each) { @olddir, @tmpdir = make_jekyll_dir }
+      after(:each) { clean_tmp_files(@tmpdir, @olddir) }
 
       context 'error handling' do
-
         it 'should fail with no arguments' do
           argv = ['unpublish']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.to raise_error(SystemExit)
+          expect { poole_no_stdout(argv).call }.to raise_error(SystemExit)
         end
 
         it 'should fail with a bad path' do
           argv = ['unpublish', '_posts/does_not_exist.md']
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.to raise_error(SystemExit)
+          expect { poole_no_stdout(argv).call }.to raise_error(SystemExit)
         end
 
         it 'should not fail with a good path' do
-          argv = ['unpublish', @p_path]
-
-          expect {
-            poole_with_args_no_stdout(argv).call
-          }.not_to raise_error
+          argv = ['unpublish', p_path]
+          expect { poole_no_stdout(argv).call }.not_to raise_error
         end
-
       end   # context error handling
 
       context 'exit message' do
-
         it 'should exit with usage with no arguments' do
           argv = ['unpublish']
-
-          output = capture_stdout do
-            begin
-              poole_with_args(argv).call
-            rescue SystemExit
-            end
-          end
-
-          output.should match(/Usage:\s+poole unpublish/)
+          output = aborted_poole_output(argv).call
+          expect(output).to match(/Usage:\s+poole unpublish/)
         end
 
         it 'should exit with a description of bad path' do
           argv = ['unpublish', '_posts/does_not_exist.md']
-
-          output = capture_stdout do
-            begin
-              poole_with_args(argv).call
-            rescue SystemExit
-            end
-          end
-
-          output.should match(/Error:\s+could not open/)
+          output = aborted_poole_output(argv).call
+          expect(output).to match(/Error:\s+could not open/)
         end
       end   # context exit message
 
+    end   # context action unpublish
+
+    context 'with default extension in config file' do
+      before(:each) { @olddir, @tmpdir = make_jekyll_dir }
+      before(:each) { write_config_file_custom_extension }
+      after(:each) { clean_tmp_files(@tmpdir, @olddir) }
+
+      it 'should override default extension for post' do
+        argv = ['post', 'post title']
+        poole_no_stdout(argv).call
+        fn = Dir.glob("_posts/*").first
+        expect(fn).to match(/textile$/)
+      end
+
+      it 'should not override if command-line layout given' do
+        layout_path = write_custom_layout
+        argv = ['post', '--layout', layout_path, '-t', 'title']
+        poole_no_stdout(argv).call
+        fn = Dir.glob("_posts/*").first
+        expect(fn).to match(/md$/)
+      end
 
-    end   # action unpublish
+      it 'should override default extension for draft' do
+        argv = ['draft', 'post title']
+        poole_no_stdout(argv).call
+        fn = Dir.glob("_drafts/*").first
+        expect(fn).to match(/textile$/)
+      end
+
+    end
 
   end
 end