Skip to content

Commit 9d6bffa

Browse files
committed
Address copilot feedbac
1 parent c743c23 commit 9d6bffa

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

docs/rules/test-case-name-property.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ This rule has one option.
1717
#### `require: 'always' | 'objects' | 'objects-with-config'`
1818

1919
- `always`: all test cases should have a `name` property (this means that no shorthand string test cases are allowed as a side effect)
20-
- `objects`: requires that an `name` property is present in all `object`-based test cases.
20+
- `objects`: requires that a `name` property is present in all `object`-based test cases.
2121
- `objects-with-config` (default): requires that test cases that have `options` or `settings` defined, should also have a `name` property.
2222

2323
Examples of **incorrect** code for this rule:
@@ -60,4 +60,4 @@ const testCase4 = {
6060

6161
## When Not to Use It
6262

63-
If aren't concerned with the nature of the test logs or don't want to require `name` on test cases.
63+
If you aren't concerned with the nature of the test logs or don't want to require `name` on test cases.

lib/rules/test-case-name-property.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ const violationMessages = {
2121
always: 'nameRequiredAlways',
2222
objects: 'nameRequiredObjects',
2323
'objects-with-config': 'nameRequiredObjectsWithConfig',
24-
} as const;
24+
} satisfies Record<Options['require'], string>;
2525

2626
type Options = {
2727
require: 'always' | 'objects' | 'objects-with-config';

0 commit comments

Comments
 (0)