Skip to content

Commit b7bebd1

Browse files
authored
Merge pull request #329 from a-nozeret/fix/327-tag-names-autofix-mixup
fix: tag name autofix confused by adjacent tags (fixes #327)
2 parents f0c246c + 0a47c8f commit b7bebd1

File tree

2 files changed

+27
-1
lines changed

2 files changed

+27
-1
lines changed

src/rules/checkTagNames.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ export default iterateJsdoc(({
5757

5858
if (preferredTagName !== tagName) {
5959
report(message, (fixer) => {
60-
const replacement = sourceCode.getText(jsdocNode).replace(`@${tagName}`, `@${preferredTagName}`);
60+
const replacement = sourceCode.getText(jsdocNode).replace(`@${tagName} `, `@${preferredTagName} `);
6161

6262
return fixer.replaceText(jsdocNode, replacement);
6363
}, jsdocTag);

test/rules/assertions/checkTagNames.js

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -309,6 +309,32 @@ export default {
309309
}
310310
}
311311
}
312+
},
313+
{
314+
code: `
315+
/**
316+
* @property {object} a
317+
* @prop {boolean} b
318+
*/
319+
function quux () {
320+
321+
}
322+
`,
323+
errors: [
324+
{
325+
line: 4,
326+
message: 'Invalid JSDoc tag (preference). Replace "prop" JSDoc tag with "property".'
327+
}
328+
],
329+
output: `
330+
/**
331+
* @property {object} a
332+
* @property {boolean} b
333+
*/
334+
function quux () {
335+
336+
}
337+
`
312338
}
313339
],
314340
valid: [

0 commit comments

Comments
 (0)