This is an automated email from the ASF dual-hosted git repository.

amashenkov pushed a commit to branch ignite-26090
in repository https://gitbox.apache.org/repos/asf/ignite-3.git


The following commit(s) were added to refs/heads/ignite-26090 by this push:
     new 06966070484 minor
06966070484 is described below

commit 06966070484c10eedf7147c183af8c6c41a42e09
Author: amashenkov <amashen...@apache.org>
AuthorDate: Tue Sep 23 11:46:32 2025 +0300

    minor
---
 .../internal/sql/engine/ItFloatingPointTest.java   | 52 +++++++++-------------
 1 file changed, 21 insertions(+), 31 deletions(-)

diff --git 
a/modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItFloatingPointTest.java
 
b/modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItFloatingPointTest.java
index 064c0d5cb13..d108bceaf3b 100644
--- 
a/modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItFloatingPointTest.java
+++ 
b/modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItFloatingPointTest.java
@@ -17,14 +17,13 @@
 
 package org.apache.ignite.internal.sql.engine;
 
+import static java.util.stream.Collectors.toList;
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.contains;
-import static org.hamcrest.Matchers.equalTo;
+import static org.hamcrest.Matchers.containsInAnyOrder;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import java.util.List;
-import java.util.stream.Collectors;
-import org.hamcrest.Matchers;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
@@ -123,34 +122,14 @@ public class ItFloatingPointTest extends 
BaseSqlMultiStatementTest {
     }
 
     @Test
-    void testOrderBySpecialValues() {
-        {
-            List<Float> values = sql("SELECT f FROM test ORDER BY f")
-                    
.stream().flatMap(List::stream).map(Float.class::cast).collect(Collectors.toList());
-
-            assertThat(values, Matchers.contains(
-                    equalTo(Float.NEGATIVE_INFINITY),
-                    equalTo(-1.0f),
-                    equalTo(-0.0f),
-                    equalTo(0.0f),
-                    equalTo(1.0f),
-                    equalTo(Float.POSITIVE_INFINITY),
-                    equalTo(Float.NaN) // NaN first or last depending on DB
-            ));
-        }
-        {
-            List<Double> values = sql("SELECT d FROM test ORDER BY d")
-                    
.stream().flatMap(List::stream).map(Double.class::cast).collect(Collectors.toList());
-            assertThat(values, Matchers.contains(
-                    equalTo(Double.NEGATIVE_INFINITY),
-                    equalTo(-1.0d),
-                    equalTo(-0.0d),
-                    equalTo(0.0d),
-                    equalTo(1.0d),
-                    equalTo(Double.POSITIVE_INFINITY),
-                    equalTo(Double.NaN) // NaN first or last depending on DB
-            ));
-        }
+    void testOrderBy() {
+        List<Float> floats = sql("SELECT f FROM test ORDER BY f")
+                
.stream().flatMap(List::stream).map(Float.class::cast).collect(toList());
+        List<Double> doubles = sql("SELECT d FROM test ORDER BY d")
+                
.stream().flatMap(List::stream).map(Double.class::cast).collect(toList());
+
+        assertThat(floats, contains(Float.NEGATIVE_INFINITY, -1.0f, -0.0f, 
0.0f, 1.0f, Float.POSITIVE_INFINITY, Float.NaN));
+        assertThat(doubles, contains(Double.NEGATIVE_INFINITY, -1.0d, -0.0d, 
0.0d, 1.0d, Double.POSITIVE_INFINITY, Double.NaN));
     }
 
     @Test
@@ -187,4 +166,15 @@ public class ItFloatingPointTest extends 
BaseSqlMultiStatementTest {
         assertThat(maxRows.get(0), contains(Float.NaN, Double.NaN));
         assertThat(avgRows.get(0), contains(Double.NaN, Double.NaN));
     }
+
+    @Test
+    void testGrouping() {
+        List<Float> floats = sql("SELECT f FROM test GROUP BY f")
+                
.stream().flatMap(List::stream).map(Float.class::cast).collect(toList());
+        List<Double> doubles = sql("SELECT d FROM test GROUP BY d")
+                
.stream().flatMap(List::stream).map(Double.class::cast).collect(toList());
+
+        assertThat(floats, containsInAnyOrder(Float.NEGATIVE_INFINITY, 
Float.POSITIVE_INFINITY, Float.NaN, -1.0f, 1.0f, 0.0f, -0.0f));
+        assertThat(doubles, containsInAnyOrder(Double.NEGATIVE_INFINITY, 
Double.POSITIVE_INFINITY, Double.NaN, -1.0d, 1.0d, 0.0d, -0.0d));
+    }
 }

Reply via email to