feat: Always error on incorrect TSV types

Author: effigies

src/issues/list.ts

@@ -123,11 +123,6 @@ export const bidsIssues: IssueDefinitionRecord = {
     reason:
       'A value in a column did not match the acceptable type for that column headers specified format.',
   },
-  TSV_VALUE_INCORRECT_TYPE_NONREQUIRED: {
-    severity: 'warning',
-    reason:
-      'A value in a column did not match the acceptable type for that column headers specified format.',
-  },
   TSV_COLUMN_TYPE_REDEFINED: {
     severity: 'warning',
     reason:

src/schema/tables.test.ts

@@ -69,7 +69,7 @@ Deno.test('tables eval* tests', async (t) => {
     const rule = schemaDefs.rules.tabular_data.modality_agnostic.Scans
     evalColumns(rule, context, schema, 'rules.tabular_data.modality_agnostic.Scans')
     assertEquals(
-      context.dataset.issues.get({ code: 'TSV_VALUE_INCORRECT_TYPE_NONREQUIRED' }).length,
+      context.dataset.issues.get({ code: 'TSV_VALUE_INCORRECT_TYPE' }).length,
       1,
     )
   })
@@ -159,7 +159,7 @@ Deno.test('tables eval* tests', async (t) => {
 
     // Overriding the default sex definition uses the provided values
     // Values in the default definition may raise issues
-    issues = context.dataset.issues.get({ code: 'TSV_VALUE_INCORRECT_TYPE_NONREQUIRED' })
+    issues = context.dataset.issues.get({ code: 'TSV_VALUE_INCORRECT_TYPE' })
     assertEquals(issues.length, 1)
     assertEquals(issues[0].subCode, 'sex')
     assertEquals(issues[0].line, 4)

src/schema/tables.ts

@@ -251,7 +251,7 @@ export function evalColumns(
     const spec = compileSignature(signature, schema.objects.formats)
 
     const issue = {
-      code: 'TSV_VALUE_INCORRECT_TYPE' + (requirement != 'required' ? '_NONREQUIRED' : ''),
+      code: 'TSV_VALUE_INCORRECT_TYPE',
       subCode: name,
       location: context.path,
       rule: schemaPath,