nus-mtp/sashimi-note

Function regexHelper has 83 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function regexHelper(diff, exp, act) {
  const ignoredAttr = ['id', 'marker-end', 'x1', 'x2', 'x', 'y1', 'y2', 'y', 'width',
    'height', 'd', 'dy', 'r', 'style', 'viewBox', 'transform', 'points'];
  const missedArray = [];

File diagramsRenderer.spec.js has 292 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import domCompare from 'dom-compare';
import DiagramsRenderer from 'src/logic/renderer/diagrams';
import documentPackager from 'src/logic/documentPackager';
import diagramsInput from './reference/diagrams/diagramsInput.txt';
import diagramsRenderedOutput from './reference/diagrams/diagramsRenderedOutput.txt';

Avoid deeply nested control flow statements.

Open

        } else if (ignoredAttr.indexOf(arr1[2]) === -1) {
          if (arr1[3] !== arr1[4]) {
            errorArray.push(line);
          }
        }

Avoid deeply nested control flow statements.

Open

      if (!textExpected.has(key) && !textActual.has(key)) {
        // If don't have, use helper function to find the contents in the respective
        // and compare
        if (!helper(key, exp, act)) {
          errorArray.push(line);

Avoid deeply nested control flow statements.

Open

        if (pos !== -1) {
          // Ignore, not an actual error
          missedArray.splice(pos, 1);
        } else if (arr3[1] !== 'element') {
          errorArray.push(line);

Avoid deeply nested control flow statements.

Open

          if (arr1[3].indexOf('raphael-marker-endblock33-obj') !== -1
              && arr1[4].indexOf('raphael-marker-endblock33-obj') !== -1) {
                // Ignore, not an actual error. Just randomly generated id value
          } else if ((arr1[3].indexOf('arrowhead') !== -1
              && arr1[4].indexOf('arrowhead') !== -1)) {

Avoid deeply nested control flow statements.

Open

          if (textExpected.get(key) !== textActual.get(key)) {
            errorArray.push(line);
          }

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

Open

    it('should handle drawing of flow charts', (done) => {
      // Clears Maps
      textActual.clear();
      textExpected.clear();
      // Retrieve HTML string from getHtmlData

• Read up
  Read up

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

Open

    it('should handle drawing of mermaid diagrams', (done) => {
      // Clears Maps
      textActual.clear();
      textExpected.clear();
      // Retrieve HTML string from getHtmlData

• Read up
  Read up

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

Open

  for (let i = 0; i < expArr.length; i+=1) {
    expStr.concat(`${expArr[i].innerText} `);
  }

• Read up
  Read up

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

Open

  for (let i = 0; i < actArr.length; i+=1) {
    actStr.concat(`${actArr[i].innerText} `);
  }

• Read up
  Read up