Add require status condition to pop()

Author: DomGarguilo

core/src/main/java/org/apache/accumulo/core/fate/accumulo/AccumuloStore.java

@@ -268,7 +268,9 @@ public void pop() {
       verifyReserved(true);
 
       Optional<Integer> top = findTop();
-      top.ifPresent(t -> newMutator(fateId).deleteRepo(t).mutate());
+      top.ifPresent(t -> newMutator(fateId)
+          .requireStatus(TStatus.FAILED_IN_PROGRESS, TStatus.SUCCESSFUL, TStatus.FAILED)
+          .deleteRepo(t).mutate());
     }
 
     @Override

test/src/main/java/org/apache/accumulo/test/fate/accumulo/FateStoreIT.java

@@ -100,6 +100,7 @@ protected void testReadWrite(FateStore<TestEnv> store, ServerContext sctx)
     assertEquals(TestRepo.class, ops.get(1).getClass());
 
     // test pop, TestOperation should be left
+    txStore.setStatus(TStatus.FAILED); // needed to satisfy the condition on pop
     txStore.pop();
     ops = txStore.getStack();
     assertEquals(1, ops.size());