Skip to content

Commit

Permalink
MONDRIAN: updated FilterTest so that it would compile and pass the tests
Browse files Browse the repository at this point in the history
[git-p4: depot-paths = "//open/mondrian-release/3.1/": change = 13297]
  • Loading branch information
Will Gorman committed Jan 7, 2010
1 parent f5d15de commit 5dd01eb
Showing 1 changed file with 17 additions and 8 deletions.
25 changes: 17 additions & 8 deletions testsrc/main/mondrian/rolap/FilterTest.java
Expand Up @@ -271,7 +271,7 @@ public void testNotInFilterKeepNullMember() throws Exception {
"Row #8: 84\n" +
"Row #9: 278\n";

checkNative(0, 10, query, fold(result), requestFreshConnection);
checkNative(0, 10, query, result, requestFreshConnection);
}

/**
Expand Down Expand Up @@ -325,14 +325,19 @@ public void testNotInFilterExcludeNullMember() throws Exception {
"Row #7: 84\n" +
"Row #8: 278\n";

checkNative(0, 9, query, fold(result), requestFreshConnection);
checkNative(0, 9, query, result, requestFreshConnection);
}

/**
* Test that null members are included when the filter excludes members
* that contain multiple levels, but none being null.
*/
public void testNotInMultiLevelMemberConstraintNonNullParent() {

if (MondrianProperties.instance().UseAggregates.get()) {
return;
}

String query =
"With " +
"Set [*NATIVE_CJ_SET] as 'NonEmptyCrossJoin([*BASE_MEMBERS_Customers],[*BASE_MEMBERS_Quarters])' " +
Expand Down Expand Up @@ -395,6 +400,10 @@ public void testNotInMultiLevelMemberConstraintNonNullParent() {
* the same parent.
*/
public void testNotInMultiLevelMemberConstraintNonNullSameParent() {
if (MondrianProperties.instance().UseAggregates.get()) {
return;
}

String query =
"With " +
"Set [*NATIVE_CJ_SET] as 'NonEmptyCrossJoin([*BASE_MEMBERS_Customers],[*BASE_MEMBERS_Quarters])' " +
Expand Down Expand Up @@ -481,7 +490,7 @@ public void testNotInMultiLevelMemberConstraintMixedNullNonNullParent() {
"{[Warehouse2].[#null].[234 West Covina Pkwy].[Freeman And Co]," +
" [Warehouse2].[971-555-6213].[3377 Coachman Place].[Jones International]})' " +
"set [NECJ] as NonEmptyCrossJoin([Filtered Warehouse Set], {[Product].[Product Family].Food}) " +
"select [NECJ] on rows from [Warehouse2]";
"select [NECJ] on 0 from [Warehouse2]";

String necjSqlDerby =
"select \"warehouse\".\"warehouse_fax\", \"warehouse\".\"wa_address1\", " +
Expand Down Expand Up @@ -588,7 +597,7 @@ public void testNotInMultiLevelMemberConstraintSingleNullParent() {
"set [Filtered Warehouse Set] as 'Filter([Warehouse2].[name].Members, [Warehouse2].CurrentMember Not In" +
"{[Warehouse2].[#null].[234 West Covina Pkwy].[Freeman And Co]})' " +
"set [NECJ] as NonEmptyCrossJoin([Filtered Warehouse Set], {[Product].[Product Family].Food}) " +
"select [NECJ] on rows from [Warehouse2]";
"select [NECJ] on 0 from [Warehouse2]";

String necjSqlDerby =
"select \"warehouse\".\"warehouse_fax\", \"warehouse\".\"wa_address1\", " +
Expand Down Expand Up @@ -781,7 +790,7 @@ public void testCmNativeFilter() {
// Get a fresh connection; Otherwise the mondrian property setting
// is not refreshed for this parameter.
boolean requestFreshConnection = true;
checkNative(0, 8, query, fold(result), requestFreshConnection);
checkNative(0, 8, query, result, requestFreshConnection);

MondrianProperties.instance().EnableNativeFilter.set(origNativeFilter);
}
Expand Down Expand Up @@ -833,7 +842,7 @@ public void testNonNativeFilterWithNullMeasure() {
MondrianProperties.instance().EnableNativeFilter.get();
MondrianProperties.instance().EnableNativeFilter.set(false);

checkNotNative(9, query, fold(result));
checkNotNative(9, query, result);

MondrianProperties.instance().EnableNativeFilter.set(origNativeFilter);
}
Expand Down Expand Up @@ -875,7 +884,7 @@ public void testNativeFilterWithNullMeasure() {
// is not refreshed for this parameter.
Connection conn = getTestContext().getFoodMartConnection(false);
TestContext context = getTestContext(conn);
context.assertQueryReturns(query, fold(result));
context.assertQueryReturns(query, result);

MondrianProperties.instance().EnableNativeFilter.set(origNativeFilter);
}
Expand Down Expand Up @@ -909,7 +918,7 @@ public void testNonNativeFilterWithCalcMember() {
MondrianProperties.instance().EnableNativeFilter.get();
MondrianProperties.instance().EnableNativeFilter.set(false);

checkNotNative(3, query, fold(result));
checkNotNative(3, query, result);
MondrianProperties.instance().EnableNativeFilter.set(origNativeFilter);
}

Expand Down

0 comments on commit 5dd01eb

Please sign in to comment.