Skip to content

fix: refactor smoke test tests to use already running gradle instance #1542

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.

Sign up for GitHub

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

Merged
merged 8 commits into from
Mar 6, 2025
Merged

fix: refactor smoke test tests to use already running gradle instance #1542

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
02a80b8
fix: refactor smoke test e2e tests to use already running gradle inst…
0marperez Feb 27, 2025
03b7f49
fix: trigger CI checks stuck in pending
0marperez Feb 27, 2025
ddf9213
fix: forgot to account for windows
0marperez Feb 27, 2025
32dfe51
fix: add .\ to gradlew.bat
0marperez Feb 27, 2025
b411d82
fix: use correct gradle wrapper command for windows
0marperez Mar 3, 2025
cb58979
misc: assert gradle task fails
0marperez Mar 4, 2025
f7577f4
Merge branch 'main' of https://github.com/awslabs/aws-sdk-kotlin into…
0marperez Mar 4, 2025
026b363
fix: wait for process to complete before reading exit code
0marperez Mar 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
45 changes: 33 additions & 12 deletions ...gen/smoke-tests/src/test/kotlin/aws/sdk/kotlin/test/codegen/smoketest/SmokeTestE2ETest.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,16 @@
* SPDX-License-Identifier: Apache-2.0
*/

package aws.sdk.kotlin.tests.codegen.smoketests
package aws.sdk.kotlin.test.codegen.smoketest

import aws.sdk.kotlin.codegen.smoketests.AWS_SERVICE_FILTER
import aws.sdk.kotlin.codegen.smoketests.AWS_SKIP_TAGS
import org.gradle.testkit.runner.GradleRunner
import aws.smithy.kotlin.runtime.util.OsFamily
import aws.smithy.kotlin.runtime.util.PlatformProvider
import java.io.File
import kotlin.test.Test
import kotlin.test.assertContains
import kotlin.test.assertTrue

class SmokeTestE2ETest {
@Test
Expand Down Expand Up @@ -61,18 +63,37 @@ private fun runSmokeTests(
expectingFailure: Boolean = false,
): String {
val sdkRootDir = System.getProperty("user.dir") + "/../../../"

val task = GradleRunner.create()
.withProjectDir(File(sdkRootDir))
.withArguments(
"--stacktrace", // Make sure unexpected errors are debuggable
"-Paws.kotlin.native=false", // FIXME: Remove `-Paws.kotlin.native=false` when Kotlin Native is ready
val processBuilder =
ProcessBuilder(
*gradleWrapperCommand(),
":tests:codegen:smoke-tests:services:$service:smokeTest",
// Make sure unexpected errors are debuggable
"--stacktrace",
// FIXME: Remove `-Paws.kotlin.native=false` when Kotlin Native is ready
"-Paws.kotlin.native=false",
)
.withEnvironment(envVars)
.forwardOutput()
.directory(File(sdkRootDir))
.redirectErrorStream(true)

processBuilder.environment().putAll(envVars)

val buildResult = if (expectingFailure) task.buildAndFail() else task.build()
Copy link
Member

@lauzadis lauzadis Mar 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we no longer asserting that the Gradle task fails? ProcessBuilder should have an option to get the exit error code which could be processed in a similar way to buildAndFail()

val process = processBuilder.start()
val output = process.inputStream.bufferedReader().use { it.readText() }
val exitCode = process.waitFor()

if (expectingFailure) {
assertTrue(exitCode != 0)
}

return buildResult.output
return output
}

/**
* Determines the appropriate Gradle wrapper command based on the operating system.
*/
private fun gradleWrapperCommand() =
if (PlatformProvider.System.osInfo().family == OsFamily.Windows) {
arrayOf("cmd.exe", "/c", "gradlew.bat")
} else {
arrayOf("./gradlew")
}
Loading