Warn about accessing private members in documented `unittest`s

[Bolpat]

Warn about accessing private members in documented `unittest`s

[Geod24]

Actually, I'll add one condition: If the documented unittest is for a private symbol, no warning should be emitted.

[Bolpat]

If you like this, it can be extended to protected and/or package symbols. I’d strongly suggest to include protected with Protected. as the documentation keyword to mark this as intentional. Package is probably harder because package visibility can be qualified, and while it’s pretty clear what Private and Protected as part of the unit test documentation means, Package isn’t as clear.

[Geod24]

I would not extend it to protected. Remember we want to minimize the amount of boilerplate here. protected is, to some extent, closer to public than to private (if the class is public). Let's start with private and see how well it works first.