Skip to content

Commit

Permalink
Revert "SNOW-1227362 Re-enable Package Related Tests (#97)"
Browse files Browse the repository at this point in the history
This reverts commit e777558.
  • Loading branch information
sfc-gh-bli committed Jul 24, 2024
1 parent 0b62663 commit c039024
Showing 1 changed file with 16 additions and 2 deletions.
18 changes: 16 additions & 2 deletions src/test/scala/com/snowflake/snowpark/UDFInternalSuite.scala
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,16 @@ class UDFInternalSuite extends TestData {
super.afterAll
}

test("Test temp udf not failing back to upload jar", JavaStoredProcExclude) {
// todo: re-enable in SNOW-1227362
// The new Geometry data type introduced in the release 1.12.0.
// The package suites use the latest Snowpark package on the server side,
// which doesn't have Geometry type, then, all package suites will fail
// before the server side release. So we have to temporarily disable those test suite
// until Snowpark 1.12.0 release.
ignore("Test temp udf not failing back to upload jar", JavaStoredProcExclude) {
val newSession = Session.builder.configFile(defaultProfile).create
val mockSession = spy(newSession)
TestUtils.addDepsToClassPath(mockSession, None, usePackages = true)
TestUtils.addDepsToClassPath(mockSession, None)
val path = UDFClassPath.getPathForClass(classOf[com.snowflake.snowpark.Session]).get

val doubleUDF = mockSession.udf.registerTemporary((x: Int) => x + x)
Expand Down Expand Up @@ -179,6 +185,13 @@ class UDFInternalSuite extends TestData {
}
}

// todo: re-enable in SNOW-1227362
// The new Geometry data type introduced in the release 1.12.0.
// The package suites use the latest Snowpark package on the server side,
// which doesn't have Geometry type, then, all package suites will fail
// before the server side release. So we have to temporarily disable those test suite
// until Snowpark 1.12.0 release.
/*
@UDFPackageTest
class PackageUDFSuite extends UDFSuite {
override def beforeAll: Unit = {
Expand Down Expand Up @@ -214,3 +227,4 @@ class PackageUDTFSuite extends UDTFSuite {
super.afterAll()
}
}
*/

0 comments on commit c039024

Please sign in to comment.