fix tests

Author: severo

tests/cases.ts

@@ -1,17 +1,5 @@
 import { RECORD_SEP, UNIT_SEP } from '../src/options/constants.js'
 
-export const cases = [
-  {
-    description: 'One row',
-    text: 'A,b,c',
-    expected: {
-      rows: [['A', 'b', 'c']],
-      errors: [],
-      meta: { delimiter: ',', renamedHeaders: null },
-    },
-  },
-]
-
 export const CORE_PARSER_TESTS = [
   {
     description: 'One row',
@@ -198,8 +186,6 @@ export const CORE_PARSER_TESTS = [
         type: 'Quotes',
         code: 'MissingQuotes',
         message: 'Quoted field unterminated',
-        row: 0,
-        index: 3,
       }],
     },
   },
@@ -213,8 +199,6 @@ export const CORE_PARSER_TESTS = [
         type: 'Quotes',
         code: 'InvalidQuotes',
         message: 'Trailing quote on quoted field is malformed',
-        row: 0,
-        index: 1,
       }],
     },
   },
@@ -228,15 +212,11 @@ export const CORE_PARSER_TESTS = [
         type: 'Quotes',
         code: 'InvalidQuotes',
         message: 'Trailing quote on quoted field is malformed',
-        row: 0,
-        index: 3,
       },
       {
         type: 'Quotes',
         code: 'MissingQuotes',
         message: 'Quoted field unterminated',
-        row: 0,
-        index: 3,
       }],
     },
   },
@@ -250,15 +230,11 @@ export const CORE_PARSER_TESTS = [
         type: 'Quotes',
         code: 'InvalidQuotes',
         message: 'Trailing quote on quoted field is malformed',
-        row: 0,
-        index: 3,
       },
       {
         type: 'Quotes',
         code: 'MissingQuotes',
         message: 'Quoted field unterminated',
-        row: 0,
-        index: 3,
       }],
     },
   },
@@ -272,15 +248,11 @@ export const CORE_PARSER_TESTS = [
         type: 'Quotes',
         code: 'InvalidQuotes',
         message: 'Trailing quote on quoted field is malformed',
-        row: 0,
-        index: 3,
       },
       {
         type: 'Quotes',
         code: 'MissingQuotes',
         message: 'Quoted field unterminated',
-        row: 0,
-        index: 3,
       }],
     },
   },
@@ -714,7 +686,6 @@ export const PARSE_TESTS = [
   //       type: 'FieldMismatch',
   //       code: 'TooFewFields',
   //       message: 'Too few fields: expected 3 fields but parsed 2',
-  //       row: 0,
   //     }],
   //   },
   // },
@@ -728,7 +699,6 @@ export const PARSE_TESTS = [
   //       type: 'FieldMismatch',
   //       code: 'TooManyFields',
   //       message: 'Too many fields: expected 3 fields but parsed 5',
-  //       row: 0,
   //     }],
   //   },
   // },

tests/parser.test.ts

@@ -9,7 +9,7 @@ describe('Papaparse core parser tests', () => {
       const config = test.config || {}
       const result = [...parse(test.text, config)]
       const data = result.map(({ row }) => row)
-      const errors = result.flatMap(({ errors: rowErrors }, row) => rowErrors.map(error => ({ ...error, row })))
+      const errors = result.flatMap(({ errors }) => errors)
       expect(data).toEqual(test.expected.data)
       expect(errors).toEqual(test.expected.errors)
       if (test.expected.meta?.charCount !== undefined) {

tests/text.test.ts

@@ -2,7 +2,7 @@ import { describe, expect, it } from 'vitest'
 
 import { parseText } from '../src/text'
 import { testEmptyLine } from '../src/utils'
-import { cases, PARSE_TESTS } from './cases'
+import { PARSE_TESTS } from './cases'
 
 describe('parseText', () => {
   it('decodes a string', () => {
@@ -27,12 +27,6 @@ describe('parseText', () => {
     // For now: only one iteration per chunk
     expect(i).toBe(1)
   })
-  it.for(cases)('$description', ({ text, expected }) => {
-    const results = [...parseText(text)]
-
-    expect(results.map(({ row }) => row)).toEqual(expected.rows)
-    // TODO(SL): check the metadata and the errors too
-  })
 
   it.for([
     { description: '', text: 'a,b,c\n\nd,e,f', expected: [['a', 'b', 'c'], ['d', 'e', 'f']] },