Fix setting keyword faces for todo keyword with hyphen. Fixes #356

Author: kristijanhusak

lua/orgmode/colors/highlights.lua

@@ -187,7 +187,7 @@ function M.parse_todo_keyword_faces(do_syn_match)
       end
     end
     if not vim.tbl_isempty(hl_opts) then
-      local hl_name = 'OrgKeywordFace' .. name
+      local hl_name = 'OrgKeywordFace' .. name:gsub('%-', '')
       local hl = ''
       for hl_item, hl_values in pairs(hl_opts) do
         hl = hl .. ' ' .. hl_item .. '=' .. table.concat(hl_values, ',')

tests/plenary/colors/colors_spec.lua

@@ -58,6 +58,7 @@ describe('Colors', function()
       org_todo_keyword_faces = {
         NEXT = ':foreground "blue" :underline on :weight bold :background red :slant italic',
         CANCELED = ':foreground green :slant italic',
+        ['IN-PROGRESS'] = ':foreground red :weight bold',
       },
     })
 
@@ -66,6 +67,7 @@ describe('Colors', function()
     assert.are.same({
       NEXT = 'OrgKeywordFaceNEXT',
       CANCELED = 'OrgKeywordFaceCANCELED',
+      ['IN-PROGRESS'] = 'OrgKeywordFaceINPROGRESS',
     }, result)
 
     assert.are.same('red', get_color_opt('OrgKeywordFaceNEXT', 'bg', 'gui'))
@@ -90,9 +92,21 @@ describe('Colors', function()
     assert.are.same('', get_color_opt('OrgKeywordFaceCANCELED', 'underline', 'gui'))
     assert.are.same('', get_color_opt('OrgKeywordFaceCANCELED', 'underline', 'cterm'))
 
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'bg', 'gui'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'bg', 'cterm'))
+    assert.are.same('red', get_color_opt('OrgKeywordFaceINPROGRESS', 'fg', 'gui'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'fg', 'cterm'))
+    assert.are.same('1', get_color_opt('OrgKeywordFaceINPROGRESS', 'bold', 'gui'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'bold', 'cterm'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'italic', 'gui'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'italic', 'cterm'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'underline', 'gui'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'underline', 'cterm'))
+
     vim.cmd([[
       hi clear OrgKeywordFaceNEXT
       hi clear OrgKeywordFaceCANCELED
+      hi clear OrgKeywordFaceINPROGRESS
     ]])
 
     vim.o.termguicolors = false
@@ -101,6 +115,7 @@ describe('Colors', function()
     assert.are.same({
       NEXT = 'OrgKeywordFaceNEXT',
       CANCELED = 'OrgKeywordFaceCANCELED',
+      ['IN-PROGRESS'] = 'OrgKeywordFaceINPROGRESS',
     }, result)
 
     assert.are.same('9', get_color_opt('OrgKeywordFaceNEXT', 'bg', 'cterm'))
@@ -124,5 +139,16 @@ describe('Colors', function()
     assert.are.same('', get_color_opt('OrgKeywordFaceCANCELED', 'bold', 'cterm'))
     assert.are.same('', get_color_opt('OrgKeywordFaceCANCELED', 'underline', 'gui'))
     assert.are.same('', get_color_opt('OrgKeywordFaceCANCELED', 'underline', 'cterm'))
+
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'bg', 'cterm'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'bg', 'gui'))
+    assert.are.same('9', get_color_opt('OrgKeywordFaceINPROGRESS', 'fg', 'cterm'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'fg', 'gui'))
+    assert.are.same('1', get_color_opt('OrgKeywordFaceINPROGRESS', 'bold', 'cterm'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'bold', 'gui'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'italic', 'cterm'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'italic', 'gui'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'underline', 'cterm'))
+    assert.are.same('', get_color_opt('OrgKeywordFaceINPROGRESS', 'underline', 'gui'))
   end)
 end)