integration tests #253
integration tests #253
Conversation
|
Will enable CI in another PR later. |
ting-yuan
force-pushed
the
kspIT
branch
2 times, most recently
from
15765ba to
4f698b6
Compare
integration-tests/src/test/kotlin/com/google/devtools/ksp/test/TestsCommon.kt
Outdated
Show resolved
Hide resolved
integration-tests/src/test/kotlin/com/google/devtools/ksp/test/TestsCommon.kt
Outdated
Show resolved
Hide resolved
integration-tests/src/test/kotlin/com/google/devtools/ksp/test/TestsCommon.kt
Outdated
Show resolved
Hide resolved
| fun OutputStream.appendText(str: String) { | ||
| this.write(str.toByteArray()) | ||
| } | ||
| class BuilderProcessor : SymbolProcessor { |
There was a problem hiding this comment.
Do you get symbol resolution when writing these tests? If not, I'd consider moving them to a separate project, and make it depends on ksp. Then you can publish these processors to the test repository, and just depend on them from the integration projects.
If there is already IDE support, feel free to ignore my comment.
There was a problem hiding this comment.
Good point. No, IDE doesn't work well with the processors. Will do in a separate patch after supporting specifying a single processor in a shared jar.
|
|
||
| @Test | ||
| fun testPlayground() { | ||
| val gradleRunner = GradleRunner.create().withProjectDir(testProjectDir) |
There was a problem hiding this comment.
I haven't used GradleRunner before. Does each test JVM fork starts its own Gradle daemon?
There was a problem hiding this comment.
According to the doc, yes, although I didn't verify it.
https://docs.gradle.org/current/userguide/test_kit.html#sec:controlling_the_build_environment
An observation from another perspective is that it's a bit slow to start. For integration tests that involves Gradle, I guess I can accept it, though.
...ation-tests/src/test/resources/playground/test-processor/src/main/kotlin/BuilderProcessor.kt
Outdated
Show resolved
Hide resolved
...ation-tests/src/test/resources/playground/test-processor/src/main/kotlin/BuilderProcessor.kt
Outdated
Show resolved
Hide resolved
...ation-tests/src/test/resources/playground/test-processor/src/main/kotlin/BuilderProcessor.kt
Outdated
Show resolved
Hide resolved
No description provided.