Skip to content
Snippets Groups Projects
Commit f307fa73 authored by Nils Christian Ehmke's avatar Nils Christian Ehmke
Browse files

Fixed a bug in the time filter

parent 0ab536fb
No related branches found
No related tags found
No related merge requests found
Hide whitespace changes
Inline Side-by-side
kieker-trace-diagnosis-application/src/main/java/kieker/diagnosis/service/filter/FilterService.java
+ 8
− 0
View file @ f307fa73
......@@ -74,6 +74,10 @@ public class FilterService extends ServiceBase {
// Compare the date
if ( aLowerDate != null ) {
final LocalDate localDate = LocalDate.from( zonedDateTime );
if ( localDate.isAfter( aLowerDate ) ) {
// We don't have to check the time. It is another day.
return true;
}
if ( localDate.isBefore( aLowerDate ) ) {
return false;
}
......@@ -125,6 +129,10 @@ public class FilterService extends ServiceBase {
// Compare the date
if ( aUpperDate != null ) {
final LocalDate localDate = LocalDate.from( zonedDateTime );
if ( localDate.isBefore( aUpperDate ) ) {
// We don't have to check the time. It is another day.
return true;
}
if ( localDate.isAfter( aUpperDate ) ) {
return false;
}
......
kieker-trace-diagnosis-application/src/test/java/kieker/diagnosis/service/aggregatedmethods/AggregatedMethodsServiceTest.java
+ 26
− 0
View file @ f307fa73
......@@ -55,6 +55,32 @@ public class AggregatedMethodsServiceTest {
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
}
@Test
public void testSearchWithRegularExpressions( ) {
// Prepare some data for the search
createMethodCall( "host1", "class1", "op1", "cause1" );
createMethodCall( "host1", "class2", "op1", "cause1" );
createMethodCall( "host1", "class1", "op3", "cause1" );
createMethodCall( "host1", "class1", "op3", "cause4" );
assertThat( ivMethodsService.countMethods( ), is( 4 ) );
// Now search with a filter
final AggregatedMethodsFilter methodsFilter = new AggregatedMethodsFilter( );
methodsFilter.setUseRegExpr( true );
methodsFilter.setHost( ".*host1.*" );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 4 ) );
methodsFilter.setClazz( ".*class1.*" );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 3 ) );
methodsFilter.setMethod( ".*op3.*" );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 2 ) );
methodsFilter.setException( ".*cause4.*" );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
}
@Test
public void testSearchTypeFilter( ) {
// Prepare some data for the search
......
kieker-trace-diagnosis-application/src/test/java/kieker/diagnosis/service/methods/MethodsServiceTest.java
+ 196
− 0
View file @ f307fa73
......@@ -3,6 +3,9 @@ package kieker.diagnosis.service.methods;
import static org.hamcrest.core.Is.is;
import static org.junit.Assert.assertThat;
import java.time.LocalDate;
import java.util.Calendar;
import org.junit.Before;
import org.junit.Test;
......@@ -55,6 +58,32 @@ public class MethodsServiceTest {
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
}
@Test
public void testSearchWithRegularExpressions( ) {
// Prepare some data for the search
createMethodCall( "host1", "class1", "op1", "cause1" );
createMethodCall( "host1", "class2", "op1", "cause1" );
createMethodCall( "host1", "class1", "op3", "cause1" );
createMethodCall( "host1", "class1", "op3", "cause4" );
assertThat( ivMethodsService.countMethods( ), is( 4 ) );
// Now search with a filter
final MethodsFilter methodsFilter = new MethodsFilter( );
methodsFilter.setUseRegExpr( true );
methodsFilter.setHost( ".*host1.*" );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 4 ) );
methodsFilter.setClazz( ".*class1.*" );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 3 ) );
methodsFilter.setMethod( ".*op3.*" );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 2 ) );
methodsFilter.setException( ".*cause4.*" );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
}
@Test
public void testSearchTypeFilter( ) {
// Prepare some data for the search
......@@ -77,6 +106,156 @@ public class MethodsServiceTest {
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
}
@Test
public void testTraceIdFilter( ) {
// Prepare some data for the search
createMethodCall( 1L );
createMethodCall( 1L );
createMethodCall( 2L );
createMethodCall( 3L );
assertThat( ivMethodsService.countMethods( ), is( 4 ) );
// Now search with a filter
final MethodsFilter methodsFilter = new MethodsFilter( );
methodsFilter.setTraceId( 1L );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 2 ) );
methodsFilter.setTraceId( 2L );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
methodsFilter.setTraceId( 3L );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
methodsFilter.setTraceId( 4L );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 0 ) );
}
@Test
public void testLowerTimeFilter( ) {
// Prepare some data for the search
createMethodCall( 2000, 05, 01, 15, 20 );
createMethodCall( 2000, 05, 05, 15, 20 );
createMethodCall( 2015, 01, 04, 15, 25 );
createMethodCall( 2015, 01, 01, 15, 20 );
assertThat( ivMethodsService.countMethods( ), is( 4 ) );
// Now search with a filter
final MethodsFilter methodsFilter = new MethodsFilter( );
methodsFilter.setLowerDate( LocalDate.of( 2000, 05, 01 ) );
methodsFilter.setLowerTime( getCalendarFor( 15, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 4 ) );
methodsFilter.setLowerDate( LocalDate.of( 2000, 05, 01 ) );
methodsFilter.setLowerTime( getCalendarFor( 14, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 4 ) );
methodsFilter.setLowerDate( LocalDate.of( 2000, 05, 01 ) );
methodsFilter.setLowerTime( getCalendarFor( 16, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 3 ) );
methodsFilter.setLowerDate( LocalDate.of( 2000, 05, 01 ) );
methodsFilter.setLowerTime( getCalendarFor( 15, 21 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 3 ) );
methodsFilter.setLowerDate( LocalDate.of( 2015, 01, 01 ) );
methodsFilter.setLowerTime( getCalendarFor( 15, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 2 ) );
methodsFilter.setLowerDate( LocalDate.of( 2015, 01, 04 ) );
methodsFilter.setLowerTime( getCalendarFor( 15, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
methodsFilter.setLowerDate( LocalDate.of( 2015, 01, 05 ) );
methodsFilter.setLowerTime( getCalendarFor( 10, 10 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 0 ) );
methodsFilter.setLowerDate( LocalDate.of( 2015, 01, 04 ) );
methodsFilter.setLowerTime( getCalendarFor( 15, 26 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 0 ) );
methodsFilter.setLowerDate( null );
methodsFilter.setLowerTime( null );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 4 ) );
methodsFilter.setLowerDate( LocalDate.of( 2015, 01, 01 ) );
methodsFilter.setLowerTime( null );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 2 ) );
methodsFilter.setLowerDate( null );
methodsFilter.setLowerTime( getCalendarFor( 15, 25 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
}
@Test
public void testUpperTimeFilter( ) {
// Prepare some data for the search
createMethodCall( 2000, 05, 01, 15, 20 );
createMethodCall( 2000, 05, 05, 15, 20 );
createMethodCall( 2015, 01, 04, 15, 25 );
createMethodCall( 2015, 01, 01, 15, 20 );
assertThat( ivMethodsService.countMethods( ), is( 4 ) );
// Now search with a filter
final MethodsFilter methodsFilter = new MethodsFilter( );
methodsFilter.setUpperDate( LocalDate.of( 2000, 05, 01 ) );
methodsFilter.setUpperTime( getCalendarFor( 15, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
methodsFilter.setUpperDate( LocalDate.of( 2000, 05, 01 ) );
methodsFilter.setUpperTime( getCalendarFor( 14, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 0 ) );
methodsFilter.setUpperDate( LocalDate.of( 2000, 05, 01 ) );
methodsFilter.setUpperTime( getCalendarFor( 16, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 1 ) );
methodsFilter.setUpperDate( LocalDate.of( 2000, 05, 05 ) );
methodsFilter.setUpperTime( getCalendarFor( 15, 21 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 2 ) );
methodsFilter.setUpperDate( LocalDate.of( 2015, 01, 01 ) );
methodsFilter.setUpperTime( getCalendarFor( 15, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 3 ) );
methodsFilter.setUpperDate( LocalDate.of( 2015, 01, 04 ) );
methodsFilter.setUpperTime( getCalendarFor( 15, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 3 ) );
methodsFilter.setUpperDate( LocalDate.of( 2015, 01, 05 ) );
methodsFilter.setUpperTime( getCalendarFor( 10, 10 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 4 ) );
methodsFilter.setUpperDate( LocalDate.of( 2015, 01, 04 ) );
methodsFilter.setUpperTime( getCalendarFor( 15, 26 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 4 ) );
methodsFilter.setUpperDate( null );
methodsFilter.setUpperTime( null );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 4 ) );
methodsFilter.setUpperDate( LocalDate.of( 2015, 01, 01 ) );
methodsFilter.setUpperTime( null );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 3 ) );
methodsFilter.setUpperDate( null );
methodsFilter.setUpperTime( getCalendarFor( 15, 25 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 4 ) );
methodsFilter.setUpperDate( null );
methodsFilter.setUpperTime( getCalendarFor( 15, 20 ) );
assertThat( ivMethodsService.searchMethods( methodsFilter ).size( ), is( 3 ) );
}
private Calendar getCalendarFor( final int aHour, final int aMinute ) {
final Calendar calendar = Calendar.getInstance( );
calendar.set( Calendar.HOUR_OF_DAY, aHour );
calendar.set( Calendar.MINUTE, aMinute );
return calendar;
}
private void createMethodCall( final String aHost, final String aClazz, final String aMethod, final String aException ) {
final MethodCall methodCall = new MethodCall( );
methodCall.setHost( aHost );
......@@ -87,4 +266,21 @@ public class MethodsServiceTest {
ivDataService.getMethods( ).add( methodCall );
}
private void createMethodCall( final long aTraceId ) {
final MethodCall methodCall = new MethodCall( );
methodCall.setTraceId( aTraceId );
ivDataService.getMethods( ).add( methodCall );
}
private void createMethodCall( final int aYear, final int aMonth, final int aDay, final int aHour, final int aMinute ) {
final Calendar calendar = Calendar.getInstance( );
calendar.set( aYear, aMonth - 1, aDay, aHour, aMinute, 0 );
final MethodCall methodCall = new MethodCall( );
methodCall.setTimestamp( calendar.getTimeInMillis( ) );
ivDataService.getMethods( ).add( methodCall );
}
}
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment