TS-38628 API changing settings, artifactory dump, cucumber maven

Author: Avanatiker

system-tests/api-changing-settings-should-dump/build.gradle.kts

@@ -1,4 +1,5 @@
 plugins {
+    com.teamscale.`kotlin-convention`
     com.teamscale.`system-test-convention`
 }
 

system-tests/api-changing-settings-should-dump/src/main/java/systemundertest/SystemUnderTest.java

@@ -0,0 +1,8 @@
+package systemundertest
+
+/** Fake system under test to generate some coverage.  */
+class SystemUnderTest {
+	fun foo() = 12
+
+	fun bar() = 13
+}

system-tests/api-changing-settings-should-dump/src/main/kotlin/systemundertest/SystemUnderTest.kt

@@ -0,0 +1,38 @@
+package com.teamscale.tia.client
+
+import com.teamscale.client.EReportFormat
+import com.teamscale.test.commons.SystemTestUtils
+import com.teamscale.test.commons.SystemTestUtils.changePartition
+import com.teamscale.test.commons.TeamscaleMockServer
+import org.assertj.core.api.Assertions
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.jupiter.api.Test
+import systemundertest.SystemUnderTest
+
+/**
+ * Runs the system under test and then changes the partition of the agent. This should cause a dump to our
+ * [TeamscaleMockServer].
+ */
+class ApiChangingSettingsShouldDumpSystemTest {
+	@Test
+	@Throws(Exception::class)
+	fun systemTest() {
+		System.setProperty("org.eclipse.jetty.util.log.class", "org.eclipse.jetty.util.log.StdErrLog")
+		System.setProperty("org.eclipse.jetty.LEVEL", "OFF")
+
+		val teamscaleMockServer = TeamscaleMockServer(SystemTestUtils.TEAMSCALE_PORT).acceptingReportUploads()
+
+		SystemUnderTest().foo()
+		changePartition(SystemTestUtils.AGENT_PORT, "some_other_value")
+
+		val session = teamscaleMockServer.onlySession
+		assertThat(session.partition).isEqualTo("partition_before_change")
+		assertThat(session.getOnlyReport(EReportFormat.JACOCO)).contains(
+			"<line nr=\"$METHOD_FOO_COVERABLE_LINE\" mi=\"0\""
+		)
+	}
+
+	companion object {
+		private const val METHOD_FOO_COVERABLE_LINE = 4
+	}
+}

system-tests/api-changing-settings-should-dump/src/test/java/com/teamscale/tia/client/ApiChangingSettingsShouldDumpSystemTest.java

@@ -1,4 +1,5 @@
 plugins {
+	com.teamscale.`kotlin-convention`
 	com.teamscale.`system-test-convention`
 }
 

system-tests/api-changing-settings-should-dump/src/test/kotlin/com/teamscale/tia/client/ApiChangingSettingsShouldDumpSystemTest.kt

@@ -0,0 +1,44 @@
+package com.teamscale.client
+
+import com.teamscale.test.commons.SystemTestUtils
+import com.teamscale.test.commons.SystemTestUtils.dumpCoverage
+import org.assertj.core.api.Assertions
+import org.assertj.core.api.Assertions.assertThat
+import org.conqat.lib.commons.collections.Pair
+import org.junit.jupiter.api.AfterEach
+import org.junit.jupiter.api.BeforeEach
+import org.junit.jupiter.api.Test
+import systemundertest.SystemUnderTest
+
+class ArtifactoryGitPropertiesDetectionTest {
+	private lateinit var artifactoryMockServer: ArtifactoryMockServer
+
+	@BeforeEach
+	fun startFakeTeamscaleServer() {
+		artifactoryMockServer = ArtifactoryMockServer(FAKE_ARTIFACTORY_PORT)
+	}
+
+	@AfterEach
+	fun shutdownFakeTeamscaleServer() {
+		artifactoryMockServer.shutdown()
+	}
+
+	@Test
+	@Throws(Exception::class)
+	fun systemTest() {
+		SystemUnderTest.foo()
+
+		dumpCoverage(SystemTestUtils.AGENT_PORT)
+
+		assertThat(artifactoryMockServer.uploadedReports).hasSize(1)
+		val reports = artifactoryMockServer.uploadedReports
+		assertThat(reports).hasSize(1)
+		// Ensure infos from src/main/resources/git.properties are picked up
+		assertThat(reports.getFirst(0))
+			.startsWith("uploads/master/1645713803000-86f9d655bf8a204d98bc3542e0d15cea38cc7c74/")
+	}
+
+	companion object {
+		private val FAKE_ARTIFACTORY_PORT = Integer.getInteger("artifactoryPort")
+	}
+}

system-tests/artifactory-git-properties-detection/build.gradle.kts

@@ -0,0 +1,75 @@
+package com.teamscale.client
+
+import org.conqat.lib.commons.collections.PairList
+import spark.*
+import java.io.ByteArrayInputStream
+import java.io.ByteArrayOutputStream
+import java.io.IOException
+import java.io.InputStream
+import java.util.function.BiConsumer
+import java.util.zip.ZipEntry
+import java.util.zip.ZipInputStream
+import javax.servlet.http.HttpServletResponse
+
+/**
+ * Mocks a Artifactory server: stores all uploaded reports so tests can run assertions on them.
+ */
+class ArtifactoryMockServer(port: Int) {
+	/** All reports uploaded to this Teamscale instance.  */
+	val uploadedReports = PairList<String, String>()
+	private val service = Service.ignite()
+
+	init {
+		service.port(port)
+		service.put(":path", ::handleReport)
+		service.exception<Exception>(Exception::class.java) { exception, _, response ->
+			response.status(HttpServletResponse.SC_INTERNAL_SERVER_ERROR)
+			response.body("Exception: ${exception.message}")
+		}
+		service.notFound { request, response ->
+			response.status(HttpServletResponse.SC_INTERNAL_SERVER_ERROR)
+			"Unexpected request: ${request.requestMethod()} ${request.uri()}"
+		}
+		service.awaitInitialization()
+	}
+
+	@Throws(IOException::class)
+	private fun handleReport(request: Request, response: Response): String {
+		processZipEntries(ByteArrayInputStream(request.bodyAsBytes())) { entry, content ->
+			if (!entry.isDirectory) {
+				uploadedReports.add("${request.params("path")} -> ${entry.getName()}", content.toString())
+			}
+		}
+		return "success"
+	}
+
+	/**
+	 * Shuts down the mock server and waits for it to be stopped.
+	 */
+	fun shutdown() {
+		service.stop()
+		service.awaitStop()
+	}
+
+	companion object {
+		@Throws(IOException::class)
+		private fun processZipEntries(
+			zipStream: InputStream,
+			entryConsumer: (ZipEntry, ByteArrayOutputStream) -> Unit
+		) {
+			val bufferSize = 1024
+			ZipInputStream(zipStream).use { zipInput ->
+				while (true) {
+					val zipEntry = zipInput.nextEntry ?: break
+					val entryContent = ByteArrayOutputStream()
+					val buffer = ByteArray(bufferSize)
+					var bytesRead: Int
+					while (zipInput.read(buffer).also { bytesRead = it } != -1) {
+						entryContent.write(buffer, 0, bytesRead)
+					}
+					entryConsumer(zipEntry, entryContent)
+				}
+			}
+		}
+	}
+}