Skip to content

Commit c08102b

Browse files
authored
Move some files from jvmTest to awtTest and from jvmMain to awtMain (#571)
* Move some files from jvmTest to awtTest Because they depend on AWT classes. jvmTest is shared between AWT and Android implementations (cherry picked from commit 616934c) * Move jvmMain -> awtMain
1 parent b4327f2 commit c08102b

34 files changed

+9
-13
lines changed

skiko/build.gradle.kts

+5-3
Original file line numberDiff line numberDiff line change
@@ -658,15 +658,15 @@ fun createObjcCompileTask(
658658
dependsOn(skiaJvmBindingsDir)
659659

660660
val srcDirs = projectDirs(
661-
"src/jvmMain/objectiveC/${os.id}"
661+
"src/awtMain/objectiveC/${os.id}"
662662
)
663663
sourceRoots.set(srcDirs)
664664
val jdkHome = File(System.getProperty("java.home") ?: error("'java.home' is null"))
665665

666666
includeHeadersNonRecursive(jdkHome.resolve("include"))
667667
includeHeadersNonRecursive(jdkHome.resolve("include/darwin"))
668668
includeHeadersNonRecursive(skiaHeadersDirs(skiaJvmBindingsDir.get()))
669-
includeHeadersNonRecursive(projectDir.resolve("src/jvmMain/cpp/include"))
669+
includeHeadersNonRecursive(projectDir.resolve("src/awtMain/cpp/include"))
670670
includeHeadersNonRecursive(projectDir.resolve("src/commonMain/cpp/common/include"))
671671

672672
compiler.set("clang")
@@ -778,12 +778,14 @@ fun createCompileJvmBindingsTask(
778778
val srcDirs = projectDirs(
779779
"src/commonMain/cpp/common",
780780
"src/jvmMain/cpp/common",
781-
"src/jvmMain/cpp/${targetOs.id}",
781+
"src/awtMain/cpp/common",
782+
"src/awtMain/cpp/${targetOs.id}",
782783
"src/jvmTest/cpp"
783784
)
784785
sourceRoots.set(srcDirs)
785786
if (targetOs != OS.Android) includeHeadersNonRecursive(jdkHome.resolve("include"))
786787
includeHeadersNonRecursive(skiaHeadersDirs(skiaJvmBindingsDir.get()))
788+
includeHeadersNonRecursive(projectDir.resolve("src/awtMain/cpp/include"))
787789
includeHeadersNonRecursive(projectDir.resolve("src/jvmMain/cpp/include"))
788790
includeHeadersNonRecursive(projectDir.resolve("src/commonMain/cpp/common/include"))
789791

skiko/src/awtTest/kotlin/org/jetbrains/skiko/AwtFontInterop.kt renamed to skiko/src/awtTest/kotlin/org/jetbrains/skiko/AwtFontInteropTest.kt

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import org.junit.Test
77
import java.awt.Font
88
import java.awt.GraphicsEnvironment
99

10-
class AwtFontInterop {
10+
class AwtFontInteropTest {
1111
private val fontManager = AwtFontManager()
1212

1313
private fun assumeOk() {

skiko/src/jvmTest/kotlin/org/jetbrains/skiko/ConvertorsTests.jvm.kt renamed to skiko/src/awtTest/kotlin/org/jetbrains/skiko/ConvertorsTest.kt

+2-4
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,10 @@
1-
package org.jetbrains.skiko.tests.org.jetbrains.skiko
1+
package org.jetbrains.skiko
22

33
import org.jetbrains.skia.*
4-
import org.jetbrains.skiko.toBitmap
5-
import org.jetbrains.skiko.toBufferedImage
64
import org.junit.Test
75
import kotlin.test.assertEquals
86

9-
class ConvertorsTests {
7+
class ConvertorsTest {
108

119
@Test
1210
fun canConvertBitmapToBufferedImage() {

skiko/src/jvmTest/kotlin/org/jetbrains/skiko/SeveralClassloaders.kt renamed to skiko/src/awtTest/kotlin/org/jetbrains/skiko/SeveralClassloadersTest.kt

+1-2
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import java.net.URL
55
import java.net.URLClassLoader
66
import java.nio.file.Paths
77
import kotlin.concurrent.thread
8-
import kotlin.test.Ignore
98

109
private class PlatformAndURLClassLoader(classpath: List<URL>) :
1110
ClassLoader(getPlatformClassLoader()) {
@@ -69,7 +68,7 @@ private fun newInstance(loader: ClassLoader, fqName: String, vararg args: Any):
6968
.newInstance(*args)
7069
}
7170

72-
class SeveralClassloaders {
71+
class SeveralClassloadersTest {
7372
@Test
7473
fun `load skiko in several classloaders (with skiko path)`() {
7574
check(skikoLibraryPath != null)

skiko/src/jvmTest/kotlin/org/jetbrains/skiko/util/ScreenshotTestRule.kt renamed to skiko/src/awtTest/kotlin/org/jetbrains/skiko/util/ScreenshotTestRule.kt

-3
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,13 @@
11
package org.jetbrains.skiko.util
22

33
import org.jetbrains.skia.Image
4-
import org.jetbrains.skiko.OS
5-
import org.jetbrains.skiko.hostOs
64
import org.jetbrains.skiko.toImage
75
import org.junit.rules.TestRule
86
import org.junit.runner.Description
97
import org.junit.runners.model.Statement
108
import java.awt.Rectangle
119
import java.awt.Robot
1210
import java.io.File
13-
import javax.imageio.ImageIO
1411

1512
// WARNING!!!
1613
// macOS has wrong colors ([128, 128, 128] isn't [128, 128, 128] on screenshot). Only white, black, red and green are correct.

0 commit comments

Comments
 (0)