This is an automated email from the ASF dual-hosted git repository.
gaojun2048 pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/seatunnel.git
The following commit(s) were added to refs/heads/dev by this push:
new d37746c7b4 [Test] Fix ConnectorSpecificationCheckTest not work (#5820)
d37746c7b4 is described below
commit d37746c7b4ab8cab4d417622be5387ea753a745a
Author: Jia Fan <[email protected]>
AuthorDate: Wed Jan 10 16:23:45 2024 +0800
[Test] Fix ConnectorSpecificationCheckTest not work (#5820)
* [Test] Fix ConnectorSpecificationCheckTest not work
* update
---
.github/workflows/backend.yml | 2 +-
seatunnel-dist/pom.xml | 1 -
.../seatunnel/api/connector/ConnectorSpecificationCheckTest.java | 8 ++++++++
3 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/.github/workflows/backend.yml b/.github/workflows/backend.yml
index d37579b8c8..e636f04c25 100644
--- a/.github/workflows/backend.yml
+++ b/.github/workflows/backend.yml
@@ -130,7 +130,7 @@ jobs:
echo "engine-e2e=$true_or_false" >> $GITHUB_OUTPUT
echo "engine-e2e_files=$file_list" >> $GITHUB_OUTPUT
- api_files=`python tools/update_modules_check/check_file_updates.py
ua $workspace apache/dev origin/$current_branch "seatunnel-api/**"
"seatunnel-common/**" "seatunnel-config/**" "seatunnel-connectors/**"
"seatunnel-core/**" "seatunnel-e2e/seatunnel-e2e-common/**"
"seatunnel-formats/**" "seatunnel-plugin-discovery/**"
"seatunnel-transforms-v2/**" "seatunnel-translation/**"
"seatunnel-e2e/seatunnel-transforms-v2-e2e/**" "seatunnel-connectors/**"
"pom.xml" "**/workflows/**" "tools/**"`
+ api_files=`python tools/update_modules_check/check_file_updates.py
ua $workspace apache/dev origin/$current_branch "seatunnel-api/**"
"seatunnel-common/**" "seatunnel-config/**" "seatunnel-connectors/**"
"seatunnel-core/**" "seatunnel-e2e/seatunnel-e2e-common/**"
"seatunnel-formats/**" "seatunnel-plugin-discovery/**"
"seatunnel-transforms-v2/**" "seatunnel-translation/**"
"seatunnel-e2e/seatunnel-transforms-v2-e2e/**" "seatunnel-connectors/**"
"pom.xml" "**/workflows/**" "tools [...]
true_or_false=${api_files%%$'\n'*}
file_list=${api_files#*$'\n'}
echo "api=$true_or_false" >> $GITHUB_OUTPUT
diff --git a/seatunnel-dist/pom.xml b/seatunnel-dist/pom.xml
index 6a2e784d0f..051c767140 100644
--- a/seatunnel-dist/pom.xml
+++ b/seatunnel-dist/pom.xml
@@ -27,7 +27,6 @@
</parent>
<artifactId>seatunnel-dist</artifactId>
- <packaging>pom</packaging>
<name>SeaTunnel : Dist</name>
<properties>
diff --git
a/seatunnel-dist/src/test/java/org/apache/seatunnel/api/connector/ConnectorSpecificationCheckTest.java
b/seatunnel-dist/src/test/java/org/apache/seatunnel/api/connector/ConnectorSpecificationCheckTest.java
index 34fe44a1cb..243d2fc5a8 100644
---
a/seatunnel-dist/src/test/java/org/apache/seatunnel/api/connector/ConnectorSpecificationCheckTest.java
+++
b/seatunnel-dist/src/test/java/org/apache/seatunnel/api/connector/ConnectorSpecificationCheckTest.java
@@ -29,11 +29,14 @@ import org.apache.seatunnel.common.utils.ReflectionUtils;
import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.Test;
+import lombok.extern.slf4j.Slf4j;
+
import java.lang.reflect.Method;
import java.util.ArrayList;
import java.util.List;
import java.util.Optional;
+@Slf4j
public class ConnectorSpecificationCheckTest {
@Test
@@ -72,6 +75,9 @@ public class ConnectorSpecificationCheckTest {
getProducedCatalogTables.isPresent(),
"Please implement `getProducedCatalogTables` method in
"
+ sourceClass.getSimpleName());
+ log.info(
+ "Check source connector {} successfully",
+ factory.getClass().getSimpleName());
}
}
@@ -107,6 +113,8 @@ public class ConnectorSpecificationCheckTest {
Assertions.assertFalse(
getConsumedType.isPresent(),
"Please remove `getConsumedType` method in " +
sinkClass.getSimpleName());
+ log.info(
+ "Check sink connector {} successfully",
factory.getClass().getSimpleName());
}
}
}
