fix: 'Tasks: Toggle task done' keeps list markers if no Global Filter (#3708)

This fixes #3708

Author: claremacrae

src/Commands/ToggleDone.ts

@@ -102,7 +102,7 @@ export const toggleLine = (line: string, path: string): EditorInsertion => {
             const statusString = regexMatch[3];
             const status = StatusRegistry.getInstance().bySymbol(statusString);
             const newStatusString = status.nextStatusSymbol;
-            return { text: line.replace(TaskRegularExpressions.taskRegex, `$1- [${newStatusString}] $4`) };
+            return { text: line.replace(TaskRegularExpressions.taskRegex, `$1$2 [${newStatusString}] $4`) };
         } else if (TaskRegularExpressions.listItemRegex.test(line)) {
             // Convert the list item to a checklist item.
             const text = line.replace(TaskRegularExpressions.listItemRegex, '$1$2 [ ]');

tests/Commands/ToggleDone.test.ts

@@ -115,7 +115,7 @@ describe('ToggleDone', () => {
         testToggleLine('1. |foobar', '1. [ ] foobar|');
     });
 
-    it.failing('should complete a task', () => {
+    it('should complete a task', () => {
         testToggleLine('|- [ ] ', '|- [x]  ✅ 2022-09-04');
         testToggleLine('- [ ] |', '- [x] | ✅ 2022-09-04');
         testToggleLine('- [ ] description|', '- [x] description| ✅ 2022-09-04');
@@ -137,7 +137,7 @@ describe('ToggleDone', () => {
         testToggleLine('- [ ] I have a |proper description', '- [x] I have a |proper description');
     });
 
-    it.failing('should un-complete a completed task', () => {
+    it('should un-complete a completed task', () => {
         testToggleLine('|- [x]  ✅ 2022-09-04', '|- [ ] ');
         testToggleLine('1. [x]  ✅ 2022-09-04|', '1. [ ] |');