You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by aa...@apache.org on 2021/09/10 20:30:03 UTC

[pulsar] branch master updated: [ZK Tests] Add breaks to prevent fall through in MockZooKeeper (#12007)

This is an automated email from the ASF dual-hosted git repository.

aahmed pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new fd9e1e29 [ZK Tests] Add breaks to prevent fall through in MockZooKeeper (#12007)
fd9e1e29 is described below

commit fd9e1e29e0a373020a0283874b30a39a8bf24c70
Author: Michael Marshall <mi...@datastax.com>
AuthorDate: Fri Sep 10 15:28:46 2021 -0500

    [ZK Tests] Add breaks to prevent fall through in MockZooKeeper (#12007)
---
 testmocks/src/main/java/org/apache/zookeeper/MockZooKeeper.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/testmocks/src/main/java/org/apache/zookeeper/MockZooKeeper.java b/testmocks/src/main/java/org/apache/zookeeper/MockZooKeeper.java
index 9517b24..c7f677d 100644
--- a/testmocks/src/main/java/org/apache/zookeeper/MockZooKeeper.java
+++ b/testmocks/src/main/java/org/apache/zookeeper/MockZooKeeper.java
@@ -931,12 +931,15 @@ public class MockZooKeeper extends ZooKeeper {
                 case ZooDefs.OpCode.create:
                     this.create(op.getPath(), ((org.apache.zookeeper.Op.Create)op).data, null, null);
                     res.add(new OpResult.CreateResult(op.getPath()));
+                    break;
                 case ZooDefs.OpCode.delete:
                     this.delete(op.getPath(), -1);
                     res.add(new OpResult.DeleteResult());
+                    break;
                 case ZooDefs.OpCode.setData:
                     this.create(op.getPath(), ((org.apache.zookeeper.Op.Create)op).data, null, null);
                     res.add(new OpResult.SetDataResult(null));
+                    break;
                 default:
             }
         }