commit 761e42c608689dc595f019a2ac6a92fa1e0d30c7
author Enrico Olivelli <eolivelli@gmail.com> Mon Feb 17 09:12:37 2020 +0100
committer GitHub <noreply@github.com> Mon Feb 17 09:12:37 2020 +0100
tree a2e003516a083872fd2f9948492a84f5d8f36a16
parent 3ae3ad445b9d22b0a6c8b82f8da383955f4d6a4d

Fix method order in order to make the execution of TestEntryLog predictable

### Motivation

TestEntyLog reuses the entryLogger variable across test methods execution, resulting in an unpredictable execution.
We can at least make the test work consistently across environments using JUnit built-in FixMethodOrder annotation

### Changes
FixMethodOrder(MethodSorters.NAME_ASCENDING)

Fixes #2260


Reviewers: Atri Sharma <atri@apache.org>

This closes #2266 from eolivelli/fix/flaky

bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/TestEntryLog.java

diff --git a/bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/TestEntryLog.java b/bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/TestEntryLog.java
index 836fdf2..fd35204 100644
--- a/bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/TestEntryLog.java
+++ b/bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/TestEntryLog.java
@@ -67,13 +67,16 @@
 import org.junit.After;
 import org.junit.Assert;
 import org.junit.Before;
+import org.junit.FixMethodOrder;
 import org.junit.Test;
+import org.junit.runners.MethodSorters;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 /**
  * Tests for EntryLog.
  */
+@FixMethodOrder(MethodSorters.NAME_ASCENDING)
 public class TestEntryLog {
     private static final Logger LOG = LoggerFactory.getLogger(TestEntryLog.class);