chore: Do not use deprecated vim.tbl_flatten() (#36)

* Do not use deprecated vim.tbl_flatten()

* Format code

---------

Co-authored-by: Jhon Pedroza <jhon@pedroza.me>

Author: soundmonster

README.md

@@ -53,7 +53,7 @@ require("neotest").setup({
       -- Must be a function that receives the mix command as a table, to return a dynamic value
       -- Default: function(cmd) return cmd end
       post_process_command = function(cmd)
-        return vim.tbl_flatten({{"env", "FOO=bar"}, cmd})
+        return vim.iter({{"env", "FOO=bar"}, cmd}):flatten():totable()
       end,
       -- Delays writes so that results are updated at most every given milliseconds
       -- Decreasing this number improves snappiness at the cost of performance

lua/neotest-elixir/core.lua

@@ -132,27 +132,30 @@ function M.build_mix_command(
   neotest_args,
   relative_to_func
 )
-  return vim.tbl_flatten({
-    {
-      "elixir",
-    },
-    -- deferent tasks have different options
-    -- for example, `test.interactive` needs to load a custom runner
-    options_for_task(mix_task_func()),
-    {
-      "-S",
-      "mix",
-      mix_task_func(), -- `test` is default
-    },
-    -- default is ExUnit.CLIFormatter
-    build_formatters(extra_formatters_func()),
-    -- default is {}
-    -- maybe `test.interactive` has different args with `test`
-    mix_task_args_func(),
-    neotest_args.extra_args or {},
-    -- test file or directory or testfile:line
-    test_target(position, relative_to_func),
-  })
+  return vim
+    .iter({
+      {
+        "elixir",
+      },
+      -- deferent tasks have different options
+      -- for example, `test.interactive` needs to load a custom runner
+      options_for_task(mix_task_func()),
+      {
+        "-S",
+        "mix",
+        mix_task_func(), -- `test` is default
+      },
+      -- default is ExUnit.CLIFormatter
+      build_formatters(extra_formatters_func()),
+      -- default is {}
+      -- maybe `test.interactive` has different args with `test`
+      mix_task_args_func(),
+      neotest_args.extra_args or {},
+      -- test file or directory or testfile:line
+      test_target(position, relative_to_func),
+    })
+    :flatten()
+    :totable()
 end
 
 -- public only for testing

lua/neotest-elixir/init.lua

@@ -54,13 +54,16 @@ function ElixirNeotestAdapter._generate_id(position, parents)
     return (relative_path .. ":" .. line_num)
   else
     return table.concat(
-      vim.tbl_flatten({
-        position.path,
-        vim.tbl_map(function(pos)
-          return pos.name
-        end, parents),
-        position.name,
-      }),
+      vim
+        .iter({
+          position.path,
+          vim.tbl_map(function(pos)
+            return pos.name
+          end, parents),
+          position.name,
+        })
+        :flatten()
+        :totable(),
       "::"
     )
   end
@@ -178,7 +181,8 @@ end
 ---@async
 ---@return neotest.Tree | nil
 function ElixirNeotestAdapter.discover_positions(path)
-  local test_block_id_list = vim.tbl_flatten({ { "test", "feature", "property" }, get_extra_block_identifiers() })
+  local test_block_id_list =
+    vim.iter({ { "test", "feature", "property" }, get_extra_block_identifiers() }):flatten():totable()
   for index, value in ipairs(test_block_id_list) do
     test_block_id_list[index] = '"' .. value .. '"'
   end