It seems to me that Druid's policy to prohibit static imports makes more
harm than good in unit test code. It discourages the usage of static
fixtures and encourages abstraction/inheritance, as I discussed here:
https://github.com/apache/incubator-druid/pull/8564#issue-319753256, see
"Refactoring of tests" section.

Alternative: allow only static imports from selected classes: Assert,
Assertions, EasyMock, etc. IntelliJ's "Static import" inspection supports
such fine configuration.

Reply via email to