Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

`no-empty-description` incorrectly flags non-empty descriptions.

See original GitHub issue

The following seem like reasonable test descriptions, but they get flagged by no-empty-description:

describe('eslint-plugin-mocha', function() {
  const description = 'description';
  it(description, function() {});

  ['test1', 'test2'].forEach(testname => {
    it(testname, function() {});
  });

  const removePrefix = 'testcase.prefix';
  it(removePrefix.replace('.prefix'), function() {});
});

   3:3  error  Unexpected empty test description  mocha/no-empty-description
   6:5  error  Unexpected empty test description  mocha/no-empty-description
  10:3  error  Unexpected empty test description  mocha/no-empty-description

Was this intended with #304?

Issue Analytics

State:
Created 2 years ago
Comments:6 (3 by maintainers)

Top GitHub Comments

1reaction

lo1tumacommented, Dec 13, 2021

Thanks for reporting. This is definitely a bug.

0reactions

neverendingqscommented, Dec 14, 2021

@neverendingqs I’ve deployed another fix in version 10.0.3. I hope I’ve covered all edge cases you described. Please let me know if you encounter any more issues with that rule.

Thanks! No more issues as far as I know.