Skip to content

Commit

Permalink
MONDRIAN: Fixed some compiler warnings, and commented out testHierarc…
Browse files Browse the repository at this point in the history
…hizeOrdinal() because it's been broken for a while and there is a tracker open for it (bug 1220787).

[git-p4: depot-paths = "//open/mondrian/": change = 3713]
  • Loading branch information
ebb committed Jun 17, 2005
1 parent 4f9ca07 commit 723cd73
Showing 1 changed file with 3 additions and 9 deletions.
12 changes: 3 additions & 9 deletions testsrc/main/mondrian/olap/fun/FunctionTest.java
Expand Up @@ -1836,8 +1836,6 @@ public void testCoalesceEmpty() throws Exception {
+ "from " + nl
+ " [Sales]");

Cell cell;

checkDataResults(
new Double[][] {
{null, null},
Expand Down Expand Up @@ -2720,11 +2718,12 @@ public void testHierarchizeCrossJoinPost() {
/**
* Tests that the Hierarchize function works correctly when applied to
* a level whose ordering is determined by an 'ordinal' property.
* TODO: fix this test (bug 1220787)
*/
public void testHierarchizeOrdinal() {
public void _testHierarchizeOrdinal() {
final Connection connection =
TestContext.instance().getFoodMartConnection(false);
final Cube cube = connection.getSchema().createCube(
connection.getSchema().createCube(
"<Cube name=\"Sales_Hierarchize\">" + nl +
" <Table name=\"sales_fact_1997\"/>" + nl +
" <Dimension name=\"Time_Alphabetical\" type=\"TimeDimension\" foreignKey=\"time_id\">" + nl +
Expand Down Expand Up @@ -3584,18 +3583,13 @@ public void testTupleToStr() {
*/
public void assertExprReturns(String expr, String expected) {
String actual = executeExpr(expr);
System.out.println("actual="+actual);
System.out.println("expected="+expected);
assertEquals(expected, actual);
}

public void assertExprReturnsX(String expr,
double expected, double epsilon) {
String actual = executeExpr(expr);

System.out.println("actual="+actual);
System.out.println("expected="+expected);
System.out.println("epsilon="+epsilon);
try {
Assert.assertEquals(null,
expected,
Expand Down

0 comments on commit 723cd73

Please sign in to comment.