@natlibfi/marc-record-validators-melinda 12.0.0-alpha.9 → 12.0.1-alpha.1

package/package.json

@@ -14,7 +14,7 @@
     "url": "https://github.com/NatLibFi/marc-record-validators-melinda"
   },
   "license": "MIT",
-  "version": "12.0.0-alpha.9",
+  "version": "12.0.1-alpha.1",
   "main": "./dist/index.js",
   "publishConfig": {
     "access": "public"
@@ -29,40 +29,42 @@
     "lint:dev": "eslint --fix ./src",
     "test": "npm run lint && npm run test:base",
     "test:base": "node --test --test-force-exit --experimental-test-coverage --test-reporter=spec './src/*.test.js' './src/**/*.test.js'",
+    "test:only": "npm run lint && npm run test:base:only",
+    "test:base:only": "node --test --test-only --test-force-exit --experimental-test-coverage --test-reporter=spec './src/*.test.js' './src/**/*.test.js'",
     "dev:test": "cross-env DEBUG=@natlibfi/* NODE_ENV=test node --watch --test --experimental-test-coverage --test-reporter=spec './src/*.test.js' './src/**/*.test.js'",
     "dev:debug": "cross-env LOG_LEVEL=debug DEBUG=@natlibfi/* NODE_ENV=test"
   },
   "dependencies": {
-    "@natlibfi/iso9-1995": "next",
-    "@natlibfi/issn-verify": "^1.0.6",
-    "@natlibfi/marc-record": "next",
-    "@natlibfi/marc-record-serializers": "next",
-    "@natlibfi/marc-record-validate": "next",
-    "@natlibfi/melinda-commons": "next",
-    "@natlibfi/sfs-4900": "next",
-    "@natlibfi/sru-client": "next",
+    "@natlibfi/iso9-1995": "^1.0.0",
+    "@natlibfi/issn-verify": "^2.0.0",
+    "@natlibfi/marc-record": "^10.0.1",
+    "@natlibfi/marc-record-serializers": "^11.0.1",
+    "@natlibfi/marc-record-validate": "^9.0.0",
+    "@natlibfi/melinda-commons": "^14.0.1",
+    "@natlibfi/sfs-4900": "^2.0.0",
+    "@natlibfi/sru-client": "^7.0.0",
     "cld3-asm": "^4.0.0",
     "clone": "^2.1.2",
     "debug": "^4.4.3",
-    "isbn3": "^2.0.0",
+    "isbn3": "^2.0.1",
     "langs": "^2.0.0",
     "undici": "^7.16.0",
     "xml2js": "^0.6.2",
     "xregexp": "^5.1.2"
   },
   "peerDependencies": {
-    "@natlibfi/marc-record-validate": "next"
+    "@natlibfi/marc-record-validate": "^9.0.0"
   },
   "devDependencies": {
-    "@natlibfi/fixugen": "next",
-    "@natlibfi/fixura": "next",
-    "cross-env": "^10.0.0",
-    "esbuild": "^0.25.12",
+    "@natlibfi/fixugen": "^3.0.0",
+    "@natlibfi/fixura": "^4.0.0",
+    "cross-env": "^10.1.0",
+    "esbuild": "^0.27.1",
     "eslint": "^9.39.1",
     "fetch-mock": "^12.6.0"
   },
   "overrides": {
     "nanoid": "^3.3.8"
   },
-  "nvolkComment": "cld3-asm 4.0.0 uses emscripten-wasm-loader ^3.0.3 which uses problematic, non-secure nanoid version 2.X.X. Isbn3 has breaking changes in 1.2.16-18 at least."
+  "nvolkComment": "cld3-asm 4.0.0 uses emscripten-wasm-loader ^3.0.3 which uses problematic, non-secure nanoid version 2.X.X."
 }

package/src/access-rights.test.js

@@ -1,7 +1,7 @@
 import assert from 'node:assert';
 import {describe, it} from 'node:test';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/access-rights.js';
+import validatorFactory from './access-rights.js';
 
 
 describe('access-rights', async () => {

package/src/addMissingField337.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/addMissingField337.js';
+import validatorFactory from './addMissingField337.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
 

package/src/addMissingField338.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/addMissingField338.js';
+import validatorFactory from './addMissingField338.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
 

package/src/cyrillux-usemarcon-replacement.test.js

@@ -1,18 +1,9 @@
 import assert from 'node:assert';
-import {after, before} from 'node:test';
 import {MarcRecord} from '@natlibfi/marc-record';
 import validatorFactory from './cyrillux-usemarcon-replacement.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
 
-before(() => {
-  MarcRecord.setValidationOptions({subfields: false, subfieldValues: false});
-});
-
-after(() => {
-  MarcRecord.setValidationOptions({});
-});
-
 generateTests({
   callback,
   path: [import.meta.dirname, '..', 'test-fixtures', 'cyrillux-usemarcon-replacement'],
@@ -23,7 +14,11 @@ generateTests({
   },
   hooks: {
     before: async () => {
+      MarcRecord.setValidationOptions({subfields: false, subfieldValues: false});
       testValidatorFactory();
+    },
+    after: async () => {
+      MarcRecord.setValidationOptions({});
     }
   }
 });

package/src/cyrillux.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/cyrillux.js';
+import validatorFactory from './cyrillux.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
 

package/src/disambiguateSeriesStatements.test.js

@@ -3,7 +3,7 @@ import {MarcRecord} from '@natlibfi/marc-record';
 import validatorFactory from './disambiguateSeriesStatements.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
-import createDebugLogger from 'debug';
+//import createDebugLogger from 'debug';
 
 generateTests({
   callback,
@@ -19,7 +19,7 @@ generateTests({
     }
   }
 });
-const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/disambiguateSeriesStatements:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/disambiguateSeriesStatements:test');
 
 async function testValidatorFactory() {
   const validator = await validatorFactory();
@@ -29,12 +29,7 @@ async function testValidatorFactory() {
   assert.equal(typeof validator.validate, 'function');
 }
 
-async function callback({getFixture, enabled = true, fix = false}) {
-  if (enabled === false) {
-    debug('TEST SKIPPED!');
-    return;
-  }
-
+async function callback({getFixture, fix = false}) {
   const validator = await validatorFactory();
   const record = new MarcRecord(getFixture('record.json'));
   const expectedResult = getFixture('expectedResult.json');

package/src/double-commas.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/double-commas.js';
+import validatorFactory from './double-commas.js';
 import {describe, it} from 'node:test';
 
 describe('double-commas', () => {

package/src/drop-terms.js

@@ -0,0 +1,162 @@
+// Author(s): Nicholas Volk
+
+import clone from 'clone';
+import createDebugLogger from 'debug';
+import {fieldToString, nvdebug} from './utils.js';
+import {getLexiconAndLanguage, getTermData, isLabel, isValidSubfield0} from './translate-terms.js';
+
+const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda:drop-terms');
+
+
+const defaultConfig = {
+  'constraints': [
+    {'tag': '648', 'lex': 'yso/fin'},
+    {'tag': '648', 'lex': 'yso/swe'},
+    {'tag': '650', 'lex': 'yso/fin'},
+    {'tag': '650', 'lex': 'yso/swe'},
+    {'tag': '651', 'lex': 'yso/fin'},
+    {'tag': '651', 'lex': 'yso/swe'},
+    {'tag': '655', 'lex': 'slm/fin'},
+    {'tag': '655', 'lex': 'slm/swe'}
+  ],
+  'keep invalid url': false, // If not true, removes illegal subfield $0. The whole field removal is later decided by 'keep 0-less'
+  'keep invalid label': false, // label ($a) is neither pref Label nor altLabel => remove whole field
+  'remove altLabel': false,
+  'remove 0-less': true
+};
+
+export default function (config = defaultConfig) {
+
+  return {
+    description: 'Drop yso and slm terms that do not match their identifiers. Use for incoming records only! Not for records already in Melinda!',
+    validate, fix
+  };
+
+  async function fix(record, validateMode = false) {
+    const clonedFields = record.fields.map(f => clone(f));
+
+    const results = await processFields(clonedFields);
+
+    const removables = results.map((f, i) => !f ? record.fields[i] : undefined).filter(f => f);
+    const modMessages = results.map((f, i) => getMod(f, i)).filter(f => f);
+    const removalMessages = removables.map(f => `Remove '${fieldToString(f)}'`);
+    const allMessages = [...modMessages, ...removalMessages];
+
+    if (validateMode) {
+      if (allMessages.length === 0) {
+        return {'message': [], 'valid': true};
+      }
+      return {'message': allMessages, 'valid': false};
+    }
+
+    removables.forEach(f => record.removeField(f));
+
+    return {message: [], fix: allMessages, valid: true};
+
+    function getMod(field, index) {
+      if (!field) {
+        return undefined;
+      }
+      const before = fieldToString(record.fields[index]);
+      const after = fieldToString(results[index]);
+      if (before !== after) {
+        if (!validateMode) {
+          record.fields[index] = field;
+        }
+        return `Modify '${before}' => '${after}'`;
+      }
+      return undefined;
+    }
+  }
+
+  function isPotentialField(f) {
+    if (!config || !config.constraints) {
+      return false;
+    }
+    return config.constraints.some(c => c.tag === f.tag && f.subfields?.some(sf => sf.code === '2' && sf.value === c.lex));
+  }
+
+  async function validate(record) {
+    return fix(record, true);
+  }
+
+
+  async function processFields(fields, results = []) {
+    const [currField, ...remainingFields] = fields;
+    if (!currField) {
+      return results;
+    }
+
+    if (!isPotentialField(currField)) { // Not interested in this field
+      return processFields(remainingFields, [...results, currField]);
+    }
+
+    removeSyntacticallyIllegalSubfield0s(currField); // iff config wants them to be removed...
+
+    const removable = await isRemovableField(currField);
+
+    return processFields(remainingFields, [...results, removable ? undefined : currField]);
+  }
+
+  function removeSyntacticallyIllegalSubfield0s(field) {
+    if (config['keep invalid url']) {
+      return;
+    }
+
+    const lexData = getLexiconAndLanguage(field);
+    if (!lexData.lang) { // This is an error of sorts. Should we proceed and remove $0s?
+      return;
+    }
+
+    field.subfields = field.subfields.filter(sf => sf.code !== '0' || isValidSubfield0(sf, lexData.lex));
+  }
+
+  async function isRemovableField(field) {
+    // nvdebug(`FOO===== ${fieldToString(field)}`);
+
+
+    // $0-less field:
+    if (!field.subfields.some(sf => sf.code === '0')) {
+      return config['remove 0-less'];
+    }
+
+    const subfield0 = field.subfields.find(sf => sf.code === '0');
+    const data = await getTermData(subfield0.value);
+    // NB! No data might be a BUG! This might delete all incoming terms if Finto is down... (we should distinguish between a miss and a failure)
+    // However, if we use this validator only for incoming records, it's fine enough.
+
+    if (data) {
+      const subfieldA = field.subfields.find(sf => sf.code === 'a');
+      if (subfieldA) {
+        const lexData = getLexiconAndLanguage(field);
+
+        // $a is the pref label. All is fine!
+        if (isLabel(data.prefLabel, subfieldA.value, lexData.lang)) {
+          debug(`altLabel found: ${subfieldA.value}`);
+          return false;
+        }
+        if (isLabel(data.altLabel, subfieldA.value, lexData.lang)) {
+          debug(`altLabel found: ${subfieldA.value}`);
+          // Oddly enough this could remove altLabel but keep totally invalid labels...
+          return config['remove altLabel'];
+        }
+        debug(`a-2-0 mismatch: ${fieldToString(field)}`);
+      }
+    }
+
+
+    if (config['keep invalid label']) {
+      // We keep the label $a. However, we can get rid of $0 if we want to (semantic reasons)
+      if (!config['keep invalid url']) {
+        nvdebug(`=============== 0-removal`);
+        field.subfields = field.subfields.filter(sf => sf.code !== '0');
+      }
+      return false;
+    }
+    return true;
+
+  }
+
+
+}
+

package/src/drop-terms.test.js

@@ -0,0 +1,81 @@
+import assert from 'node:assert/strict';
+//import createDebugLogger from 'debug';
+import fetchMock from 'fetch-mock';
+
+import validatorFactory from './drop-terms.js';
+
+import {MarcRecord} from '@natlibfi/marc-record';
+import {READERS} from '@natlibfi/fixura';
+import generateTests from '@natlibfi/fixugen';
+
+
+import {fakeData} from '../test-fixtures/translate-terms-data.js';
+
+
+const uris = [
+  'http://www.yso.fi/onto/yso/p444',
+  'http://www.yso.fi/onto/yso/p500',
+  'http://www.yso.fi/onto/yso/p13299',
+  'http://www.yso.fi/onto/yso/p111739',
+  'http://www.yso.fi/onto/yso/p6197061979',
+  'http://www.yso.fi/onto/yso/p6196061969',
+  'http://urn.fi/URN:NBN:fi:au:slm:s161'
+];
+
+const useMock = 1;
+
+
+generateTests({
+  callback,
+  path: [import.meta.dirname, '..', 'test-fixtures', 'drop-terms'],
+  useMetadataFile: true,
+  recurse: false,
+  fixura: {
+    reader: READERS.JSON
+  },
+  hooks: {
+    before: async () => {
+
+      if (useMock) {
+        fetchMock.mockGlobal()
+        .get(`https://api.finto.fi/rest/v1/data?uri=${uris[0]}&format=application%2Fjson`, {status: 200, headers: {}, body: fakeData})
+        .get(`https://api.finto.fi/rest/v1/data?uri=${uris[1]}&format=application%2Fjson`, {status: 200, headers: {}, body: fakeData})
+        .get(`https://api.finto.fi/rest/v1/data?uri=${uris[2]}&format=application%2Fjson`, {status: 200, headers: {}, body: fakeData})
+        .get(`https://api.finto.fi/rest/v1/data?uri=${uris[3]}&format=application%2Fjson`, {status: 200, headers: {}, body: fakeData})
+        .get(`https://api.finto.fi/rest/v1/data?uri=${uris[4]}&format=application%2Fjson`, {status: 200, headers: {}, body: fakeData})
+        .get(`https://api.finto.fi/rest/v1/data?uri=${uris[5]}&format=application%2Fjson`, {status: 200, headers: {}, body: fakeData})
+        .get(`https://api.finto.fi/rest/v1/data?uri=${uris[6]}&format=application%2Fjson`, {status: 200, headers: {}, body: fakeData});
+      }
+
+      testValidatorFactory();
+    }
+  }
+});
+
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/drop-terms:test');
+
+async function testValidatorFactory() {
+  const validator = await validatorFactory();
+
+  assert.equal(typeof validator, 'object');
+  assert.equal(typeof validator.description, 'string');
+  assert.equal(typeof validator.validate, 'function');
+}
+
+async function callback({getFixture, fix = false, config = undefined}) {
+  const validator = await validatorFactory(config);
+  const record = new MarcRecord(getFixture('record.json'));
+  const expectedResult = getFixture('expectedResult.json');
+  // console.log(expectedResult); // eslint-disable-line
+
+  if (!fix) {
+    const result = await validator.validate(record);
+    assert.deepEqual(result, expectedResult);
+    const originalRecord = new MarcRecord(getFixture('record.json'));
+    assert.deepEqual(record, originalRecord); // Validation should not change the record
+    return;
+  }
+  const expectedResult2 = new MarcRecord(expectedResult);
+  await validator.fix(record);
+  assert.deepEqual(record, expectedResult2);
+}

package/src/empty-fields.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/empty-fields.js';
+import validatorFactory from './empty-fields.js';
 import {after, before, describe, it} from 'node:test';
 
 before(() => {

package/src/ending-punctuation.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/ending-punctuation.js';
+import validatorFactory from './ending-punctuation.js';
 import {describe, it} from 'node:test';
 
 
@@ -1598,7 +1598,7 @@ describe('ending-punctuation', () => {
             ]
           },
           { // Exception: term ending in dot:
-            tag : '648',
+            tag: '648',
             ind1: ' ',
             ind2: '7',
             subfields: [
@@ -1628,15 +1628,19 @@ describe('ending-punctuation', () => {
       const recordValid650FinNo = new MarcRecord({
         leader: '',
         fields: [
-          {tag: '650', ind1: ' ', ind2: '7', subfields: [
-            {code: 'a', value: 'kirjastot'},
-            {code: 'x', value: 'atk-järjestelmät'},
-            {code: '2', value: 'kauno/fin'}
-          ]},
-          {tag: '650', ind1: ' ', ind2: '7', subfields: [
-            {code: 'a', value: 'ajovalot'},
-            {code: '2', value: 'juho'}
-          ]}
+          {
+            tag: '650', ind1: ' ', ind2: '7', subfields: [
+              {code: 'a', value: 'kirjastot'},
+              {code: 'x', value: 'atk-järjestelmät'},
+              {code: '2', value: 'kauno/fin'}
+            ]
+          },
+          {
+            tag: '650', ind1: ' ', ind2: '7', subfields: [
+              {code: 'a', value: 'ajovalot'},
+              {code: '2', value: 'juho'}
+            ]
+          }
         ]
       });
 
@@ -1765,15 +1769,19 @@ describe('ending-punctuation', () => {
       const recordInvalid650FinYes = new MarcRecord({
         leader: '',
         fields: [
-          {tag: '650', ind1: ' ', ind2: '7', subfields: [
-            {code: 'a', value: 'kirjastot'},
-            {code: 'x', value: 'atk-järjestelmät.'},
-            {code: '2', value: 'kauno/fin'}
-          ]},
-          {tag: '650', ind1: ' ', ind2: '7', subfields: [
-            {code: 'a', value: 'ajovalot.'},
-            {code: '2', value: 'juho'}
-          ]}
+          {
+            tag: '650', ind1: ' ', ind2: '7', subfields: [
+              {code: 'a', value: 'kirjastot'},
+              {code: 'x', value: 'atk-järjestelmät.'},
+              {code: '2', value: 'kauno/fin'}
+            ]
+          },
+          {
+            tag: '650', ind1: ' ', ind2: '7', subfields: [
+              {code: 'a', value: 'ajovalot.'},
+              {code: '2', value: 'juho'}
+            ]
+          }
         ]
       });
 

package/src/field-008-18-34-character-groups.test.js

@@ -1,9 +1,9 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/field-008-18-34-character-groups.js';
+import validatorFactory from './field-008-18-34-character-groups.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
-import createDebugLogger from 'debug';
+//import createDebugLogger from 'debug';
 
 generateTests({
   callback,
@@ -19,7 +19,7 @@ generateTests({
     }
   }
 });
-const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/field-008-18-34-character-groups:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/field-008-18-34-character-groups:test');
 
 async function testValidatorFactory() {
   const validator = await validatorFactory();
@@ -30,12 +30,7 @@ async function testValidatorFactory() {
   assert.equal(typeof validator.validate, 'function');
 }
 
-async function callback({getFixture, enabled = true, fix = false}) {
-  if (enabled === false) {
-    debug('TEST SKIPPED!');
-    return;
-  }
-
+async function callback({getFixture, fix = false}) {
   const validator = await validatorFactory();
   const record = new MarcRecord(getFixture('record.json'));
   const expectedResult = getFixture('expectedResult.json');

package/src/field-505-separators.test.js

@@ -3,7 +3,7 @@ import {MarcRecord} from '@natlibfi/marc-record';
 import validatorFactory from './field-505-separators.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
-import createDebugLogger from 'debug';
+//import createDebugLogger from 'debug';
 
 generateTests({
   callback,
@@ -19,7 +19,7 @@ generateTests({
     }
   }
 });
-const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/field-505-separators:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/field-505-separators:test');
 
 async function testValidatorFactory() {
   const validator = await validatorFactory();
@@ -29,12 +29,7 @@ async function testValidatorFactory() {
   assert.equal(typeof validator.validate, 'function');
 }
 
-async function callback({getFixture, enabled = true, fix = false}) {
-  if (enabled === false) {
-    debug('TEST SKIPPED!');
-    return;
-  }
-
+async function callback({getFixture, fix = false}) {
   const validator = await validatorFactory();
 
   const recordFixture = getFixture('record.json');

package/src/field-521-fix.test.js

@@ -3,7 +3,7 @@ import {MarcRecord} from '@natlibfi/marc-record';
 import validatorFactory from './field-521-fix.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
-import createDebugLogger from 'debug';
+//import createDebugLogger from 'debug';
 
 generateTests({
   callback,
@@ -19,7 +19,7 @@ generateTests({
     }
   }
 });
-const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/fix521:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/fix521:test');
 
 async function testValidatorFactory() {
   const validator = await validatorFactory();
@@ -29,12 +29,7 @@ async function testValidatorFactory() {
   assert.equal(typeof validator.validate, 'function');
 }
 
-async function callback({getFixture, enabled = true, fix = false}) {
-  if (enabled === false) {
-    debug('TEST SKIPPED!');
-    return;
-  }
-
+async function callback({getFixture, fix = false}) {
   const validator = await validatorFactory();
   const record = new MarcRecord(getFixture('record.json'));
   const expectedResult = getFixture('expectedResult.json');

package/src/field-exclusion.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/field-exclusion.js';
+import validatorFactory from './field-exclusion.js';
 import {describe, it} from 'node:test';
 
 // Factory validation
@@ -729,13 +729,15 @@ describe('field-exclusion', () => {
     it('Finds the record invalid (spec)', async () => {
       const validator = await validatorFactory(config);
       const {valid, message} = await validator.validate(recordInvalidMulti);
-      assert.deepEqual({valid, message}, {valid: false, message: [
-        'Field $648 should be excluded',
-        'Field $650 should be excluded',
-        'Field $650 should be excluded',
-        'Field $651 should be excluded',
-        'Field $655 should be excluded'
-      ]});
+      assert.deepEqual({valid, message}, {
+        valid: false, message: [
+          'Field $648 should be excluded',
+          'Field $650 should be excluded',
+          'Field $650 should be excluded',
+          'Field $651 should be excluded',
+          'Field $655 should be excluded'
+        ]
+      });
     });
 
     it('Repairs invalid multi record', async () => {

package/src/field-structure.test.js

@@ -1,7 +1,7 @@
 import assert from 'node:assert';
 import {describe, it} from 'node:test';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/field-structure.js';
+import validatorFactory from './field-structure.js';
 
 
 // Factory validation

package/src/fields-present.test.js

@@ -1,7 +1,7 @@
 import {describe, it} from 'node:test';
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/fields-present.js';
+import validatorFactory from './fields-present.js';