The initial quick fix was a naive replacement assert parameter expression by a assertThat(<expr>).isTrue() call. Ideally that expression would be expanded to a better assertion if possible, falling back to the less-than-ideal isTrue assertion if necessary. In some situations a follow-up quick fix application will fix subpar assertions, but it would be better to get that right with one fix application. A mechanism to expand binary and unary assertions into assertk assertions could be broadly useful across detectors.
Split from #78
The initial quick fix was a naive replacement
assertparameter expression by aassertThat(<expr>).isTrue()call. Ideally that expression would be expanded to a better assertion if possible, falling back to the less-than-idealisTrueassertion if necessary. In some situations a follow-up quick fix application will fix subpar assertions, but it would be better to get that right with one fix application. A mechanism to expand binary and unary assertions into assertk assertions could be broadly useful across detectors.Split from #78