Updating tests to fix build issues. (#286)
This commit is contained in:
parent
e65e56380b
commit
8d6c7692cb
1 changed files with 18 additions and 4 deletions
|
|
@ -26,6 +26,7 @@ import java.util.LinkedHashMap;
|
||||||
import java.util.List;
|
import java.util.List;
|
||||||
import java.util.Map.Entry;
|
import java.util.Map.Entry;
|
||||||
|
|
||||||
|
import com.amazonaws.services.kinesis.metrics.interfaces.IMetricsScope;
|
||||||
import org.junit.After;
|
import org.junit.After;
|
||||||
import org.junit.Assert;
|
import org.junit.Assert;
|
||||||
import org.junit.Before;
|
import org.junit.Before;
|
||||||
|
|
@ -804,12 +805,20 @@ public class RecordProcessorCheckpointerTest {
|
||||||
// First call to checkpoint
|
// First call to checkpoint
|
||||||
RecordProcessorCheckpointer processingCheckpointer =
|
RecordProcessorCheckpointer processingCheckpointer =
|
||||||
new RecordProcessorCheckpointer(shardInfo, checkpoint, null, metricsFactory);
|
new RecordProcessorCheckpointer(shardInfo, checkpoint, null, metricsFactory);
|
||||||
|
IMetricsScope scope = null;
|
||||||
|
if (MetricsHelper.isMetricsScopePresent()) {
|
||||||
|
scope = MetricsHelper.getMetricsScope();
|
||||||
|
MetricsHelper.unsetMetricsScope();
|
||||||
|
}
|
||||||
ExtendedSequenceNumber sequenceNumber = new ExtendedSequenceNumber("5019");
|
ExtendedSequenceNumber sequenceNumber = new ExtendedSequenceNumber("5019");
|
||||||
processingCheckpointer.setLargestPermittedCheckpointValue(sequenceNumber);
|
processingCheckpointer.setLargestPermittedCheckpointValue(sequenceNumber);
|
||||||
processingCheckpointer.checkpoint();
|
processingCheckpointer.checkpoint();
|
||||||
Assert.assertEquals(sequenceNumber, checkpoint.getCheckpoint(shardId));
|
Assert.assertEquals(sequenceNumber, checkpoint.getCheckpoint(shardId));
|
||||||
verify(metricsFactory).createMetrics();
|
verify(metricsFactory).createMetrics();
|
||||||
Assert.assertFalse(MetricsHelper.isMetricsScopePresent());
|
Assert.assertFalse(MetricsHelper.isMetricsScopePresent());
|
||||||
|
if (scope != null) {
|
||||||
|
MetricsHelper.setMetricsScope(scope);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
|
|
@ -817,15 +826,20 @@ public class RecordProcessorCheckpointerTest {
|
||||||
// First call to checkpoint
|
// First call to checkpoint
|
||||||
RecordProcessorCheckpointer processingCheckpointer =
|
RecordProcessorCheckpointer processingCheckpointer =
|
||||||
new RecordProcessorCheckpointer(shardInfo, checkpoint, null, metricsFactory);
|
new RecordProcessorCheckpointer(shardInfo, checkpoint, null, metricsFactory);
|
||||||
NullMetricsScope scope = new NullMetricsScope();
|
boolean shouldUnset = false;
|
||||||
MetricsHelper.setMetricsScope(scope);
|
if (!MetricsHelper.isMetricsScopePresent()) {
|
||||||
|
shouldUnset = true;
|
||||||
|
MetricsHelper.setMetricsScope(new NullMetricsScope());
|
||||||
|
}
|
||||||
ExtendedSequenceNumber sequenceNumber = new ExtendedSequenceNumber("5019");
|
ExtendedSequenceNumber sequenceNumber = new ExtendedSequenceNumber("5019");
|
||||||
processingCheckpointer.setLargestPermittedCheckpointValue(sequenceNumber);
|
processingCheckpointer.setLargestPermittedCheckpointValue(sequenceNumber);
|
||||||
processingCheckpointer.checkpoint();
|
processingCheckpointer.checkpoint();
|
||||||
Assert.assertEquals(sequenceNumber, checkpoint.getCheckpoint(shardId));
|
Assert.assertEquals(sequenceNumber, checkpoint.getCheckpoint(shardId));
|
||||||
verify(metricsFactory, never()).createMetrics();
|
verify(metricsFactory, never()).createMetrics();
|
||||||
Assert.assertTrue(MetricsHelper.isMetricsScopePresent());
|
Assert.assertTrue(MetricsHelper.isMetricsScopePresent());
|
||||||
assertEquals(scope, MetricsHelper.getMetricsScope());
|
assertEquals(NullMetricsScope.class, MetricsHelper.getMetricsScope().getClass());
|
||||||
|
if (shouldUnset) {
|
||||||
MetricsHelper.unsetMetricsScope();
|
MetricsHelper.unsetMetricsScope();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue