This is an automated email from the ASF dual-hosted git repository.
yangjie01 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 2723bb11e3c7 [SPARK-46476][BUILD][CORE][CONNECT] Move `IvyTestUtils`
back to `src/test` directory
2723bb11e3c7 is described below
commit 2723bb11e3c7b9e2252e26cca904e2e53aaa68fb
Author: yangjie01 <[email protected]>
AuthorDate: Thu Dec 21 20:50:34 2023 +0800
[SPARK-46476][BUILD][CORE][CONNECT] Move `IvyTestUtils` back to `src/test`
directory
### What changes were proposed in this pull request?
This pr move `IvyTestUtils` back to `src/test` directory because it has
been in the `src/test` directory before the refactoring work of
https://github.com/apache/spark/pull/43354.
Meanwhile, in order to make the `core` and `connect-client-jvm` module use
`IvyTestUtils` in the tests, this pr has added the corresponding Maven
dependencies in the respective `pom.xml` files.
### Why are the changes needed?
Move `IvyTestUtils` back to `src/test` directory
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Pass GitHub Actions
### Was this patch authored or co-authored using generative AI tooling?
No
Closes #44440 from LuciferYang/mv-IvyTestUtils-to-test-dir.
Authored-by: yangjie01 <[email protected]>
Signed-off-by: yangjie01 <[email protected]>
---
.../{main => test}/scala/org/apache/spark/util/IvyTestUtils.scala | 0
connector/connect/client/jvm/pom.xml | 7 +++++++
core/pom.xml | 7 +++++++
3 files changed, 14 insertions(+)
diff --git
a/common/utils/src/main/scala/org/apache/spark/util/IvyTestUtils.scala
b/common/utils/src/test/scala/org/apache/spark/util/IvyTestUtils.scala
similarity index 100%
rename from common/utils/src/main/scala/org/apache/spark/util/IvyTestUtils.scala
rename to common/utils/src/test/scala/org/apache/spark/util/IvyTestUtils.scala
diff --git a/connector/connect/client/jvm/pom.xml
b/connector/connect/client/jvm/pom.xml
index eb98e4203a96..8057a33df178 100644
--- a/connector/connect/client/jvm/pom.xml
+++ b/connector/connect/client/jvm/pom.xml
@@ -82,6 +82,13 @@
<artifactId>scalacheck_${scala.binary.version}</artifactId>
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>org.apache.spark</groupId>
+ <artifactId>spark-common-utils_${scala.binary.version}</artifactId>
+ <version>${project.version}</version>
+ <classifier>tests</classifier>
+ <scope>test</scope>
+ </dependency>
<!-- Use mima to perform the compatibility check -->
<dependency>
<groupId>com.typesafe</groupId>
diff --git a/core/pom.xml b/core/pom.xml
index f2d1f4eb144a..c093213bd6b9 100644
--- a/core/pom.xml
+++ b/core/pom.xml
@@ -451,6 +451,13 @@
<classifier>tests</classifier>
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>org.apache.spark</groupId>
+ <artifactId>spark-common-utils_${scala.binary.version}</artifactId>
+ <version>${project.version}</version>
+ <classifier>tests</classifier>
+ <scope>test</scope>
+ </dependency>
<!--
This spark-tags test-dep is needed even though it isn't used in this
module, otherwise testing-cmds that exclude
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
