Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Josh Goldberg <git@joshuakgoldberg.com>
  • Loading branch information
bradzacher and JoshuaKGoldberg committed Oct 27, 2022
1 parent a3f7b95 commit e220691
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions packages/utils/src/eslint-utils/rule-tester/RuleTester.ts
Expand Up @@ -146,15 +146,15 @@ class RuleTester extends BaseRuleTester.RuleTester {
};
(this.staticThis.describe as DescribeWithSkip).skip(name, () => {
this.staticThis.it(
'All test skipped due to unsatisfied constructor dependency constraints',
'All tests skipped due to unsatisfied constructor dependency constraints',
() => {},
);
});
} else {
// otherwise just declare an empty test
this.staticThis.describe(name, () => {
this.staticThis.it(
'All test skipped due to unsatisfied constructor dependency constraints',
'All tests skipped due to unsatisfied constructor dependency constraints',
() => {
// some frameworks error if there are no assertions
assert.equal(true, true);
Expand Down
Expand Up @@ -750,7 +750,7 @@ describe('RuleTester', () => {
mockedDescribe.mock.lastCall?.[1]();
expect(mockedIt.mock.lastCall).toMatchInlineSnapshot(`
[
"All test skipped due to unsatisfied constructor dependency constraints",
"All tests skipped due to unsatisfied constructor dependency constraints",
[Function],
]
`);
Expand Down

0 comments on commit e220691

Please sign in to comment.