Skip to content

Commit 7e8c822

Browse files
whummerwhummer
committed
fix project setup and classpath for SDK v1/v2 utils
1 parent 4868a56 commit 7e8c822

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

46 files changed

+223
-180
lines changed

‎.travis.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ script:
1717
- set -e
1818
- nohup docker pull localstack/localstack > /dev/null &
1919
- make compile
20-
- mvn -q -DskipTests test
20+
- MVN_TEST_ARGS="-q -DskipTests" make test
2121
- make test
2222
- docker ps -a
2323

‎Makefile‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,11 @@ publish-maven: ## Publish artifacts to Maven Central
1313
ADDITIONAL_MVN_TARGETS=deploy ADDITIONAL_MVN_ARGS="-DskipTests" make build
1414

1515
test-v1:
16-
USE_SSL=1 SERVICES=serverless,kinesis,sns,sqs,iam,cloudwatch mvn -Pawssdkv1 \
17-
-Dtest="!cloud.localstack.awssdkv2.*Test" test
16+
USE_SSL=1 SERVICES=serverless,kinesis,sns,sqs,iam,cloudwatch mvn $(MVN_TEST_ARGS) -Pawssdkv1 \
17+
-Dtest="cloud.localstack.awssdkv1.*Test" test
1818

1919
test-v2:
20-
USE_SSL=1 SERVICES=serverless,kinesis,sns,sqs,iam,cloudwatch mvn -Pawssdkv2 \
20+
USE_SSL=1 SERVICES=serverless,kinesis,sns,sqs,iam,cloudwatch mvn $(MVN_TEST_ARGS) -Pawssdkv2 \
2121
-Dtest="cloud.localstack.awssdkv2.*Test" test
2222

2323
test: ## Run Java/JUnit tests for AWS SDK v1 and v2

‎pom.xml‎

Lines changed: 40 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -201,6 +201,24 @@
201201
</profile>
202202
<profile>
203203
<id>awssdkv1</id>
204+
<build>
205+
<plugins>
206+
<plugin>
207+
<groupId>org.apache.maven.plugins</groupId>
208+
<artifactId>maven-compiler-plugin</artifactId>
209+
<configuration>
210+
<excludes>
211+
<exclude>**/awssdkv2/**/*.java</exclude>
212+
<exclude>**/deprecated/**/*.java</exclude>
213+
</excludes>
214+
<testExcludes>
215+
<exclude>**/awssdkv2/**/*.java</exclude>
216+
<exclude>**/deprecated/**/*.java</exclude>
217+
</testExcludes>
218+
</configuration>
219+
</plugin>
220+
</plugins>
221+
</build>
204222
<dependencies>
205223
<dependency>
206224
<groupId>com.amazonaws</groupId>
@@ -248,6 +266,26 @@
248266
</profile>
249267
<profile>
250268
<id>awssdkv2</id>
269+
<build>
270+
<plugins>
271+
<plugin>
272+
<groupId>org.apache.maven.plugins</groupId>
273+
<artifactId>maven-compiler-plugin</artifactId>
274+
<configuration>
275+
<excludes>
276+
<exclude>**/awssdkv1/**/*.java</exclude>
277+
<exclude>**/deprecated/**/*.java</exclude>
278+
<exclude>**/LambdaExecutor.java</exclude>
279+
</excludes>
280+
<testExcludes>
281+
<exclude>**/awssdkv1/**/*.java</exclude>
282+
<exclude>**/deprecated/**/*.java</exclude>
283+
<exclude>**/LambdaExecutor.java</exclude>
284+
</testExcludes>
285+
</configuration>
286+
</plugin>
287+
</plugins>
288+
</build>
251289
<dependencies>
252290
<dependency>
253291
<groupId>software.amazon.awssdk</groupId>
@@ -276,18 +314,6 @@
276314
<version>${aws.sdkv2.version}</version>
277315
<scope>provided</scope>
278316
</dependency>
279-
<dependency>
280-
<groupId>software.amazon.awssdk</groupId>
281-
<artifactId>sns</artifactId>
282-
<version>${aws.sdkv2.version}</version>
283-
<scope>provided</scope>
284-
</dependency>
285-
<dependency>
286-
<groupId>software.amazon.awssdk</groupId>
287-
<artifactId>sqs</artifactId>
288-
<version>${aws.sdkv2.version}</version>
289-
<scope>provided</scope>
290-
</dependency>
291317
<dependency>
292318
<groupId>software.amazon.awssdk</groupId>
293319
<artifactId>ssm</artifactId>
@@ -300,12 +326,6 @@
300326
<version>${aws.sdkv2.version}</version>
301327
<scope>provided</scope>
302328
</dependency>
303-
<dependency>
304-
<groupId>software.amazon.awssdk</groupId>
305-
<artifactId>kinesis</artifactId>
306-
<version>${aws.sdkv2.version}</version>
307-
<scope>provided</scope>
308-
</dependency>
309329
<dependency>
310330
<groupId>software.amazon.awssdk</groupId>
311331
<artifactId>s3</artifactId>
@@ -477,12 +497,12 @@
477497
<plugin>
478498
<groupId>org.apache.maven.plugins</groupId>
479499
<artifactId>maven-surefire-plugin</artifactId>
480-
<version>2.21.0</version>
500+
<version>2.22.1</version>
481501
<dependencies>
482502
<dependency>
483503
<groupId>org.apache.maven.surefire</groupId>
484504
<artifactId>surefire-junit4</artifactId>
485-
<version>2.21.0</version>
505+
<version>2.22.1</version>
486506
</dependency>
487507
<dependency>
488508
<groupId>org.junit.platform</groupId>

‎src/main/java/cloud/localstack/LambdaExecutor.java‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,9 @@
11
package cloud.localstack;
22

3-
import cloud.localstack.lambda.DDBEventParser;
4-
import cloud.localstack.lambda.KinesisEventParser;
5-
import cloud.localstack.lambda.S3EventParser;
3+
import cloud.localstack.awssdkv1.lambda.DDBEventParser;
4+
import cloud.localstack.awssdkv1.lambda.KinesisEventParser;
5+
import cloud.localstack.awssdkv1.lambda.S3EventParser;
6+
67
import com.amazonaws.services.lambda.runtime.Context;
78
import com.amazonaws.services.lambda.runtime.RequestHandler;
89
import com.amazonaws.services.lambda.runtime.RequestStreamHandler;

src/main/java/cloud/localstack/lambda/DDBEventParser.java renamed to src/main/java/cloud/localstack/awssdkv1/lambda/DDBEventParser.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package cloud.localstack.lambda;
1+
package cloud.localstack.awssdkv1.lambda;
22

33
import com.amazonaws.services.dynamodbv2.model.*;
44
import com.amazonaws.services.lambda.runtime.events.DynamodbEvent;

src/main/java/cloud/localstack/lambda/KinesisEventParser.java renamed to src/main/java/cloud/localstack/awssdkv1/lambda/KinesisEventParser.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package cloud.localstack.lambda;
1+
package cloud.localstack.awssdkv1.lambda;
22

33
import com.amazonaws.services.lambda.runtime.events.KinesisEvent;
44
import com.amazonaws.services.lambda.runtime.events.KinesisEvent.KinesisEventRecord;

src/main/java/cloud/localstack/lambda/S3EventParser.java renamed to src/main/java/cloud/localstack/awssdkv1/lambda/S3EventParser.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
package cloud.localstack.lambda;
1+
package cloud.localstack.awssdkv1.lambda;
2+
23
import static cloud.localstack.LambdaExecutor.get;
34
import com.amazonaws.services.lambda.runtime.events.S3Event;
45
import com.amazonaws.services.s3.event.S3EventNotification;

‎src/main/java/cloud/localstack/awssdkv2/PowerMockLocalStack.java‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@
77
import org.powermock.modules.junit4.*;
88
import static org.mockito.Mockito.when;
99

10-
import com.amazonaws.services.dynamodbv2.datamodeling.DynamoDB;
11-
1210
import software.amazon.awssdk.core.client.builder.SdkAsyncClientBuilder;
1311
import software.amazon.awssdk.services.cloudwatch.*;
1412
import software.amazon.awssdk.services.dynamodb.DynamoDbAsyncClient;

‎src/main/java/cloud/localstack/docker/annotation/EC2HostNameResolver.java‎

Lines changed: 0 additions & 23 deletions
This file was deleted.

src/test/java/cloud/localstack/CWMetricsTest.java renamed to src/test/java/cloud/localstack/awssdkv1/CWMetricsTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
1-
package cloud.localstack;
1+
package cloud.localstack.awssdkv1;
22

33
import cloud.localstack.docker.annotation.LocalstackDockerProperties;
4+
import cloud.localstack.LocalstackTestRunner;
45
import cloud.localstack.awssdkv1.TestUtils;
56

67
import com.amazonaws.services.cloudwatch.AmazonCloudWatch;

0 commit comments

Comments
 (0)