Merge pull request #39 from djds23/make-title-string-literal

Merge pull request 39

Author: jekyllbot

lib/jekyll-compose/file_info.rb

@@ -10,11 +10,11 @@ def file_name
   end
 
   def content
-    <<-CONTENT.gsub /^\s+/, ''
-      ---
-      layout: #{params.layout}
-      title: #{params.title}
-      ---
-    CONTENT
+    front_matter = YAML.dump({
+      'layout' => params.layout,
+      'title' => params.title,
+    })
+
+    front_matter + "---\n"
   end
 end

spec/file_info_spec.rb

@@ -0,0 +1,46 @@
+RSpec.describe(Jekyll::Compose::FileInfo) do
+  describe '#content' do
+    context 'with a title of only words' do
+      let(:expected_result) {<<-CONTENT.gsub(/^\s+/, '')
+          ---
+          layout: post
+          title: A test arg parser
+          ---
+        CONTENT
+      }
+
+      let(:parsed_args) { Jekyll::Compose::ArgParser.new(
+          ['A test arg parser'],
+          {}
+        )
+      }
+
+      it 'does not wrap the title in quotes' do
+        file_info = described_class.new parsed_args
+        expect(file_info.content).to eq(expected_result)
+      end
+    end
+
+    context 'with a title that includes a colon' do
+      let(:expected_result) {<<-CONTENT.gsub(/^\s+/, '')
+          ---
+          layout: post
+          title: 'A test: arg parser'
+          ---
+        CONTENT
+      }
+
+      let(:parsed_args) { Jekyll::Compose::ArgParser.new(
+          ['A test: arg parser'],
+          {}
+        )
+      }
+
+      it 'does wrap the title in quotes' do
+        file_info = described_class.new parsed_args
+        expect(file_info.content).to eq(expected_result)
+      end
+    end
+  end
+end
+