Remove another @AwaitsFix (elastic#473)

costin · Dec 16, 2022 · c99d2e5 · c99d2e5
1 parent 26fbf96
commit c99d2e5
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/.../esql/src/test/java/org/elasticsearch/xpack/esql/optimizer/LogicalPlanOptimizerTests.java b/.../esql/src/test/java/org/elasticsearch/xpack/esql/optimizer/LogicalPlanOptimizerTests.java
@@ -124,15 +124,15 @@ public void testCombineProjectionWhilePreservingAlias() {
         assertThat(Expressions.name(alias.child()), containsString("first_name"));
     }
 
-    @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch-internal/issues/378")
     public void testCombineProjectionWithAggregation() {
         var plan = plan("""
             from test
             | stats avg(salary) by last_name, first_name
             """);
 
-        var agg = as(plan, Aggregate.class);
-        assertThat(Expressions.names(agg.aggregates()), contains("last_name"));
+        var limit = as(plan, Limit.class);
+        var agg = as(limit.child(), Aggregate.class);
+        assertThat(Expressions.names(agg.aggregates()), contains("avg(salary)", "last_name", "first_name"));
         assertThat(Expressions.names(agg.groupings()), contains("last_name", "first_name"));
     }