Skip to content

Commit

Permalink
[MINOR] Fix edge case tests to reflect new changes
Browse files Browse the repository at this point in the history
  • Loading branch information
Baunsgaard committed Jan 30, 2024
1 parent 64174bd commit 2cd782f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ public void castToFrame() {
double maxp1 = Integer.MAX_VALUE + 1.0;
MatrixBlock mb = TestUtils.generateTestMatrixBlock(100, 100, maxp1, maxp1, 1.0, 23);
FrameBlock f = DataConverter.convertToFrameBlock(mb);
assertTrue(f.getSchema()[0] == ValueType.INT64);
assertTrue(f.getSchema()[0] == ValueType.FP64);
}

@Test
Expand All @@ -50,8 +50,7 @@ public void castToFrame3() {
public void castErrorValue() {
MatrixBlock mb = new MatrixBlock(10, 10, Double.parseDouble("2.572306572E9"));
FrameBlock f = DataConverter.convertToFrameBlock(mb);
assertTrue(f.getSchema()[0] == ValueType.INT64);

assertTrue(f.getSchema()[0] == ValueType.FP64);
}

@Test
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ private void runTransformEncodeDecodeTest(ExecType et, boolean sparse, String fm
SCRIPT_DIR + TEST_DIR + SPEC, output("FO")};

// run test
LOG.error(runTest(null));
runTest(null);

// compare matrices (values recoded to identical codes)
FrameReader reader = FrameReaderFactory.createFrameReader(FileFormat.safeValueOf(fmt));
Expand Down

0 comments on commit 2cd782f

Please sign in to comment.