Skip to content
Snippets Groups Projects
Commit e80b4f6d authored by Lukas Stadler's avatar Lukas Stadler
Browse files

fix BinaryBooleanNodeTest to properly recognize empty logical vectors

parent 7eed31b9
Branches
No related tags found
No related merge requests found
Hide whitespace changes
Inline Side-by-side
com.oracle.truffle.r.nodes.test/src/com/oracle/truffle/r/nodes/test/BinaryBooleanNodeTest.java
+ 3
− 4
View file @ e80b4f6d
...@@ -211,15 +211,14 @@ public class BinaryBooleanNodeTest extends BinaryVectorTest { ...@@ -211,15 +211,14 @@ public class BinaryBooleanNodeTest extends BinaryVectorTest {
RAbstractVector vector = originalVector.copy(); RAbstractVector vector = originalVector.copy();
assumeFalse(isLogicOp(factory)); assumeFalse(isLogicOp(factory));
assumeFalse(vector.getRType() == RType.Raw); assumeFalse(vector.getRType() == RType.Raw);
RAbstractVector result = createEmptyLogicalVector(); assertThat(executeArithmetic(factory, vector, RNull.instance), isEmptyVectorOf(RType.Logical));
assertThat(executeArithmetic(factory, vector, RNull.instance), is(result)); assertThat(executeArithmetic(factory, RNull.instance, vector), isEmptyVectorOf(RType.Logical));
assertThat(executeArithmetic(factory, RNull.instance, vector), is(result));
} }
@Theory @Theory
public void testBothNull(BooleanOperationFactory factory) { public void testBothNull(BooleanOperationFactory factory) {
assumeFalse(isLogicOp(factory)); assumeFalse(isLogicOp(factory));
assertThat(executeArithmetic(factory, RNull.instance, RNull.instance), is(createEmptyLogicalVector())); assertThat(executeArithmetic(factory, RNull.instance, RNull.instance), isEmptyVectorOf(RType.Logical));
} }
@Theory @Theory
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment