fix: clear out test inconsistencies

Author: Danielku15

.vscode-test.mjs

@@ -37,6 +37,7 @@ let createCommonOptions = (label) => {
 
       mocha: {
         ui: 'bdd',
+        timeout: 60_000,
       },
     };
   }

src/extract/evaluate.ts

@@ -87,15 +87,15 @@ export async function extractWithEvaluation(
         try {
           const startMapped = originalPositionFor(sourceMap, {
             line: startLine,
-            column: startColumn,
+            column: startColumn - 1,
           });
           if (startMapped.line !== null) {
             startLine = startMapped.line;
             startColumn = startMapped.column + 1; // columns are 0-based in trace-mapping lib
           }
           const endMapped = originalPositionFor(sourceMap, {
             line: endLine,
-            column: endColumn,
+            column: endColumn - 1,
           });
           if (endMapped.line !== null) {
             endLine = endMapped.line;

src/source-map.ts

@@ -13,24 +13,26 @@ const smUrlComment = '//# sourceMappingURL=';
 
 export interface IMappingAccessor {
   /**
-   * @param line base-1 line
+   * @param line base-0 line
    * @param col base-0 column
    */
   originalPositionFor(line: number, col: number): vscode.Location;
 }
 
 export const identityMapping = (file: vscode.Uri): IMappingAccessor => ({
   originalPositionFor(line, col) {
-    // VS Code positions are base 0, adjust the line
     return new vscode.Location(file, new vscode.Position(line, col));
   },
 });
 
 const smMappingAccessor = (file: vscode.Uri, sm: TraceMap): IMappingAccessor => ({
   originalPositionFor(line, column) {
-    const { source, line: smLine, column: smCol } = originalPositionFor(sm, { line, column });
+    const {
+      source,
+      line: smLine,
+      column: smCol,
+    } = originalPositionFor(sm, { line: line + 1, column: column });
     if (!source) {
-      // VS Code positions are base 0, adjust the line
       return new vscode.Location(file, new vscode.Position(line, column));
     }
 

test-workspaces/simple/goodbye.test.js

@@ -1,42 +1,7 @@
 const { strictEqual } = require('node:assert');
 
 describe('math', () => {
-
   it('division', async () => {
     strictEqual(2 / 1, 2);
   });
-
-  it('test', ()=>{ })
-
-  it('test2', ()=>{ })
-
-
-
-
-
-
-
-
-  it('test3', ()=>{ })
-
 });
-
-
-describe('xx', async () => {
-
-
-
-  it('test', ()=>{ })
-
-  it('test2', ()=>{ })
-
-
-
-
-
-
-
-
-  it('test3', ()=>{ })
-
-})