alex-plekhanov commented on a change in pull request #9774:
URL: https://github.com/apache/ignite/pull/9774#discussion_r795537415
##########
File path:
modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/logical/SqlScriptRunner.java
##########
@@ -659,6 +659,8 @@ private void checkResultTuples(List<List<?>> res) {
/** */
private void checkEquals(String msg, String expectedStr, Object
actual) {
+ expectedStr = F.isEmpty(expectedStr) ? expectedStr :
expectedStr.trim();
Review comment:
Why do we need it in the runner? It's better to fix tests if some tests
are failed because of spaces. Looks like `test_trim.test` failed after this
fix.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
