Skip to content

Commit

Permalink
[ShadeTest]Add integration shade test for pulsar-client and pulsar-ci…
Browse files Browse the repository at this point in the history
…ent-admin. (apache#8962)

### Motivation
Currently, there are no encryption integration shade test cases in pulsar-client and pulsar-client-admin
apache#8850 add these test cases for pulsar-client-all

### Modification
* Add encryption integration test cases for pulsar-client and pulsar-client-admin.
* Include the shading of pulsar-client-api.
  • Loading branch information
RobertIndie authored Dec 23, 2020
1 parent 6e74a9b commit c17253b
Show file tree
Hide file tree
Showing 34 changed files with 1,635 additions and 17 deletions.
9 changes: 9 additions & 0 deletions pulsar-client-admin-shaded/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,12 @@
<artifactId>pulsar-client-admin-original</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>
<groupId>${project.groupId}</groupId>
<artifactId>pulsar-client-messagecrypto-bc</artifactId>
<version>${project.parent.version}</version>
<optional>true</optional>
</dependency>
</dependencies>
<build>
<plugins>
Expand All @@ -58,6 +64,7 @@
<artifactSet>
<includes>
<include>org.apache.pulsar:pulsar-client-original</include>
<include>org.apache.pulsar:pulsar-client-api</include>
<include>org.apache.pulsar:pulsar-client-admin-original</include>
<include>org.apache.commons:commons-lang3</include>
<include>commons-codec:commons-codec</include>
Expand Down Expand Up @@ -92,6 +99,8 @@
<include>org.yaml:snakeyaml</include>
<include>io.swagger:*</include>
<include>org.apache.bookkeeper:bookkeeper-common-allocator</include>
<!-- Issue #6834, Since Netty ByteBuf shaded, we need also shade this module -->
<include>org.apache.pulsar:pulsar-client-messagecrypto-bc</include>
</includes>
</artifactSet>
<filters>
Expand Down
14 changes: 1 addition & 13 deletions pulsar-client-all/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,6 @@
<name>Pulsar Client All</name>

<dependencies>
<dependency>
<groupId>${project.groupId}</groupId>
<artifactId>pulsar-client-api</artifactId>
<version>${project.parent.version}</version>
</dependency>
<dependency>
<groupId>${project.groupId}</groupId>
<artifactId>pulsar-client-original</artifactId>
Expand Down Expand Up @@ -81,14 +76,6 @@
<includes>**/ProtobufSchema.class</includes>
<outputDirectory>${project.build.directory}/classes</outputDirectory>
</artifactItem>
<artifactItem>
<groupId>${project.groupId}</groupId>
<artifactId>pulsar-client-api</artifactId>
<version>${project.version}</version>
<type>jar</type>
<overWrite>true</overWrite>
<outputDirectory>${project.build.directory}/classes</outputDirectory>
</artifactItem>
</artifactItems>
</configuration>
</execution>
Expand All @@ -113,6 +100,7 @@
<artifactSet>
<includes>
<include>org.apache.pulsar:pulsar-client-original</include>
<include>org.apache.pulsar:pulsar-client-api</include>
<include>org.apache.pulsar:pulsar-client-admin-original</include>
<include>org.apache.commons:commons-lang3</include>
<include>commons-codec:commons-codec</include>
Expand Down
1 change: 1 addition & 0 deletions pulsar-client-shaded/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,7 @@
<artifactSet>
<includes>
<include>org.apache.pulsar:pulsar-client-original</include>
<include>org.apache.pulsar:pulsar-client-api</include>
<include>org.apache.bookkeeper:bookkeeper-common-allocator</include>
<include>org.apache.commons:commons-lang3</include>
<include>commons-codec:commons-codec</include>
Expand Down
7 changes: 7 additions & 0 deletions tests/pulsar-client-admin-shade-test/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,13 @@
<scope>test</scope>
</dependency>

<dependency>
<groupId>org.apache.pulsar</groupId>
<artifactId>pulsar-client-messagecrypto-bc</artifactId>
<version>${project.version}</version>
<scope>test</scope>
</dependency>

</dependencies>

<build>
Expand Down
Loading

0 comments on commit c17253b

Please sign in to comment.