Fix bug: Report errors on extends expression in JS even if an @augments tag is present

[ghost]

Sequel to #18775
Turns out getBaseConstructorTypeOfClass was the only place we checked the extends expression, so that PR turned off checking if an @augments tag was present.
It would seem neater to have a checkExpression(baseTypeNode) call in checkClassLikeDeclaration, but that would mean we check it twice in the usual case.

[sandersn]

Looks good, but I'm curious — in JS, which wins, @augments or extends? Or do we issue an error if they differ? I'd say we should issue an error and then let extends win.

[ghost]

The PR for @extends is still open at #18706

[sandersn]

No, what happens for

/** @augments A */
class B extends C { }

shouldn't we issue an error saying that you can't inherit from both A and C ?

[ghost]

See the test jsdocAugments_nameMismatch, added in #18775.

[sandersn]

Thanks 👍