Skip to content

Commit cb9276e

Browse files
authored
Chore: Improve test coverage in a few places (#167)
1 parent 1da1acc commit cb9276e

File tree

4 files changed

+12
-2
lines changed

4 files changed

+12
-2
lines changed

lib/rules/prefer-object-rule.js

+1
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ module.exports = {
5252
token => token.type === 'Punctuator' && token.value === '('
5353
);
5454

55+
/* istanbul ignore if */
5556
if (!openParenToken) {
5657
// this shouldn't happen, but guarding against crashes just in case
5758
return null;

package.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -33,10 +33,10 @@
3333
"eslint-utils": "^2.1.0"
3434
},
3535
"nyc": {
36-
"branches": 97,
36+
"branches": 98,
3737
"functions": 98,
3838
"lines": 99,
39-
"statements": 98
39+
"statements": 99
4040
},
4141
"devDependencies": {
4242
"@not-an-aardvark/node-release-script": "^0.1.0",

tests/lib/rules/require-meta-docs-description.js

+1
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ const RuleTester = require('eslint').RuleTester;
1414
const ruleTester = new RuleTester({ parserOptions: { ecmaVersion: 6 } });
1515
ruleTester.run('require-meta-docs-description', rule, {
1616
valid: [
17+
'foo()',
1718
`
1819
module.exports = {
1920
meta: { docs: { description: 'disallow unused variables' } },

tests/lib/rules/test-case-property-ordering.js

+8
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,14 @@ const RuleTester = require('eslint').RuleTester;
1919
const ruleTester = new RuleTester();
2020
ruleTester.run('test-case-property-ordering', rule, {
2121
valid: [
22+
`
23+
new RuleTester().run('foo', bar, {
24+
valid: [
25+
'foo',
26+
RuleTester.only('foo'),
27+
]
28+
});
29+
`,
2230
`
2331
new RuleTester().run('foo', bar, {
2432
valid: [

0 commit comments

Comments
 (0)