Fixed testcases that seems to have always been wrong... Should be double checked by someone else.
diff --git a/core/bootstrap/src/main/java/org/apache/zest/bootstrap/layered/LayeredLayerAssembler.java b/core/bootstrap/src/main/java/org/apache/zest/bootstrap/layered/LayeredLayerAssembler.java index ccec8d2..46a3a83 100644 --- a/core/bootstrap/src/main/java/org/apache/zest/bootstrap/layered/LayeredLayerAssembler.java +++ b/core/bootstrap/src/main/java/org/apache/zest/bootstrap/layered/LayeredLayerAssembler.java
@@ -28,15 +28,15 @@ { private HashMap<Class<? extends ModuleAssembler>, ModuleAssembler> assemblers = new HashMap<>(); - protected ModuleAssembly createModule( LayerAssembly layer, Class<? extends ModuleAssembler> modulerAssemblerClass ) + protected ModuleAssembly createModule( LayerAssembly layer, Class<? extends ModuleAssembler> moduleAssemblerClass ) { try { - ModuleAssembler moduleAssembler = instantiateAssembler( layer, modulerAssemblerClass ); - String moduleName = createModuleName( modulerAssemblerClass ); - LayeredApplicationAssembler.setNameIfPresent( modulerAssemblerClass, moduleName ); + ModuleAssembler moduleAssembler = instantiateAssembler( layer, moduleAssemblerClass ); + String moduleName = createModuleName( moduleAssemblerClass ); + LayeredApplicationAssembler.setNameIfPresent( moduleAssemblerClass, moduleName ); ModuleAssembly module = layer.module( moduleName ); - assemblers.put( modulerAssemblerClass, moduleAssembler ); + assemblers.put( moduleAssemblerClass, moduleAssembler ); ModuleAssembly assembly = moduleAssembler.assemble( layer, module ); if( assembly == null ) { @@ -46,7 +46,7 @@ } catch( Exception e ) { - throw new IllegalArgumentException( "Unable to instantiate module with " + modulerAssemblerClass.getSimpleName(), e ); + throw new IllegalArgumentException( "Unable to instantiate module with " + moduleAssemblerClass.getSimpleName(), e ); } }
diff --git a/core/testsupport/src/main/java/org/apache/zest/test/indexing/AbstractNamedQueryTest.java b/core/testsupport/src/main/java/org/apache/zest/test/indexing/AbstractNamedQueryTest.java index cac1c7e..ecfbc2a 100755 --- a/core/testsupport/src/main/java/org/apache/zest/test/indexing/AbstractNamedQueryTest.java +++ b/core/testsupport/src/main/java/org/apache/zest/test/indexing/AbstractNamedQueryTest.java
@@ -112,8 +112,8 @@ .newQueryBuilder( Nameable.class ) .where( queries.get( "script03" ) ) ); System.out.println( "*** script03: " + query ); - verifyUnorderedResults( query, "Joe Doe", "Ann Doe", "Jack Doe", "Penang", "Kuala Lumpur", "Cooking", "Gaming", - "Programming", "Cars" ); + verifyUnorderedResults( query, "Joe Doe", "Felix", "Ann Doe", "Jack Doe", "Penang", "Kuala Lumpur", "Cooking", + "Gaming", "Programming", "Cars" ); } @Test @@ -267,14 +267,15 @@ throws EntityFinderException { Nameable nameable = templateFor( Nameable.class ); + Predicate<Composite> predicate = queries.get( "script17" ); final Query<Nameable> query = unitOfWork.newQuery( this.moduleInstance .newQueryBuilder( Nameable.class ) - .where( queries.get( "script17" ) ) ); + .where( predicate ) ); query.orderBy( orderBy( nameable.name() ) ); query.firstResult( 3 ); query.maxResults( 3 ); System.out.println( "*** script17: " + query ); - verifyOrderedResults( query, "Gaming", "Jack Doe", "Joe Doe" ); + verifyOrderedResults( query, "Felix", "Gaming", "Jack Doe" ); } @Test @@ -282,13 +283,14 @@ throws EntityFinderException { Nameable nameable = templateFor( Nameable.class ); + Predicate<Composite> predicate = queries.get( "script18" ); final Query<Nameable> query = unitOfWork.newQuery( this.moduleInstance .newQueryBuilder( Nameable.class ) - .where( queries.get( "script18" ) ) ); + .where( predicate ) ); query.orderBy( orderBy( nameable.name() ) ); System.out.println( "*** script18: " + query ); - verifyOrderedResults( query, "Ann Doe", "Cars", "Cooking", "Gaming", "Jack Doe", "Joe Doe", "Kuala Lumpur", - "Penang", "Programming" ); + verifyOrderedResults( query, "Ann Doe", "Cars", "Cooking", "Felix", "Gaming", "Jack Doe", "Joe Doe", + "Kuala Lumpur", "Penang", "Programming" ); } @Test @@ -301,7 +303,7 @@ .where( queries.get( "script19" ) ) ); query.orderBy( orderBy( nameable.name() ) ); System.out.println( "*** script19: " + query ); - verifyOrderedResults( query, "Gaming", "Jack Doe", "Joe Doe", "Kuala Lumpur", "Penang", "Programming" ); + verifyOrderedResults( query, "Felix", "Gaming", "Jack Doe", "Joe Doe", "Kuala Lumpur", "Penang", "Programming" ); } @Test
diff --git a/core/testsupport/src/main/java/org/apache/zest/test/indexing/AbstractQueryTest.java b/core/testsupport/src/main/java/org/apache/zest/test/indexing/AbstractQueryTest.java index 852094a..b442434 100644 --- a/core/testsupport/src/main/java/org/apache/zest/test/indexing/AbstractQueryTest.java +++ b/core/testsupport/src/main/java/org/apache/zest/test/indexing/AbstractQueryTest.java
@@ -117,7 +117,7 @@ QueryBuilder<Nameable> qb = this.moduleInstance.newQueryBuilder( Nameable.class ); Query<Nameable> query = unitOfWork.newQuery( qb ); System.out.println( "*** script03: " + query ); - verifyUnorderedResults( query, "Joe Doe", "Ann Doe", "Jack Doe", "Penang", "Kuala Lumpur", "Cooking", "Gaming", + verifyUnorderedResults( query, "Felix", "Joe Doe", "Ann Doe", "Jack Doe", "Penang", "Kuala Lumpur", "Cooking", "Gaming", "Programming", "Cars" ); } @@ -314,7 +314,7 @@ query.firstResult( 3 ); query.maxResults( 2 ); System.out.println( "*** script17: " + query ); - verifyOrderedResults( query, "Gaming", "Jack Doe" ); + verifyOrderedResults( query, "Felix", "Gaming" ); } @Test @@ -327,7 +327,7 @@ Query<Nameable> query = unitOfWork.newQuery( qb ); query.orderBy( orderBy( nameable.name() ) ); System.out.println( "*** script18: " + query ); - verifyOrderedResults( query, "Ann Doe", "Cars", "Cooking", "Gaming", "Jack Doe", "Joe Doe", "Kuala Lumpur", + verifyOrderedResults( query, "Ann Doe", "Cars", "Cooking", "Felix", "Gaming", "Jack Doe", "Joe Doe", "Kuala Lumpur", "Penang", "Programming" ); } @@ -341,7 +341,7 @@ Query<Nameable> query = unitOfWork.newQuery( qb.where( gt( nameable.name(), "D" ) ) ); query.orderBy( orderBy( nameable.name() ) ); System.out.println( "*** script19: " + query ); - verifyOrderedResults( query, "Gaming", "Jack Doe", "Joe Doe", "Kuala Lumpur", "Penang", "Programming" ); + verifyOrderedResults( query, "Felix", "Gaming", "Jack Doe", "Joe Doe", "Kuala Lumpur", "Penang", "Programming" ); } @Test
diff --git a/core/testsupport/src/main/java/org/apache/zest/test/indexing/NameableAssert.java b/core/testsupport/src/main/java/org/apache/zest/test/indexing/NameableAssert.java index e6f8c52..32ee9db 100644 --- a/core/testsupport/src/main/java/org/apache/zest/test/indexing/NameableAssert.java +++ b/core/testsupport/src/main/java/org/apache/zest/test/indexing/NameableAssert.java
@@ -19,6 +19,7 @@ import java.util.ArrayList; import java.util.Arrays; +import java.util.Collection; import java.util.Collections; import java.util.HashMap; import java.util.List; @@ -97,7 +98,8 @@ public static String[] allNames() { - return world.values().toArray( new String[ world.size() ] ); + Collection<String> values = world.values(); + return values.toArray( new String[ world.size() ] ); } public static void verifyUnorderedResults( final Iterable<? extends Nameable> results, final String... names )
diff --git a/core/testsupport/src/main/java/org/apache/zest/test/indexing/TestData.java b/core/testsupport/src/main/java/org/apache/zest/test/indexing/TestData.java index 48cd6b5..e765eb0 100644 --- a/core/testsupport/src/main/java/org/apache/zest/test/indexing/TestData.java +++ b/core/testsupport/src/main/java/org/apache/zest/test/indexing/TestData.java
@@ -238,6 +238,7 @@ Cat felix = catBuilder.instance(); felix.name().set( "Felix" ); catBuilder.newInstance(); + NameableAssert.trace( felix ); } unitOfWork.complete(); }