Merge pull request #1392 from nellh/issue-56-evidence

fix: Update issue 56 to provide evidence in string format

Author: rwblair

bids-validator/validators/tsv/__tests__/checkAge89.spec.js

@@ -0,0 +1,14 @@
+import { checkAge89 } from '../checkAge89'
+
+describe('checkAge89()', () => {
+  it('returns evidence in the expected string format', () => {
+    // Evidence should always be a human-readable string
+    const issues = []
+    const exampleParticipants = [['age'], [90]]
+    const mockFile = {}
+    checkAge89(exampleParticipants, mockFile, issues)
+    expect(issues).toHaveLength(1)
+    expect(issues[0]).toHaveProperty('evidence')
+    expect(typeof issues[0].evidence).toBe('string')
+  })
+})

bids-validator/validators/tsv/checkAge89.js

@@ -1,6 +1,6 @@
 const Issue = require('../../utils').issues.Issue
 
-const checkAge89 = function(rows, file, issues) {
+export const checkAge89 = function(rows, file, issues) {
   const header = rows[0]
   const ageIdColumn = header.indexOf('age')
   for (let a = 0; a < rows.length; a++) {
@@ -11,7 +11,7 @@ const checkAge89 = function(rows, file, issues) {
       issues.push(
         new Issue({
           file: file,
-          evidence: line,
+          evidence: line.join(','),
           line: a + 1,
           reason: 'age of participant is above 89 ',
           code: 56,