Skip to content

Commit 9e686d4

Browse files
authored
remove useless check in tests (#1471)
1 parent 423ae3e commit 9e686d4

File tree

7 files changed

+0
-21
lines changed

7 files changed

+0
-21
lines changed

tests-qpid-jms-client/src/test/java/io/streamnative/pulsar/handlers/amqp/AmqpProtocolHandlerTestBase.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -373,12 +373,6 @@ public static void setFieldValue(Class clazz, Object classObj, String fieldName,
373373
field.set(classObj, fieldValue);
374374
}
375375

376-
public void checkPulsarServiceState() {
377-
for (PulsarService pulsarService : pulsarServiceList) {
378-
Mockito.when(pulsarService.getState()).thenReturn(PulsarService.State.Started);
379-
}
380-
}
381-
382376
/**
383377
* Get available proxy port.
384378
* @return random proxy port

tests-qpid-jms-client/src/test/java/io/streamnative/pulsar/handlers/amqp/AmqpTestBase.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,6 @@ public void setup() throws Exception {
6868
NamespaceName.get(ns), "__lookup__").toString());
6969
}
7070
}
71-
72-
checkPulsarServiceState();
7371
}
7472

7573
@AfterClass

tests-qpid-jms-client/src/test/java/io/streamnative/pulsar/handlers/amqp/qpid/core/AoPBrokerAdmin.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,8 +70,6 @@ protected void setup() throws Exception {
7070
NamespaceName.get(ns), "__lookup__").toString());
7171
}
7272
}
73-
74-
checkPulsarServiceState();
7573
}
7674

7775
@Override

tests/src/test/java/com/rabbitmq/client/test/BrokerTestCase.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,6 @@ public void setup() throws Exception {
126126
}
127127
((PulsarClientImpl) getPulsarServiceList().get(0).getClient()).getLookup().
128128
getBroker(TopicName.get("public/vhost1/test")).join();
129-
checkPulsarServiceState();
130129
assumeTrue(shouldRun());
131130
openConnection();
132131
openChannel();

tests/src/test/java/io/streamnative/pulsar/handlers/amqp/AmqpProtocolHandlerTestBase.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -400,12 +400,6 @@ public static void setFieldValue(Class clazz, Object classObj, String fieldName,
400400
field.set(classObj, fieldValue);
401401
}
402402

403-
public void checkPulsarServiceState() {
404-
for (PulsarService pulsarService : pulsarServiceList) {
405-
doReturn(PulsarService.State.Started).when(pulsarService).getState();
406-
}
407-
}
408-
409403
/**
410404
* Get available proxy port.
411405
* @return random proxy port

tests/src/test/java/io/streamnative/pulsar/handlers/amqp/AmqpTestBase.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,6 @@ public void setup() throws Exception {
8383
NamespaceName.get(ns), "__lookup__").toString());
8484
}
8585
}
86-
87-
checkPulsarServiceState();
8886
}
8987

9088
@AfterClass

tests/src/test/java/io/streamnative/pulsar/handlers/amqp/AmqpTokenAuthenticationTestBase.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,6 @@ public void setup() throws Exception {
9393
NamespaceName.get(ns), "__lookup__").toString());
9494
}
9595
}
96-
97-
checkPulsarServiceState();
9896
}
9997

10098
@AfterClass

0 commit comments

Comments
 (0)