eslint - Nice way to get rid of no-unused-expressions linter error with chai

Question

In my Chai tests I often find myself wanting to use their assertions that are something like .to.be.empty, .to.be.true e.t.c., because I find them to be cleaner to read than .to.be.length(1) or .to.be.equal(true). However, this breaks my linter (I'm using default Airbnb linting).

I could use the // disable-eslint-line syntax, but then I'd have to add it to every single line that reads like that and that seems tedious.

I've also read about the DirtyChai library, but that would require me to go back through my entire testing library adding brackets to them all which seems like something I shouldn't have to do simply to get my linter to pass something it should probably be OK with in the first place.

Does anyone know a nicer way to handle this than the ways I've outlined above?

Answer 1

You can disable the rule for the entire file using eslint-disable at the top of the file in question:

/* eslint-disable no-unused-expressions */
expect(someTrueValue).to.be.true; 

However, adding this at the top of every test file can be tedious. To disable this rule for all relevant files, you can:

1. Put a new .eslintc configuration file in the same directory as your test files, configured to disable that rule. This allows you to use the default configuration for all other rules while ignoring that rule specifically only on files in that folder. ESLint calls this Configuration Cascading.

   {
       "rules": {
           "no-unused-expressions": "off"
       }
   }
   

2. Use the overrides key in your main .eslintrc file to disable rules for groups of files with glob pattern matching:

   {
       "overrides": [
           {
               "files": ["*.test.js", "*.spec.js"],
               "rules": {
                   "no-unused-expressions": "off"
               }
           }
       ]
   }
   

This also allows you to disable other rules which become troublesome in testing, such as no-underscore-dangle when using rewire.