NatLibFi/marc-record-js

Function doTest has 203 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  function doTest(metadata) {

    // Get input & expected output
    const {getFixture} = metadata;
    const {input, result, immutable, noinput, validationOptions} = metadata;

Function runOperation has 162 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function runOperation(op) {
      const {name, args} = op;

      //-------------------------------------------------------------------------
      if (name === 'nop') {

Avoid too many return statements within this function.

Open

        return record.get(what);

Avoid too many return statements within this function.

Open

        return cloned;

Avoid too many return statements within this function.

Open

        return record;

Avoid too many return statements within this function.

Open

        return record.toObject();

Avoid too many return statements within this function.

Open

        return record.getTypeOfMaterial();

Avoid too many return statements within this function.

Open

        return record;

Avoid too many return statements within this function.

Open

          return new MarcRecord({leader, fields}, validationOptions);

Avoid too many return statements within this function.

Open

        return record.getControlfields();

Avoid too many return statements within this function.

Open

          return record.isCF();

Avoid too many return statements within this function.

Open

        return created;

Avoid too many return statements within this function.

Open

        return fields;

Avoid too many return statements within this function.

Open

        return record;

Avoid too many return statements within this function.

Open

        return record;

Avoid too many return statements within this function.

Open

        return MarcRecord.getValidationOptions();

Avoid too many return statements within this function.

Open

        return result;

Avoid too many return statements within this function.

Open

          return record.isCR();

Avoid too many return statements within this function.

Open

        return false;

Avoid too many return statements within this function.

Open

          return record.isMU();

Avoid too many return statements within this function.

Open

          return record.isMX();

Avoid too many return statements within this function.

Open

          return record.isVM();

Avoid too many return statements within this function.

Open

          return record.pop(what); // eslint-disable-line functional/immutable-data

Avoid too many return statements within this function.

Open

        return MarcRecord.setValidationOptions(args);

Avoid too many return statements within this function.

Open

          return record.isBK();

Avoid too many return statements within this function.

Open

          return record.isMP();

Avoid too many return statements within this function.

Open

        return record.getDatafields();

Avoid too many return statements within this function.

Open

        return record.toString().split('\n');

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  generateTests({
    callback: doTest,
    path: [__dirname, '..', 'test-fixtures', 'index'],
    useMetadataFile: true,
    recurse: true,

• Read up
  Read up