-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
micronaut framework implemented. #673
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR!
This PR does not build and compile. Could you please apply the changes, and the open the project from settings.gradle.kts
level. I observe gradle-wrapper
additions which should not be case for any change.
Also instead of micronaut-starter, lets keep micronaut
as the name for the folder.
examples/micronaut-example/gradle/wrapper/gradle-wrapper.properties
Outdated
Show resolved
Hide resolved
starters/micronaut-starter/stove-micronaut-testing-e2e/gradle/wrapper/gradle-wrapper.properties
Outdated
Show resolved
Hide resolved
starters/micronaut-starter/stove-micronaut-testing-e2e/.idea/.gitignore
Outdated
Show resolved
Hide resolved
starters/micronaut-starter/stove-micronaut-testing-e2e/build.gradle.kts
Outdated
Show resolved
Hide resolved
starters/micronaut-starter/stove-micronaut-testing-e2e/build.gradle.kts
Outdated
Show resolved
Hide resolved
...-testing-e2e/src/main/kotlin/com/trendyol/stove/testing/BaseApplicationContextInitializer.kt
Outdated
Show resolved
Hide resolved
...stove-micronaut-testing-e2e/src/test/kotlin/com/trendyol/stove/testing/BridgeSystemKtTest.kt
Outdated
Show resolved
Hide resolved
starters/micronaut/stove-micronaut-testing-e2e/build.gradle.kts
Outdated
Show resolved
Hide resolved
examples/micronaut-example/src/test/kotlin/com/stove/micronaut/example/e2e/TestSystemConfig.kt
Outdated
Show resolved
Hide resolved
...s/micronaut-example/src/test/kotlin/com/stove/micronaut/example/e2e/ProductControllerTest.kt
Outdated
Show resolved
Hide resolved
...s/micronaut-example/src/test/kotlin/com/stove/micronaut/example/e2e/ProductControllerTest.kt
Outdated
Show resolved
Hide resolved
...t-example/src/main/kotlin/stove/micronaut/example/infrastructure/http/SupplierHttpService.kt
Outdated
Show resolved
Hide resolved
Signed-off-by: Anılcan <[email protected]>
Signed-off-by: Anılcan <[email protected]>
Signed-off-by: Anılcan <[email protected]>
detektPlugins(libs.detekt.formatting) | ||
} | ||
|
||
spotless { | ||
kotlin { | ||
ktlint(libs.versions.ktlint.get()).setEditorConfigPath(rootProject.layout.projectDirectory.file(".editorconfig")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not change.
@@ -76,13 +76,12 @@ subprojects.of("lib", "spring", "examples", "ktor") { | |||
testImplementation(libs.kotest.runner.junit5) | |||
testImplementation(libs.kotest.framework.api) | |||
testImplementation(libs.kotest.property) | |||
testImplementation(libs.kotest.arrow) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why removed?
@@ -141,10 +140,11 @@ val publishedProjects = listOf( | |||
"stove-testing-e2e-redis", | |||
"stove-ktor-testing-e2e", | |||
"stove-spring-testing-e2e", | |||
"stove-spring-testing-e2e-kafka" | |||
"stove-spring-testing-e2e-kafka", | |||
"stove-micronaut-testing-e2e" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can remove it for now, until it gets matured, we don't want to publish it.
import org.slf4j.* | ||
|
||
class Stove : AbstractProjectConfig() { | ||
private val logger: Logger = LoggerFactory.getLogger("WireMockMonitor") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WireMockMonitor can't be, it is MicronautExample
@@ -67,6 +72,23 @@ kotlinx-io-reactor-extensions = { module = "io.projectreactor.kotlin:reactor-kot | |||
kotlinx-serialization-json = { module = "org.jetbrains.kotlinx:kotlinx-serialization-json-jvm", version.ref = "kotlinx-serialization" } | |||
kotlinx-serialization-protobuf = { module = "org.jetbrains.kotlinx:kotlinx-serialization-protobuf", version.ref = "kotlinx-serialization" } | |||
|
|||
# Micronaut | |||
snakeyaml = { module = "org.yaml:snakeyaml", version.ref = "snakeyaml" } | |||
micronaut-parent = { module = "io.micronaut.platform:micronaut-parent", version = "4.7.1" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version misalignments, above 4.4.3, but here 4.7.1, all micronaut packages share the same version.
import io.micronaut.runtime.Micronaut | ||
|
||
@StoveDsl | ||
abstract class BaseApplicationContextInitializer : ApplicationEventListener<StartupEvent> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yo don't need this one I believe.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a spring-boot initializer approach, and for micronaut it is not applicable.
} | ||
|
||
companion object { | ||
@JvmStatic fun main(args: Array<String>) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why there is a main here?
override suspend fun afterProject() = TestSystem.stop() | ||
} | ||
|
||
class BridgeSystemTests : FunSpec({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These tests fail. Could you take a look?
micronaut example created
micronaut starter created