I’m a frequent user of the [ExpectedExceptionAttribute] but found out lately that its use is not recommended (anymore).
While the class itself is not deprecated, there are good reasons to use the Assert.ThrowsException() or Assert.ThrowsExceptionAsync() when writing unit tests.
A full explanation can be found here: Finally: Proper Exception Assertion in MSTest | Jim’s Rules Regarding Everything
I believe a SonarCloud rule would make sense here - it can be low-level (reported as a Minor)
Please let me know what you think.