feat(capture): Make sure tags are written as :tag: and not just plain text.

Author: Alec Stewart

lua/orgmode/capture/template/init.lua

@@ -81,10 +81,10 @@ local function prompt_tags(single, tags_source)
 
     local tags = utils.parse_tags_string(input)
     if single then
-      return tags[1] or ''
+      return tags[1] and utils.tags_to_string({ tags[1] }) or ''
     end
 
-    return table.concat(tags, ':')
+    return utils.tags_to_string(tags)
   end)
 end
 

tests/plenary/capture/templates_spec.lua

@@ -98,7 +98,7 @@ describe('Capture template', function()
       local template = Template:new({
         template = '* TODO %^g',
       })
-      assert.are.same({ '* TODO mytag' }, template:compile():wait())
+      assert.are.same({ '* TODO :mytag:' }, template:compile():wait())
     end)
   end)
 
@@ -109,7 +109,7 @@ describe('Capture template', function()
       local template = Template:new({
         template = '* TODO %^G',
       })
-      assert.are.same({ '* TODO tag1:tag2' }, template:compile():wait())
+      assert.are.same({ '* TODO :tag1:tag2:' }, template:compile():wait())
     end)
   end)
 
@@ -120,7 +120,7 @@ describe('Capture template', function()
       local template = Template:new({
         template = '* TODO %^{tag1|tag2}G',
       })
-      assert.are.same({ '* TODO tag1' }, template:compile():wait())
+      assert.are.same({ '* TODO :tag1:' }, template:compile():wait())
     end)
   end)
 
@@ -160,7 +160,7 @@ describe('Capture template', function()
         template = '* TODO %^g',
         target = files['target.org'],
       })
-      assert.are.same({ '* TODO target_headline' }, template:compile():wait())
+      assert.are.same({ '* TODO :target_headline:' }, template:compile():wait())
     end)
   end)
 
@@ -188,7 +188,7 @@ describe('Capture template', function()
       local template = Template:new({
         template = '* TODO %^G',
       })
-      assert.are.same({ '* TODO target_headline:other_headline' }, template:compile():wait())
+      assert.are.same({ '* TODO :target_headline:other_headline:' }, template:compile():wait())
     end)
   end)
 end)