fix: check if docs is nil

SergioRibera · Dec 27, 2023 · 7267fb5 · 7267fb5
1 parent 7d368a7
commit 7267fb5
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 4 deletions.
diff --git a/lua/cmp-dotenv/dotenv.lua b/lua/cmp-dotenv/dotenv.lua
@@ -64,7 +64,7 @@ function M.as_completion()
       docs = 'Content: ' .. v.value
     end
 
-    if v.docs then
+    if v.docs ~= nil then
       docs = v.docs .. '\n\n' .. docs
     end
 

diff --git a/spec/dotenv/.env.local b/spec/dotenv/.env.local
@@ -1,4 +1,6 @@
 # Local Documentation
+#
+# Local Documentation Next Line
 VARIABLE=Hello From Local
 # Local Test
 OTHER_VARIABLE=Other Local Value

diff --git a/spec/dotenv_spec.lua b/spec/dotenv_spec.lua
@@ -27,7 +27,10 @@ describe('Load dotenv workspace', function()
     dotenv.load()
     local all_env = dotenv.get_all_env()
     assert.are.same(3, vim.tbl_count(all_env))
-    assert.are.same({ value = 'Hello From Local', docs = 'Local Documentation' }, all_env.VARIABLE)
+    assert.are.same(
+      { value = 'Hello From Local', docs = 'Local Documentation\n\nLocal Documentation Next Line' },
+      all_env.VARIABLE
+    )
     assert.are.same({ value = 'Local Data', docs = nil }, all_env.LOCAL_VAR)
     assert.are.same({ value = 'Other Local Value', docs = 'Local Test' }, all_env.OTHER_VARIABLE)
   end)
@@ -39,7 +42,10 @@ describe('Load dotenv workspace', function()
     dotenv.load()
     local all_env = dotenv.get_all_env()
     assert.are.same(3, vim.tbl_count(all_env))
-    assert.are.same({ value = 'Hello From Local', docs = 'Local Documentation' }, all_env.VARIABLE)
+    assert.are.same(
+      { value = 'Hello From Local', docs = 'Local Documentation\n\nLocal Documentation Next Line' },
+      all_env.VARIABLE
+    )
     assert.are.same({ value = 'Local Data', docs = nil }, all_env.LOCAL_VAR)
   end)
 
@@ -65,6 +71,9 @@ describe('Completion dotenv workspace', function()
     local variable = tbl_find(all, 'VARIABLE')
     assert.are.same('VARIABLE', variable.label)
     assert.are.same('VARIABLE', variable.insertText)
-    assert.are.same('Local Documentation\n\nContent: Hello From Local', variable.documentation.value)
+    assert.are.same(
+      'Local Documentation\n\nLocal Documentation Next Line\n\nContent: Hello From Local',
+      variable.documentation.value
+    )
   end)
 end)