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

package/src/fix-33X.test.js

@@ -22,7 +22,7 @@ generateTests({
 async function testValidatorFactory() {
   const validator = await validatorFactory();
 
-  assert.equal(typeof validator, 'objecr');
+  assert.equal(typeof validator, 'object');
   assert.equal(typeof validator.description, 'string');
   assert.equal(typeof validator.validate, 'function');
   assert.equal(typeof validator.fix, 'function');

package/src/fix-country-codes.test.js

@@ -3,7 +3,7 @@ import {MarcRecord} from '@natlibfi/marc-record';
 import validatorFactory from './fix-country-codes.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/fix-country-codes:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/fix-country-codes: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/fix-sami-041.js

@@ -0,0 +1,113 @@
+// Author(s): Nicholas Volk
+
+//import createDebugLogger from 'debug';
+import clone from 'clone';
+
+import {fieldToString} from './utils.js';
+
+// const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda:fix-sami-041');
+
+
+
+export default function () {
+  /* 'sma': eteläsaame, 'sme': pohjoissaame, 'smj': luulajansaame, 'smn': inarinsaame, 'sms': koltansaame */
+  const samiLanguages = ['sma', 'sme', 'smj', 'smn', 'sms'];
+  const relevantSubfieldCodes = ['a', 'd']; // Subfield codes that should also have 'smi' if a sami langauge is used. Confirmed by A.R. via Slack 2025-12-05
+
+  return {
+    description: 'Add corresponing \'smi\' subfield before a specific sami language subfields, if needed',
+    validate, fix
+  };
+
+  function fix(record, validateMode = false) {
+    const relevantFields = record.fields.filter(f => isRelevantField(f, validateMode)).map(f => validateMode ? clone(f) : f); // NV! relevant fields are cloned in validation mode!
+    // Nothing to do:
+    if (relevantFields.length === 0) {
+      if (validateMode) {
+        return {message: [], valid: true};
+      }
+      return {message: [], fix: [], valid: true};
+    }
+
+    const relevantFieldsAsStrings = relevantFields.map(f => fieldToString(f)); // get original values
+
+    relevantFields.forEach(f => processField(f));
+    const modFieldsAsStrings = relevantFields.map(f => fieldToString(f));
+    const report = [...updateAndReport008(), ...createReport(relevantFieldsAsStrings, modFieldsAsStrings)];
+
+    if (validateMode) {
+      return {'message': report, 'valid': false};
+    }
+
+    return {message: [], fix: report, valid: true};
+
+    function updateAndReport008() {
+      const [f008] = record.get('008').map(f => validateMode ? clone(f) : f);
+
+      if (!f008) {
+        return [];
+      }
+      const currLang = f008.value.substr(35, 3);
+      if (!samiLanguages.includes(currLang)) { // NB! If original 008/35-37 was not a sami language, we don't change anything!
+        return [];
+      }
+      const origValue = f008.value;
+      const firstRelevantSubfield = relevantFields[0].subfields.find(sf => sf.code === 'a' || sf.code === 'd'); // NB! don't use relevantSubfieldCodes here!
+      if (firstRelevantSubfield.value === 'smi') { // First relevant subfield is 
+        f008.value = `${f008.value.substr(0, 35)}smi${f008.value.substr(38)}`;
+      }
+      return createReport([origValue], [f008.value]);
+    }
+  }
+
+  function createReport(origArray, modArray) {
+    return origArray.map((entry, index) => createEntry(entry, index));
+
+    function createEntry(item, index) {
+      return `'${item}' => '${modArray[index]}'`;
+    }
+  }
+
+  function validate(record) {
+    return fix(record, true);
+  }
+
+  function processField(f) {
+    f.subfields = prosessSubfields(f.subfields);
+    return f;
+  }
+
+  function prosessSubfields(incomingSubfields, outgoingSubfields = []) {
+    const [currSubfield, ...otherSubfields] = incomingSubfields;
+    if (!currSubfield) {
+      return outgoingSubfields;
+    }
+    if (!isRelevantSamiSubfield(currSubfield, [...outgoingSubfields, ...otherSubfields])) {
+      return prosessSubfields(otherSubfields, [...outgoingSubfields, currSubfield]);
+    }
+
+    const smiSubfield = {
+      code: currSubfield.code,
+      value: 'smi'
+    };
+    return prosessSubfields(otherSubfields, [...outgoingSubfields, smiSubfield, currSubfield]);
+  }
+
+  function isRelevantField(f) {
+    if (f.tag !== '041' || !f.subfields || f.subfields.some(sf => sf.code === '2')) {
+      return false;
+    }
+    return f.subfields.some(sf => isRelevantSamiSubfield(sf, f.subfields)); // it's ok to pass sf in f.subfields also
+  }
+
+  function isRelevantSamiSubfield(sf, otherSubfields) {
+    if (!relevantSubfieldCodes.includes(sf.code) || !samiLanguages.includes(sf.value)) {
+      return false;
+    }
+    if (otherSubfields.some(sf2 => sf2.code === sf.code && sf2.value === 'smi')) {
+      return false;
+    }
+    return true;
+  }
+}
+

package/src/fix-sami-041.test.js

@@ -0,0 +1,52 @@
+import assert from 'node:assert/strict';
+//import createDebugLogger from 'debug';
+
+import validatorFactory from './fix-sami-041.js';
+
+import {MarcRecord} from '@natlibfi/marc-record';
+import {READERS} from '@natlibfi/fixura';
+import generateTests from '@natlibfi/fixugen';
+
+generateTests({
+  callback,
+  path: [import.meta.dirname, '..', 'test-fixtures', 'fix-sami-041'],
+  useMetadataFile: true,
+  recurse: false,
+  fixura: {
+    reader: READERS.JSON
+  },
+  hooks: {
+    before: async () => {
+      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}) {
+  const validator = await validatorFactory();
+  const record = new MarcRecord(getFixture('record.json'));
+  const expectedResult = getFixture('expectedResult.json');
+  //const expectedResult = new MarcRecord(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); // The record should now change in validation-only
+    return;
+  }
+
+  await validator.fix(record);
+  assert.deepEqual(record, new MarcRecord(expectedResult));
+}

package/src/fixRelatorTerms.test.js

@@ -1,9 +1,9 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/fixRelatorTerms.js';
+import validatorFactory from './fixRelatorTerms.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
-import createDebugLogger from 'debug';
+//import createDebugLogger from 'debug';
 
 generateTests({
   callback,
@@ -20,7 +20,7 @@ generateTests({
   }
 });
 
-const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/fixRelatorTerms:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/fixRelatorTerms: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/fixed-fields.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/fixed-fields.js';
+import validatorFactory from './fixed-fields.js';
 import {describe, it} from 'node:test';
 
 
@@ -27,11 +27,13 @@ describe('fixed-fields: language', () => {
       const validator = await validatorFactory([
         {leader: true, length: 6, rules: [{position: [0, 6], pattern: /[abcdefg]/u}]},
         {tag: /^FOO$/u, length: 12, rules: [{position: 0, pattern: /f/u}]},
-        {tag: /^BAR$/u, length: 6, rules: [
-          {position: 0, pattern: /[fb]/u},
-          {position: 1, pattern: /a/u, dependencies: [{position: 0, pattern: /b/u}]},
-          {position: [2, 3], pattern: /u/u, dependencies: [{position: 0, pattern: /[^b]/u}]}
-        ]}
+        {
+          tag: /^BAR$/u, length: 6, rules: [
+            {position: 0, pattern: /[fb]/u},
+            {position: 1, pattern: /a/u, dependencies: [{position: 0, pattern: /b/u}]},
+            {position: [2, 3], pattern: /u/u, dependencies: [{position: 0, pattern: /[^b]/u}]}
+          ]
+        }
       ]);
       const record = new MarcRecord({
         leader: 'bacgfe',
@@ -56,11 +58,13 @@ describe('fixed-fields: language', () => {
       const validator = await validatorFactory([
         {leader: true, length: 6, rules: [{position: [0, 6], pattern: /[abcdefg]/u}]},
         {tag: /^FOO$/u, length: 12, rules: [{position: 0, pattern: /f/u}]},
-        {tag: /^BAR$/u, length: 6, rules: [
-          {position: 0, pattern: /[fb]/u},
-          {position: 1, pattern: /a/u, dependencies: [{position: 0, pattern: /b/u}]},
-          {position: [2, 3], pattern: /u/u, dependencies: [{position: 0, pattern: /[^a]/u}]}
-        ]},
+        {
+          tag: /^BAR$/u, length: 6, rules: [
+            {position: 0, pattern: /[fb]/u},
+            {position: 1, pattern: /a/u, dependencies: [{position: 0, pattern: /b/u}]},
+            {position: [2, 3], pattern: /u/u, dependencies: [{position: 0, pattern: /[^a]/u}]}
+          ]
+        },
         {tag: /^FUB$/u, length: 5}
       ]);
       const record = new MarcRecord({
@@ -83,13 +87,15 @@ describe('fixed-fields: language', () => {
 
       const result = await validator.validate(record);
 
-      assert.deepEqual(result, {valid: false, messages: [
-        'Leader has invalid values at positions: 3 (Rule index 0)',
-        'Field FOO has invalid values at positions: 0 (Rule index 0)',
-        'Field BAR has invalid values at positions: 1 (Rule index 1)',
-        'Field BAR has invalid values at positions: 2,3 (Rule index 2)',
-        'Field FUB has invalid length'
-      ]});
+      assert.deepEqual(result, {
+        valid: false, messages: [
+          'Leader has invalid values at positions: 3 (Rule index 0)',
+          'Field FOO has invalid values at positions: 0 (Rule index 0)',
+          'Field BAR has invalid values at positions: 1 (Rule index 1)',
+          'Field BAR has invalid values at positions: 2,3 (Rule index 2)',
+          'Field FUB has invalid length'
+        ]
+      });
     });
   });
 });

package/src/identical-fields.test.js

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

package/src/index.js

@@ -21,6 +21,7 @@ import Fix33X from './fix-33X.js';
 import FixCountryCodes from './fix-country-codes.js';
 import FixLanguageCodes from './fix-language-codes.js';
 import FixRelatorTerms from './fixRelatorTerms.js';
+import FixSami041 from './fix-sami-041.js';
 import FixedFields from './fixed-fields.js';
 import IdenticalFields from './identical-fields.js';
 import IndicatorFixes from './indicator-fixes.js';
@@ -39,6 +40,7 @@ import NormalizeUTF8Diacritics from './normalize-utf8-diacritics.js';
 import Punctuation from './punctuation/index.js';
 import Punctuation2 from './punctuation2.js';
 import ReindexSubfield6OccurenceNumbers from './reindexSubfield6OccurenceNumbers.js';
+import Remove041zxx from './remove-041-zxx.js';
 import RemoveDuplicateDataFields from './removeDuplicateDataFields.js';
 import RemoveInferiorDataFields from './removeInferiorDataFields.js';
 import ResolvableExtReferences from './resolvable-ext-references-melinda.js';
@@ -52,6 +54,7 @@ import SortTags from './sort-tags.js';
 import SubfieldValueNormalizations from './subfieldValueNormalizations.js';
 import SubfieldExclusion from './subfield-exclusion.js';
 import Sync007And300 from './sync-007-and-300.js';
+import SyncLanguage from './sync-language.js';
 import TranslateTerms from './translate-terms.js';
 import TypeOfDateF008 from './typeOfDate-008.js';
 import UnicodeDecomposition from './unicode-decomposition.js';
@@ -105,11 +108,13 @@ export {
   FixCountryCodes,
   FixLanguageCodes,
   FixRelatorTerms,
+  FixSami041,
   FixedFields,
   IdenticalFields,
   IndicatorFixes,
   IsbnIssn,
   ItemLanguage,
+
   MergeField500Lisapainokset,
   MergeRelatorTermFields,
   Modernize502, //
@@ -123,6 +128,7 @@ export {
   Punctuation2,
   ResolveOrphanedSubfield6s, // Do this before reindexing! (thus I'm not sticking with alphabetical order here)
   ReindexSubfield6OccurenceNumbers,
+  Remove041zxx,
   RemoveDuplicateDataFields,
   RemoveInferiorDataFields,
   ResolvableExtReferences,
@@ -133,8 +139,9 @@ export {
   SubfieldExclusion,
   SubfieldValueNormalizations,
   Sync007And300,
-  TypeOfDateF008,
+  SyncLanguage,
   TranslateTerms,
+  TypeOfDateF008,
   UnicodeDecomposition,
   UpdateField540,
   Urn,

package/src/indicator-fixes.js

@@ -241,6 +241,17 @@ function getLanguages(record) {
 
 }
 
+function translated041Indicator1(field){
+  if (field.tag !== '041' || !field.subfields) {
+    return;
+  }
+  if (field.ind1 !== '1' && field.subfields.some(sf => ['h', 'k', 'm', 'n'].includes(sf.code))) {
+    field.ind1 = '1';
+    return;
+  }
+
+}
+
 function recordNormalizeIndicators(record) {
   recordNormalize490(record);
 
@@ -252,6 +263,7 @@ function recordNormalizeIndicators(record) {
 }
 
 function fieldNormalizeIndicators(field, record, languages) {
+  translated041Indicator1(field);
   normalize084Indicator1(field);
   normalize245Indicator1(field, record);
   noDisplayConstantGenerated520Indicator1(field);

package/src/indicator-fixes.test.js

@@ -3,7 +3,7 @@ import {MarcRecord} from '@natlibfi/marc-record';
 import validatorFactory from './indicator-fixes.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
-import createDebugLogger from 'debug';
+//import createDebugLogger from 'debug';
 
 generateTests({
   callback,
@@ -17,7 +17,7 @@ generateTests({
     before: () => testValidatorFactory()
   }
 });
-const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/indicator-fixes:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/indicator-fixes:test');
 
 async function testValidatorFactory() {
   const validator = await validatorFactory();
@@ -27,12 +27,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/isbn-issn.js

@@ -1,5 +1,5 @@
 import ISBN from 'isbn3';
-import validateISSN from '@natlibfi/issn-verify';
+import {issn as validateISSN} from '@natlibfi/issn-verify';
 
 // handleInvalid: move invalid 020$a to 020$z, and invalid 022$a to 022$y
 export default ({hyphenateISBN = false, handleInvalid = false} = {}) => {

package/src/isbn-issn.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/isbn-issn.js';
+import validatorFactory from './isbn-issn.js';
 import {describe, it} from 'node:test';
 
 describe('isbn-issn', () => {
@@ -186,11 +186,13 @@ describe('isbn-issn', () => {
       });
       const result = await validator.validate(record);
 
-      assert.deepEqual(result, {valid: false, messages: [
-        'ISBN (9789519155470) is not valid',
-        'ISBN (9068-31-372-X) is not valid',
-        'ISBN (386006004X) is not valid'
-      ]});
+      assert.deepEqual(result, {
+        valid: false, messages: [
+          'ISBN (9789519155470) is not valid',
+          'ISBN (9068-31-372-X) is not valid',
+          'ISBN (386006004X) is not valid'
+        ]
+      });
     });
 
     it.skip('Finds the record invalid (Missing ISBN)');

package/src/item-language.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert';
 import {MarcRecord} from '@natlibfi/marc-record';
-import validatorFactory from '../src/item-language.js';
+import validatorFactory from './item-language.js';
 import {describe, it} from 'node:test';
 
 describe('item-language', () => {
@@ -17,7 +17,7 @@ describe('item-language', () => {
       assert.equal(err instanceof Error, true);
       assert.equal(err.message, 'No tagPattern provided');
       return true;
-     });
+    });
     //await assert(validatorFactory()).to.be.rejectedWith(Error, 'No tagPattern provided');
   });
 

package/src/merge-fields.test.js

@@ -4,7 +4,7 @@ import validatorFactory from './merge-fields/index.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
 import createDebugLogger from 'debug';
-import { nvdebug } from './utils.js';
+
 
 generateTests({
   callback,
@@ -30,13 +30,8 @@ async function testValidatorFactory() {
   assert.equal(typeof validator.validate, 'function');
 }
 
-async function callback({getFixture, enabled = true, fix = false, tagPattern = false}) {
-  if (enabled === false) {
-    debug('TEST SKIPPED!');
-    return;
-  }
-
-  nvdebug(`TAG PATTERN: ${tagPattern}`);
+async function callback({getFixture, fix = false, tagPattern = false}) {
+  debug(`Run test using tag pattern ${tagPattern}`);
 
   const validator = await validatorFactory(tagPattern);
   const record = new MarcRecord(getFixture('record.json'));

package/src/mergeField500Lisapainokset.test.js

@@ -3,7 +3,7 @@ import {MarcRecord} from '@natlibfi/marc-record';
 import validatorFactory from './mergeField500Lisapainokset.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/mergeField500Lisapainokset:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/mergeField500Lisapainokset: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/mergeRelatorTermFields.test.js

@@ -3,7 +3,7 @@ import {MarcRecord} from '@natlibfi/marc-record';
 import validatorFactory from './mergeRelatorTermFields.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/mergeRelatorTermFields:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/mergeRelatorTermFields: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/multiple-subfield-0.test.js

@@ -3,7 +3,7 @@ import {MarcRecord} from '@natlibfi/marc-record';
 import validatorFactory from './multiple-subfield-0.js';
 import {READERS} from '@natlibfi/fixura';
 import generateTests from '@natlibfi/fixugen';
-import createDebugLogger from 'debug';
+//import createDebugLogger from 'debug';
 
 generateTests({
   callback,
@@ -20,7 +20,7 @@ generateTests({
   }
 });
 
-const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/multiple-subfield-0:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/multiple-subfield-0: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/normalize-dashes.test.js

@@ -3,7 +3,7 @@ import {MarcRecord} from '@natlibfi/marc-record';
 import validatorFactory from './normalize-dashes.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/normalize-dashes:test');
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/normalize-dashes: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');