sijie closed pull request #2732: [integration tests] add a smoketest for pulsar
standalone
URL: https://github.com/apache/pulsar/pull/2732
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/PulsarContainer.java
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/PulsarContainer.java
index ad77b6280f..7650d8feff 100644
---
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/PulsarContainer.java
+++
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/PulsarContainer.java
@@ -101,6 +101,8 @@ protected void configure() {
}
}
+ protected void beforeStart() {}
+
@Override
public void start() {
if (httpPort > 0 && servicePort < 0) {
@@ -118,7 +120,8 @@ public void start() {
createContainerCmd.withName(getContainerName());
createContainerCmd.withEntrypoint(serviceEntryPoint);
});
-
+
+ beforeStart();
super.start();
log.info("Start pulsar service {} at container {}", serviceName,
containerName);
}
diff --git
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/StandaloneContainer.java
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/StandaloneContainer.java
new file mode 100644
index 0000000000..60a574861d
--- /dev/null
+++
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/StandaloneContainer.java
@@ -0,0 +1,65 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pulsar.tests.integration.containers;
+
+import static java.time.temporal.ChronoUnit.SECONDS;
+
+import java.time.Duration;
+import org.testcontainers.containers.wait.strategy.HttpWaitStrategy;
+
+/**
+ * A pulsar container that runs standalone.
+ */
+public class StandaloneContainer extends PulsarContainer<StandaloneContainer> {
+
+ public static final String NAME = "standalone";
+
+ public StandaloneContainer(String clusterName) {
+ super(clusterName,
+ NAME,
+ NAME + "-cluster",
+ "bin/pulsar",
+ BROKER_PORT,
+ BROKER_HTTP_PORT);
+ }
+
+ @Override
+ protected void configure() {
+ super.configure();
+ setCommand("standalone");
+ }
+
+ @Override
+ protected void beforeStart() {
+ // update the wait strategy until public/default namespace is created
+ this.waitStrategy = new HttpWaitStrategy()
+ .forPort(BROKER_HTTP_PORT)
+ .forStatusCode(200)
+ .forPath("/admin/v2/namespaces/public/default")
+ .withStartupTimeout(Duration.of(300, SECONDS));
+ }
+
+ public String getPlainTextServiceUrl() {
+ return "pulsar://" + getContainerIpAddress() + ":" +
getMappedPort(BROKER_PORT);
+ }
+
+ public String getHttpServiceUrl() {
+ return "http://"; + getContainerIpAddress() + ":" +
getMappedPort(BROKER_HTTP_PORT);
+ }
+}
diff --git
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/semantics/SemanticsTest.java
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/semantics/SemanticsTest.java
index 84dfad1cb5..37f3983720 100644
---
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/semantics/SemanticsTest.java
+++
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/semantics/SemanticsTest.java
@@ -41,7 +41,7 @@
import org.apache.pulsar.client.api.SubscriptionType;
import org.apache.pulsar.client.impl.BatchMessageIdImpl;
import org.apache.pulsar.client.impl.TopicMessageIdImpl;
-import org.apache.pulsar.tests.integration.topologies.PulsarClusterTestBase;
+import org.apache.pulsar.tests.integration.suites.PulsarTestSuite;
import org.testng.annotations.Test;
import org.testng.collections.Lists;
@@ -49,7 +49,7 @@
* Test pulsar produce/consume semantics
*/
@Slf4j
-public class SemanticsTest extends PulsarClusterTestBase {
+public class SemanticsTest extends PulsarTestSuite {
//
// Test Basic Publish & Consume Operations
@@ -57,34 +57,7 @@
@Test(dataProvider = "ServiceUrlAndTopics")
public void testPublishAndConsume(String serviceUrl, boolean isPersistent)
throws Exception {
- String topicName = generateTopicName("testpubconsume", isPersistent);
-
- int numMessages = 10;
-
- try (PulsarClient client = PulsarClient.builder()
- .serviceUrl(serviceUrl)
- .build()) {
-
- try (Consumer<String> consumer = client.newConsumer(Schema.STRING)
- .topic(topicName)
- .subscriptionName("my-sub")
- .subscribe()) {
-
- try (Producer<String> producer =
client.newProducer(Schema.STRING)
- .topic(topicName)
- .create()) {
-
- for (int i = 0; i < numMessages; i++) {
- producer.send("smoke-message-" + i);
- }
- }
-
- for (int i = 0; i < numMessages; i++) {
- Message<String> m = consumer.receive();
- assertEquals("smoke-message-" + i, m.getValue());
- }
- }
- }
+ super.testPublishAndConsume(serviceUrl, isPersistent);
}
@Test(dataProvider = "ServiceUrls")
diff --git
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/standalone/SmokeTest.java
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/standalone/SmokeTest.java
new file mode 100644
index 0000000000..2b658fa97a
--- /dev/null
+++
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/standalone/SmokeTest.java
@@ -0,0 +1,32 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pulsar.tests.integration.standalone;
+
+import org.apache.pulsar.tests.integration.suites.PulsarStandaloneTestSuite;
+import org.testng.annotations.Test;
+
+
+public class SmokeTest extends PulsarStandaloneTestSuite {
+
+ @Test(dataProvider = "StandaloneServiceUrlAndTopics")
+ public void testPublishAndConsume(String serviceUrl, boolean isPersistent)
throws Exception {
+ super.testPublishAndConsume(serviceUrl, isPersistent);
+ }
+
+}
diff --git
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/suites/PulsarStandaloneTestSuite.java
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/suites/PulsarStandaloneTestSuite.java
new file mode 100644
index 0000000000..15e82e35ca
--- /dev/null
+++
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/suites/PulsarStandaloneTestSuite.java
@@ -0,0 +1,43 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pulsar.tests.integration.suites;
+
+import org.apache.pulsar.tests.integration.topologies.PulsarStandaloneTestBase;
+import org.testng.ITest;
+import org.testng.annotations.AfterSuite;
+import org.testng.annotations.BeforeSuite;
+
+public class PulsarStandaloneTestSuite extends PulsarStandaloneTestBase
implements ITest {
+
+ @BeforeSuite
+ public void setUpCluster() throws Exception {
+ super.startCluster();
+ }
+
+ @AfterSuite
+ public void tearDownCluster() throws Exception {
+ super.stopCluster();
+ }
+
+
+ @Override
+ public String getTestName() {
+ return "pulsar-standalone-suite";
+ }
+}
diff --git
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/topologies/PulsarClusterTestBase.java
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/topologies/PulsarClusterTestBase.java
index 6181192db0..883cd6f877 100644
---
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/topologies/PulsarClusterTestBase.java
+++
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/topologies/PulsarClusterTestBase.java
@@ -21,13 +21,12 @@
import lombok.extern.slf4j.Slf4j;
import org.testng.annotations.DataProvider;
-import java.util.concurrent.ThreadLocalRandom;
import java.util.stream.Stream;
import static java.util.stream.Collectors.joining;
@Slf4j
-public abstract class PulsarClusterTestBase {
+public abstract class PulsarClusterTestBase extends PulsarTestBase {
@DataProvider(name = "ServiceUrlAndTopics")
public static Object[][] serviceUrlAndTopics() {
@@ -112,36 +111,4 @@ public void tearDownCluster() {
}
}
- public static String randomName(int numChars) {
- StringBuilder sb = new StringBuilder();
- for (int i = 0; i < numChars; i++) {
- sb.append((char) (ThreadLocalRandom.current().nextInt(26) + 'a'));
- }
- return sb.toString();
- }
-
- protected static String generateNamespaceName() {
- return "ns-" + randomName(8);
- }
-
- protected static String generateTopicName(String topicPrefix, boolean
isPersistent) {
- return generateTopicName("default", topicPrefix, isPersistent);
- }
-
- protected static String generateTopicName(String namespace, String
topicPrefix, boolean isPersistent) {
- String topicName = new StringBuilder(topicPrefix)
- .append("-")
- .append(randomName(8))
- .append("-")
- .append(System.currentTimeMillis())
- .toString();
- if (isPersistent) {
- return "persistent://public/" + namespace + "/" + topicName;
- } else {
- return "non-persistent://public/" + namespace + "/" + topicName;
- }
- }
-
-
-
}
diff --git
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/topologies/PulsarStandaloneTestBase.java
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/topologies/PulsarStandaloneTestBase.java
new file mode 100644
index 0000000000..48c7fe6beb
--- /dev/null
+++
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/topologies/PulsarStandaloneTestBase.java
@@ -0,0 +1,83 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pulsar.tests.integration.topologies;
+
+import static org.testng.Assert.assertEquals;
+
+import lombok.extern.slf4j.Slf4j;
+import org.apache.pulsar.tests.integration.containers.StandaloneContainer;
+import org.apache.pulsar.tests.integration.docker.ContainerExecResult;
+import org.testcontainers.containers.Network;
+import org.testng.annotations.DataProvider;
+
+/**
+ * A test base to run tests on standalone cluster.
+ *
+ * <p>Ideally we should run all integration tests on both cluster mode and
standalone
+ * mode. However the apache ci can't really afford to do so. so we run all the
integration
+ * tests on cluster mode. We only run basic validation and test new features
(e.g. state)
+ * on standalone.
+ */
+@Slf4j
+public abstract class PulsarStandaloneTestBase extends PulsarTestBase {
+
+ @DataProvider(name = "StandaloneServiceUrlAndTopics")
+ public static Object[][] serviceUrlAndTopics() {
+ return new Object[][] {
+ // plain text, persistent topic
+ {
+ container.getPlainTextServiceUrl(),
+ true,
+ },
+ // plain text, non-persistent topic
+ {
+ container.getPlainTextServiceUrl(),
+ false
+ }
+ };
+ }
+
+ protected static Network network;
+ protected static StandaloneContainer container;
+
+ protected void startCluster() throws Exception {
+ network = Network.newNetwork();
+ String clusterName = PulsarClusterTestBase.randomName(8);
+ container = new StandaloneContainer(clusterName)
+ .withNetwork(network)
+ .withNetworkAliases(StandaloneContainer.NAME + "-" + clusterName);
+ container.tailContainerLog();
+ container.start();
+ log.info("Pulsar cluster {} is up running:", clusterName);
+ log.info("\tBinary Service Url : {}",
container.getPlainTextServiceUrl());
+ log.info("\tHttp Service Url : {}", container.getHttpServiceUrl());
+
+ // add cluster to public tenant
+ ContainerExecResult result = container.execCmd(
+ "/pulsar/bin/pulsar-admin", "namespaces", "policies",
"public/default");
+ assertEquals(0, result.getExitCode());
+ log.info("public/default namespace policies are {}",
result.getStdout());
+ }
+
+ protected void stopCluster() throws Exception {
+ container.stop();
+ network.close();
+ }
+
+}
diff --git
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/topologies/PulsarTestBase.java
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/topologies/PulsarTestBase.java
new file mode 100644
index 0000000000..2724be2003
--- /dev/null
+++
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/topologies/PulsarTestBase.java
@@ -0,0 +1,93 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pulsar.tests.integration.topologies;
+
+import static org.testng.Assert.assertEquals;
+
+import java.util.concurrent.ThreadLocalRandom;
+import org.apache.pulsar.client.api.Consumer;
+import org.apache.pulsar.client.api.Message;
+import org.apache.pulsar.client.api.Producer;
+import org.apache.pulsar.client.api.PulsarClient;
+import org.apache.pulsar.client.api.Schema;
+
+public class PulsarTestBase {
+
+ public static String randomName(int numChars) {
+ StringBuilder sb = new StringBuilder();
+ for (int i = 0; i < numChars; i++) {
+ sb.append((char) (ThreadLocalRandom.current().nextInt(26) + 'a'));
+ }
+ return sb.toString();
+ }
+
+ protected static String generateNamespaceName() {
+ return "ns-" + randomName(8);
+ }
+
+ protected static String generateTopicName(String topicPrefix, boolean
isPersistent) {
+ return generateTopicName("default", topicPrefix, isPersistent);
+ }
+
+ protected static String generateTopicName(String namespace, String
topicPrefix, boolean isPersistent) {
+ String topicName = new StringBuilder(topicPrefix)
+ .append("-")
+ .append(randomName(8))
+ .append("-")
+ .append(System.currentTimeMillis())
+ .toString();
+ if (isPersistent) {
+ return "persistent://public/" + namespace + "/" + topicName;
+ } else {
+ return "non-persistent://public/" + namespace + "/" + topicName;
+ }
+ }
+
+ public void testPublishAndConsume(String serviceUrl, boolean isPersistent)
throws Exception {
+ String topicName = generateTopicName("testpubconsume", isPersistent);
+
+ int numMessages = 10;
+
+ try (PulsarClient client = PulsarClient.builder()
+ .serviceUrl(serviceUrl)
+ .build()) {
+
+ try (Consumer<String> consumer = client.newConsumer(Schema.STRING)
+ .topic(topicName)
+ .subscriptionName("my-sub")
+ .subscribe()) {
+
+ try (Producer<String> producer =
client.newProducer(Schema.STRING)
+ .topic(topicName)
+ .create()) {
+
+ for (int i = 0; i < numMessages; i++) {
+ producer.send("smoke-message-" + i);
+ }
+ }
+
+ for (int i = 0; i < numMessages; i++) {
+ Message<String> m = consumer.receive();
+ assertEquals("smoke-message-" + i, m.getValue());
+ }
+ }
+ }
+ }
+
+}
diff --git a/tests/integration/src/test/resources/pulsar-standalone.xml
b/tests/integration/src/test/resources/pulsar-standalone.xml
new file mode 100644
index 0000000000..8a5a823867
--- /dev/null
+++ b/tests/integration/src/test/resources/pulsar-standalone.xml
@@ -0,0 +1,28 @@
+<!--
+
+ Licensed to the Apache Software Foundation (ASF) under one
+ or more contributor license agreements. See the NOTICE file
+ distributed with this work for additional information
+ regarding copyright ownership. The ASF licenses this file
+ to you under the Apache License, Version 2.0 (the
+ "License"); you may not use this file except in compliance
+ with the License. You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+ Unless required by applicable law or agreed to in writing,
+ software distributed under the License is distributed on an
+ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ KIND, either express or implied. See the License for the
+ specific language governing permissions and limitations
+ under the License.
+
+-->
+<!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd"; >
+<suite name="Pulsar Standalone Tests" verbose="2" annotations="JDK">
+ <test name="pulsar-standalone-suite" preserve-order="true" >
+ <classes>
+ <class
name="org.apache.pulsar.tests.integration.standalone.SmokeTest" />
+ </classes>
+ </test>
+</suite>
\ No newline at end of file
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
