@natlibfi/marc-record-validators-melinda 10.13.0 → 10.13.1-alpha.2

package/src/normalize-qualifying-information.spec.js

@@ -0,0 +1,52 @@
+import {expect} from 'chai';
+import {MarcRecord} from '@natlibfi/marc-record';
+import validatorFactory from './normalize-qualifying-information';
+import {READERS} from '@natlibfi/fixura';
+import generateTests from '@natlibfi/fixugen';
+import createDebugLogger from 'debug';
+
+generateTests({
+  callback,
+  path: [__dirname, '..', 'test-fixtures', 'normalize-qualifying-information'],
+  useMetadataFile: true,
+  recurse: false,
+  fixura: {
+    reader: READERS.JSON
+  },
+  mocha: {
+    before: () => testValidatorFactory()
+  }
+});
+const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/normalize-qualifying-information:test');
+
+async function testValidatorFactory() {
+  const validator = await validatorFactory();
+
+  expect(validator)
+    .to.be.an('object')
+    .that.has.any.keys('description', 'validate');
+
+  expect(validator.description).to.be.a('string');
+  expect(validator.validate).to.be.a('function');
+}
+
+async function callback({getFixture, enabled = true, fix = false}) {
+  if (enabled === false) {
+    debug('TEST SKIPPED!');
+    return;
+  }
+
+  const validator = await validatorFactory();
+  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);
+    expect(result).to.eql(expectedResult);
+    return;
+  }
+
+  await validator.fix(record);
+  expect(record).to.eql(expectedResult);
+}

package/src/normalizeSubfieldValueForComparison.js

@@ -8,6 +8,7 @@ const debug = createDebugLogger('@natlibfi/melinda-marc-record-merge-reducers:no
 const debugDev = debug.extend('dev');
 
 export function subfieldContainsPartData(tag, subfieldCode) {
+  // NB! Used by reducers' mergeSubield.js
   if (subfieldCode === 'v' && ['490', '800', '810', '811', '830'].includes(tag)) {
     return true;
   }
@@ -18,8 +19,9 @@ export function subfieldContainsPartData(tag, subfieldCode) {
 }
 
 function splitPartData(originalValue) {
+  // Remove brackets from number-only:
   const value = originalValue.replace(/^\[([0-9]+)\][-.,:; ]*$/ui, '$1'); // eslint-disable-line prefer-named-capture-group
-  const splitPoint = value.lastIndexOf(' ');
+  const splitPoint = value.lastIndexOf(' '); // MRA-627: "5, 2017" should be split here. Think of this later on...
   if (splitPoint === -1) {
     return [undefined, value];
   }
@@ -33,13 +35,19 @@ function normalizePartType(originalValue) {
     return undefined;
   }
   const value = originalValue.toLowerCase();
-  // Return Finnish singular nominative. Best-ish for debug purposes...
-  if (['osa', 'part', 'teil'].includes(value)) {
+
+  // Return Finnish singular nominative. Choise of language is arbitrary. This is best-ish for debug purposes...
+  if (['n:o', 'no', 'nr', 'nro', 'number', 'numero', 'nummer'].includes(value)) {
+    return 'numero';
+  }
+  if (['band', 'bd', 'häfte', 'nide', 'osa', 'part', 'teil', 'vol', 'vol.', 'volume'].includes(value)) {
     return 'osa';
   }
+
   if (['p.', 'page', 'pages', 'pp.', 's.', 'sidor', 'sivu', 'sivut'].includes(value)) {
     return 'sivu';
   }
+
   return value;
 }
 
@@ -67,12 +75,15 @@ function splitAndNormalizePartData(value) {
 }
 
 export function partsAgree(value1, value2, tag, subfieldCode) {
+  // Note, that parts can not be normalized away, as "2" can agree with "Part 2" and "Raita 2" and "Volume 2"...
+  // NB! Used by reducers' mergeSubield.js
   if (!subfieldContainsPartData(tag, subfieldCode)) {
     return false;
   }
   const [partType1, partNumber1] = splitAndNormalizePartData(value1);
   const [partType2, partNumber2] = splitAndNormalizePartData(value2);
   if (partNumber1 !== partNumber2) {
+    // MRA-627: This should/could accept 5/1997
     return false;
   }
   if (partType1 === undefined || partType2 === undefined || partType1 === partType2) {

package/src/prepublicationUtils.js

@@ -41,8 +41,12 @@ export function fieldRefersToTarkistettuEnnakkotieto(field) {
 
 
 export function fieldRefersToEnnakkotieto(field) {
-  // NB! This matches also 'TARKISTETTU ENNAKKOTIETO' case!
-  return containsSubstringInSubfieldA(field, 'ENNAKKOTIETO');
+  // NB! This no longer matches 'TARKISTETTU ENNAKKOTIETO' case! Bug or Feature?
+  if (containsSubstringInSubfieldA(field, 'ENNAKKOTIETO') && !fieldRefersToTarkistettuEnnakkotieto(field)) {
+    return true;
+  }
+  // MRA-420: "EI VIELÄ ILMESTYNYT" is a Helmet note, that is semantically similar to ENNAKKOTIETO:
+  return containsSubstringInSubfieldA(field, 'EI VIELÄ ILMESTYNYT');
 }
 
 
@@ -68,27 +72,6 @@ export function firstFieldHasBetterPrepubEncodingLevel(field1, field2) {
   return false;
 }
 
-/*
-export function firstFieldHasEqualOrBetterPrepubEncodingLevel(field1, field2) {
-  // Could be optimized...
-  if (fieldRefersToKoneellisestiTuotettuTietue(field1)) {
-    return true;
-  }
-  if (fieldRefersToKoneellisestiTuotettuTietue(field2)) {
-    return false;
-  }
-  if (fieldRefersToTarkistettuEnnakkotieto(field1)) {
-    return true;
-  }
-  if (fieldRefersToTarkistettuEnnakkotieto(field2)) {
-    return false;
-  }
-  if (fieldRefersToEnnakkotieto(field1)) {
-    return true;
-  }
-  return !fieldRefersToEnnakkotieto(field2);
-}
-*/
 
 /*
 function hasEnnakkotietoSubfield(field) {
@@ -117,7 +100,7 @@ export function getRelevant5XXFields(record, f500 = false, f594 = false) {
 
   function hasRelevantPrepubData(field) {
     // Check prepub ($a):
-    if (!fieldRefersToKoneellisestiTuotettuTietue(field) && !fieldRefersToEnnakkotieto(field)) {
+    if (!fieldRefersToKoneellisestiTuotettuTietue(field) && !fieldRefersToTarkistettuEnnakkotieto(field) && !fieldRefersToEnnakkotieto(field)) {
       return false;
     }
     // Check relevance (594$5):
@@ -209,7 +192,7 @@ export function deleteAllPrepublicationNotesFromField500InNonPubRecord(record) {
     return;
   }
 
-  // MET-306: keep "koneellisesti tuotettu tietue" if encodng level is '2':
+  // MET-306: keep "koneellisesti tuotettu tietue" if encoding level is '2':
   const f500 = getRelevant5XXFields(record, true, false).filter(field => encodingLevel === '2' ? !fieldRefersToKoneellisestiTuotettuTietue(field) : true);
   if (f500.length === 0) {
     return;

package/src/punctuation2.js

@@ -117,7 +117,7 @@ const addX10bDot = {'name': 'Add X10 pre-$b dot', 'add': '.', 'code': 'ab', 'fol
 const addX10eComma = {'add': ',', 'code': 'abe', 'followedBy': 'e', 'context': defaultNeedsPuncAfter};
 const addX10Dot = {'name': 'Add X10 final dot', 'add': '.', 'code': 'abe', 'followedBy': '#', 'context': defaultNeedsPuncAfter};
 const addLanguageComma = {'name': 'Add comma before 810$l', 'add': ',', 'code': 'tv', 'followedBy': 'l', 'context': defaultNeedsPuncAfter2};
-const addColonToRelationshipInformation = {'name': 'Add \':\' to 7X0 $i relationship info', 'add': ':', 'code': 'i', 'context': /[a-z)åäö]$/u};
+const addColonToRelationshipInformation = {'name': 'Add \':\' to 7X0 $i relationship info', 'add': ':', 'code': 'i', 'context': defaultNeedsPuncAfter2};
 
 // 490:
 const addSemicolonBeforeVolumeDesignation = {'name': 'Add " ;" before $v', 'add': ' ;', 'code': 'atxyz', 'followedBy': 'v', 'context': /[^;]$/u};
@@ -181,11 +181,12 @@ const cleanCrappyPunctuationRules = {
 const cleanLegalX00Comma = {'code': 'abcde', 'followedBy': 'cdegj', 'context': /.,$/u, 'remove': /,$/u};
 // Accept upper case letters in X00$b, since they are probably Roman numerals.
 const cleanLegalX00bDot = {'code': 'b', 'followedBy': 't#', context: /^[IVXLCDM]+\.$/u, 'remove': /\.$/u};
+const cleanLegalX00iColon = {'code': 'i', 'followedBy': 'a', 'remove': / *:$/u}; // NB! context is not needed
 const cleanLegalX00Dot = {'code': 'abcdetvl', 'followedBy': 'tu#', 'context': /(?:[a-z0-9)]|å|ä|ö)\.$/u, 'remove': /\.$/u};
 const cleanLanguageComma = {'name': 'language comma', 'code': 'tv', 'followedBy': 'l', 'context': /.,$/u, 'remove': /,$/u};
 
 
-const legalX00punc = [cleanLegalX00Comma, cleanLegalX00bDot, cleanLegalX00Dot, cleanLanguageComma];
+const legalX00punc = [cleanLegalX00Comma, cleanLegalX00iColon, cleanLegalX00bDot, cleanLegalX00Dot, cleanLanguageComma];
 
 const cleanLegalX10Comma = {'name': 'X10comma', 'code': 'abe', 'followedBy': 'e', 'context': /.,$/u, 'remove': /,$/u};
 const cleanLegalX10Dot = {'name': 'X10dot', 'code': 'ab', 'followedBy': 'b#', 'context': /.\.$/u, 'remove': /\.$/u};

package/src/removeInferiorDataFields.js

@@ -176,6 +176,57 @@ export function removeInferiorChains(record, fix = true) {
   return deletedStringsArray;
 }
 
+function deriveIndividualDeletables490(fieldAsString) {
+  if (!fieldAsString.match(/^490/u)) {
+    return [];
+  }
+
+  /* eslint-disable */
+  let deletable490s = [];
+
+  // $6-less version (keep this first)
+  let tmp = fieldAsString.replace(/ ‡6 [^‡]+ ‡/u, ' ‡');
+  if ( tmp !== fieldAsString) {
+    fieldAsString = tmp; // NB! Carry on with $6-less version!
+    deletable490s.push(tmp);
+  }
+
+  // Without final $v or $x:
+  tmp = fieldAsString.replace(/ *[;,] ‡[vx] [^‡]+$/u, '');
+  if ( tmp !== fieldAsString) {
+    deletable490s.push(tmp);
+  }
+
+  // Add intermedia $x-less version
+  tmp = fieldAsString.replace(/, ‡x [^‡]+(, ‡x| ; ‡v)/u, '$1');
+  // Add final $v/$x-less version
+  if ( tmp !== fieldAsString) {
+    deletable490s.push(tmp);
+  }
+
+  // Add $xv-less version
+  tmp = fieldAsString.replace(/, ‡x [^‡]+ ‡v [^‡]+$/u, '');
+  if ( tmp !== fieldAsString) {
+    deletable490s.push(tmp);
+  }
+
+  // MRA-433-ish (non-chain): 490 ind1=1 vs ind1=0: remove latter
+  if (fieldAsString.match(/^490 1/) ) {
+    // TODO: $x-less and $v-less versions...
+    tmp = `490 0${fieldAsString.substring(5)}`;
+    deletable490s.push(tmp);
+    const arr = deriveIndividualDeletables490(tmp);
+    arr.forEach(val => deletable490s.push(val));
+  }
+
+  nvdebug(`${deletable490s.length} derivation(s) for ${fieldAsString}`);
+  if (deletable490s.length > 0) {
+    nvdebug(deletable490s.join('\n'));
+  }
+   /* eslint-enable */
+  return deletable490s;
+}
+
 function deriveIndividualDeletables(record) {
   /* eslint-disable */
   let deletableStringsArray = [];
@@ -187,7 +238,6 @@ function deriveIndividualDeletables(record) {
   function fieldDeriveIndividualDeletables(field) {
     const fieldAsString = fieldToString(field);
 
-    nvdebug(`Derivations for ${fieldAsString}`);
     // Proof-of-concept rule:
     let tmp = fieldAsString;
     if (field.tag.match(/^[1678]00$/u)) {
@@ -197,22 +247,38 @@ function deriveIndividualDeletables(record) {
       }
     }
 
+    if (field.tag === '505') { // MRA-413-ish
+      if (fieldAsString.match(/^.0.*-- ‡t/u)) {
+        tmp = fieldAsString;
+        tmp = tmp.replace(/ -- ‡t /gu, ' -- ');
+        tmp = tmp.replace(/ ‡[rg] /gu, ' ');
+        tmp = tmp.replace(/ ‡t /u, ' ‡a '); // first $t, not
+        tmp = tmp.replace(/^505 (.)0/u, '505 $1#');
+        if (tmp !== fieldAsString) {
+          deletableStringsArray.push(tmp);
+        }
+        //nvdebug(`505 ORIGINAL: '${fieldAsString}'`)
+        //nvdebug(`505 DERIVATE: '${tmp}'`)
+      }
+    }
+
     // MET-381: remove occurence number TAG-00, if TAG-NN existists
     if (field.tag === '880') {
       tmp = fieldAsString;
       if (tmp.match(/ ‡6 [0-9][0-9][0-9]-(?:[1-9][0-9]|0[1-9])/)) {
         tmp = tmp.replace(/( ‡6 [0-9][0-9][0-9])-[0-9]+/, '$1-00');
-        nvdebug(`MET-381: ADD TO DELETABLES: ${tmp}`);
+        nvdebug(`MET-381: ADD TO DELETABLES: '${tmp}'`);
         deletableStringsArray.push(tmp);
         if (tmp.match(/ ‡6 [0-9][0-9][0-9]-00\/[^ ]+ /)) {
           tmp = tmp.replace(/( ‡6 [0-9][0-9][0-9]-00)[^ ]+/, '$1');
-          nvdebug(`MET-381: ADD TO DELETABLES: ${tmp}`);
+          nvdebug(`MET-381: ADD TO DELETABLES: '${tmp}'`);
           deletableStringsArray.push(tmp);
         }
       }
     }
 
-
+    const d490 = deriveIndividualDeletables490(fieldAsString);
+    d490.forEach(str => deletableStringsArray.push(str));
 
     // Remove keepless versions:
     tmp = fieldAsString;
@@ -221,13 +287,7 @@ function deriveIndividualDeletables(record) {
       deletableStringsArray.push(tmp);
     }
 
-    // MRA-433-ish (non-chain): 490 ind1=1 vs ind1=0: remove latter
-    if (fieldAsString.match(/^490 1/) ) {
-      tmp = fieldAsString.replace(/^490 1/u, '490 0');
-      deletableStringsArray.push(tmp);
-    }
   }
-
   /* eslint-enable */
   return deletableStringsArray; // we should do uniq!
 

package/src/utils.js

@@ -61,3 +61,15 @@ export function isControlSubfieldCode(subfieldCode) {
   }
   return false;
 }
+
+export function getCatalogingLanguage(record) {
+  const [field040] = record.get(/^040$/u);
+  if (!field040) {
+    return null;
+  }
+  const [b] = field040.subfields.filter(sf => sf.code === 'b');
+  if (!b) {
+    return null;
+  }
+  return b.value;
+}

package/test-fixtures/field-505-separators/01/expectedResult.json

@@ -0,0 +1,7 @@
+{
+  "message": [
+    "'TODO: 505 8# ‡a Foo ; Bar ; Lorum ; Ipsum' => '505 8# ‡a Foo -- Bar -- Lorum -- Ipsum'",
+    "'TODO: 505 80 ‡t Foo ; ‡t Bar ; ‡t Lorum ; ‡t Ipsum' => '505 80 ‡t Foo -- ‡t Bar -- ‡t Lorum -- ‡t Ipsum'"
+  ],
+  "valid": false
+}

package/test-fixtures/field-505-separators/01/metadata.json

@@ -0,0 +1,7 @@
+{
+  "description": "Validate field 505 separators",
+  "comment": "NB! Handles only ' ; ' => ' -- ' conversions",
+  "enabled": true,
+  "fix": false,
+  "only": false
+}

package/test-fixtures/field-505-separators/01/record.json

@@ -0,0 +1,25 @@
+{
+  "fields": [
+    {
+      "tag": "005",
+      "value": "20220202020202.0"
+    },
+    { "tag": "505", "ind1": "8", "ind2": " ", "subfields": [
+        { "code": "a", "value": "Foo ; Bar ; Lorum ; Ipsum" }
+    ]},
+    { "tag": "505", "ind1": "8", "ind2": " ", "subfields": [
+        { "code": "a", "value": "No -- action --required" }
+    ]},
+    { "tag": "505", "ind1": "8", "ind2": "0", "subfields": [
+        { "code": "t", "value": "Foo ;" },
+        { "code": "t", "value": "Bar ;" },
+        { "code": "t", "value": "Lorum ;" },
+        { "code": "t", "value": "Ipsum" }
+    ]},
+    { "tag": "505", "ind1": "8", "ind2": "0", "subfields": [
+      { "code": "t", "value": "Replace --" },
+      { "code": "t", "value": "only" },
+      { "code": "t", "value": "semicolon" }
+    ]}
+  ]
+}

package/test-fixtures/field-505-separators/02/expectedResult.json

@@ -0,0 +1,27 @@
+{
+  "_validationOptions": {},
+  "fields": [
+    {
+      "tag": "005",
+      "value": "20220202020202.0"
+    },
+    { "tag": "505", "ind1": "8", "ind2": " ", "subfields": [
+        { "code": "a", "value": "Foo -- Bar -- Lorum -- Ipsum" }
+    ]},
+    { "tag": "505", "ind1": "8", "ind2": " ", "subfields": [
+        { "code": "a", "value": "No -- action --required" }
+    ]},
+    { "tag": "505", "ind1": "8", "ind2": "0", "subfields": [
+        { "code": "t", "value": "Foo --" },
+        { "code": "t", "value": "Bar --" },
+        { "code": "t", "value": "Lorum --" },
+        { "code": "t", "value": "Ipsum" }
+    ]},
+    { "tag": "505", "ind1": "8", "ind2": "0", "subfields": [
+      { "code": "t", "value": "Replace --" },
+      { "code": "t", "value": "only" },
+      { "code": "t", "value": "semicolon" }
+    ]}
+  ],
+  "leader": ""
+}

package/test-fixtures/field-505-separators/02/metadata.json

@@ -0,0 +1,7 @@
+{
+  "description": "Fix field 505 separators",
+  "comment": "NB! Handles only ' ; ' => ' -- ' conversions",
+  "enabled": true,
+  "fix": true,
+  "only": false
+}

package/test-fixtures/field-505-separators/02/record.json

@@ -0,0 +1,25 @@
+{
+  "fields": [
+    {
+      "tag": "005",
+      "value": "20220202020202.0"
+    },
+    { "tag": "505", "ind1": "8", "ind2": " ", "subfields": [
+        { "code": "a", "value": "Foo ; Bar ; Lorum ; Ipsum" }
+    ]},
+    { "tag": "505", "ind1": "8", "ind2": " ", "subfields": [
+        { "code": "a", "value": "No -- action --required" }
+    ]},
+    { "tag": "505", "ind1": "8", "ind2": "0", "subfields": [
+        { "code": "t", "value": "Foo ;" },
+        { "code": "t", "value": "Bar ;" },
+        { "code": "t", "value": "Lorum ;" },
+        { "code": "t", "value": "Ipsum" }
+    ]},
+    { "tag": "505", "ind1": "8", "ind2": "0", "subfields": [
+      { "code": "t", "value": "Replace --" },
+      { "code": "t", "value": "only" },
+      { "code": "t", "value": "semicolon" }
+    ]}
+  ]
+}

package/test-fixtures/fix-relator-terms/f01/expectedResult.json

@@ -0,0 +1,14 @@
+{
+  "_validationOptions": {},
+  "leader": "01331cam a22003494i 4500",
+  "fields": [
+    { "tag": "040", "ind1": " ", "ind2": " ", "subfields": [ { "code": "b", "value": "fin" } ] },
+    { "tag": "700", "ind1": " ", "ind2": " ",
+      "subfields": [
+          { "code": "a", "value": "Nimi," },
+          { "code": "e", "value": "säveltäjä,"},
+          { "code": "e", "value": "esittäjä,"},
+          { "code": "e", "value": "sanoittaja." }
+      ] }
+  ]
+}

package/test-fixtures/fix-relator-terms/f01/metadata.json

@@ -0,0 +1,6 @@
+{
+  "description": "F01: Fix relator terms: expand abbreviations based on 040$b",
+  "enabled": true,
+  "fix": true,
+  "only": false
+}

package/test-fixtures/fix-relator-terms/f01/record.json

@@ -0,0 +1,13 @@
+{
+  "leader": "01331cam a22003494i 4500",
+  "fields": [
+    { "tag": "040", "ind1": " ", "ind2": " ", "subfields": [ { "code": "b", "value": "fin" } ] },
+    { "tag": "700", "ind1": " ", "ind2": " ",
+      "subfields": [
+          { "code": "a", "value": "Nimi," },
+          { "code": "e", "value": "säv.,"},
+          { "code": "e", "value": "esitt.,"},
+          { "code": "e", "value": "san." }
+      ] }
+  ]
+}

package/test-fixtures/fix-relator-terms/f01b/expectedResult.json

@@ -0,0 +1,12 @@
+{
+  "_validationOptions": {},
+  "leader": "01331cam a22003494i 4500",
+  "fields": [
+    { "tag": "700", "ind1": " ", "ind2": " ",
+      "subfields": [
+          { "code": "a", "value": "Nimi," },
+          { "code": "e", "value": "esittäjä,"},
+          { "code": "e", "value": "sanoittaja." }
+      ] }
+  ]
+}

package/test-fixtures/fix-relator-terms/f01b/metadata.json

@@ -0,0 +1,6 @@
+{
+  "description": "F01b: Fix relator terms: expand abbreviations (no 040$b)",
+  "enabled": true,
+  "fix": true,
+  "only": false
+}

package/test-fixtures/fix-relator-terms/f01b/record.json

@@ -0,0 +1,11 @@
+{
+  "leader": "01331cam a22003494i 4500",
+  "fields": [
+    { "tag": "700", "ind1": " ", "ind2": " ",
+      "subfields": [
+          { "code": "a", "value": "Nimi," },
+          { "code": "e", "value": "esitt.,"},
+          { "code": "e", "value": "san." }
+      ] }
+  ]
+}

package/test-fixtures/fix-relator-terms/f02/expectedResult.json

@@ -0,0 +1,12 @@
+{
+  "_validationOptions": {},
+  "leader": "01331cam a22003494i 4500",
+  "fields": [
+    { "tag": "040", "ind1": " ", "ind2": " ", "subfields": [ { "code": "b", "value": "fin" } ] },
+    { "tag": "700", "ind1": " ", "ind2": " ",
+      "subfields": [
+          { "code": "a", "value": "Nimi," },
+          { "code": "e", "value": "kirjoittaja."}
+      ] }
+  ]
+}

package/test-fixtures/fix-relator-terms/f02/metadata.json

@@ -0,0 +1,6 @@
+{
+  "description": "F02: Fix relator terms: expand abbreviations",
+  "enabled": true,
+  "fix": true,
+  "only": false
+}

package/test-fixtures/fix-relator-terms/f02/record.json

@@ -0,0 +1,11 @@
+{
+  "leader": "01331cam a22003494i 4500",
+  "fields": [
+    { "tag": "040", "ind1": " ", "ind2": " ", "subfields": [ { "code": "b", "value": "fin" } ] },
+    { "tag": "700", "ind1": " ", "ind2": " ",
+      "subfields": [
+          { "code": "a", "value": "Nimi," },
+          { "code": "e", "value": "författare."}
+      ] }
+  ]
+}

package/test-fixtures/normalize-qualifying-information/01/expectedResult.json

@@ -0,0 +1,8 @@
+{
+  "message": [
+    "'TODO: 020 ## ‡a fv67890 ‡q häftad' => '020 ## ‡a fv67890 ‡q mjuka pärmar'",
+    "'TODO: 020 ## ‡a 1234567890 ‡q SID.' => '020 ## ‡a 1234567890 ‡q kovakantinen'",
+    "'TODO: 024 8# ‡a MEL123 ‡q Nidottu' => '024 8# ‡a MEL123 ‡q pehmeäkantinen'"
+  ],
+  "valid": false
+}

package/test-fixtures/normalize-qualifying-information/01/metadata.json

@@ -0,0 +1,7 @@
+{
+  "description": "Validate qualifying information terms",
+  "comment": "MELINDA-8740",
+  "enabled": true,
+  "fix": false,
+  "only": false
+}

package/test-fixtures/normalize-qualifying-information/01/record.json

@@ -0,0 +1,25 @@
+{
+  "fields": [
+    {
+      "tag": "005",
+      "value": "20220202020202.0"
+    },
+    { "tag": "020", "ind1": " ", "ind2": " ", "subfields": [
+      { "code": "a", "value": "fv67890" },
+      { "code": "q", "value": "häftad" }
+    ]},
+    { "tag": "020", "ind1": " ", "ind2": " ", "subfields": [
+        { "code": "a", "value": "1234567890" },
+        { "code": "q", "value": "SID." }
+    ]},
+    { "tag": "024", "ind1": "8", "ind2": " ", "subfields": [
+      { "code": "a", "value": "MEL123" },
+      { "code": "q", "value": "Nidottu" }
+    ]},
+    { "tag": "028", "ind1": "0", "ind2": " ", "subfields": [
+      { "code": "a", "value": "TätäEnVieläOsaa" },
+      { "code": "q", "value": "(sid.)" }
+    ]}
+
+  ]
+}

package/test-fixtures/normalize-qualifying-information/02/expectedResult.json

@@ -0,0 +1,27 @@
+{
+  "_validationOptions": {},
+  "fields": [
+    {
+      "tag": "005",
+      "value": "20220202020202.0"
+    },
+    { "tag": "020", "ind1": " ", "ind2": " ", "subfields": [
+      { "code": "a", "value": "fv67890" },
+      { "code": "q", "value": "mjuka pärmar" }
+    ]},
+    { "tag": "020", "ind1": " ", "ind2": " ", "subfields": [
+        { "code": "a", "value": "1234567890" },
+        { "code": "q", "value": "kovakantinen" }
+    ]},
+    { "tag": "024", "ind1": "8", "ind2": " ", "subfields": [
+      { "code": "a", "value": "MEL123" },
+      { "code": "q", "value": "pehmeäkantinen" }
+    ]},
+    { "tag": "028", "ind1": "0", "ind2": " ", "subfields": [
+      { "code": "a", "value": "TätäEnVieläOsaa" },
+      { "code": "q", "value": "(sid.)" }
+    ]}
+
+  ],
+  "leader": ""
+}

package/test-fixtures/normalize-qualifying-information/02/metadata.json

@@ -0,0 +1,7 @@
+{
+  "description": "Fix qualifying information terms",
+  "comment": "MELINDA-8740",
+  "enabled": true,
+  "fix": true,
+  "only": false
+}