Merge branch '1.2.x'

This commit is contained in:
Andy Wilkinson 2015-04-09 13:33:32 +01:00
commit d3c70fa198
2 changed files with 15 additions and 1 deletions

View File

@ -63,7 +63,7 @@ public class InMemoryTraceRepository implements TraceRepository {
Trace trace = new Trace(new Date(), map);
synchronized (this.traces) {
while (this.traces.size() >= this.capacity) {
this.traces.remove(this.capacity - 1);
this.traces.remove(this.reverse ? this.capacity - 1 : 0);
}
if (this.reverse) {
this.traces.add(0, trace);

View File

@ -41,6 +41,20 @@ public class InMemoryTraceRepositoryTests {
List<Trace> traces = this.repository.findAll();
assertEquals(2, traces.size());
assertEquals("bar", traces.get(0).getInfo().get("bar"));
assertEquals("foo", traces.get(1).getInfo().get("bar"));
}
@Test
public void reverseFalse() {
this.repository.setReverse(false);
this.repository.setCapacity(2);
this.repository.add(Collections.<String, Object> singletonMap("foo", "bar"));
this.repository.add(Collections.<String, Object> singletonMap("bar", "foo"));
this.repository.add(Collections.<String, Object> singletonMap("bar", "bar"));
List<Trace> traces = this.repository.findAll();
assertEquals(2, traces.size());
assertEquals("bar", traces.get(1).getInfo().get("bar"));
assertEquals("foo", traces.get(0).getInfo().get("bar"));
}
}