refactor: remove todos (#16)

lua/neotest-bash/core/result_builder.lua

@@ -11,9 +11,6 @@ function ResultBuilder.build_results(spec, result, tree)
 	local results = ResultList:new()
 	local is_file = string.match(spec.symbol, "_test.sh") ~= nil
 
-	-- TODO: as bashunit does not provide reporting yet,
-	-- we run tests running one command per test funtion
-	-- so we do not expect to find files as spec.symbol
 	if not is_file then
 		for _, node in tree:iter_nodes() do
 			local node_data = node:data()

lua/neotest-bash/core/root_finder.lua

@@ -7,7 +7,6 @@ SpecBuilder = {}
 ---@param dir string @Directory to treat as cwd
 ---@return string | nil @Absolute root dir of test suite
 function SpecBuilder.findRoot(dir)
-	-- TODO: check what happens when it's a single file
 	local matchers = {
 		".git",
 		"lib",

lua/neotest-bash/util/result_list.lua

@@ -3,7 +3,7 @@ ExitCode = {
 }
 
 local ResultList = {
-	-- TODO: needs to be static until bashunit provides reporting
+	-- needs to be static
 	_results = {},
 
 	new = function(self)

tests/core/spec_builder_spec.lua

@@ -23,7 +23,6 @@ local function mock_args_tree(data)
 end
 
 describe("spec_builder", function()
-	-- TODO: pending until bashunit has reporting
 	pending("should run command for a test file", function()
 		-- given
 		local args = mock_args_tree({
@@ -48,7 +47,6 @@ describe("spec_builder", function()
 		assert.are.same(expected_spec, result)
 	end)
 
-	-- TODO: pending until bashunit has reporting
 	pending("should run command for a test function inside a test file", function()
 		-- given
 		local args = mock_args_tree({
@@ -73,7 +71,6 @@ describe("spec_builder", function()
 		assert.are.same(expected_spec, result)
 	end)
 
-	-- TODO: we run a list of commands until we have bashunit reporting
 	async.it("should run a list of commands from a tree node selection", function()
 		-- given
 		--