Merge pull request #5212 from getsentry/fix/cache-tracing-evict-system-test

adinauer · web-flow · commit d7b3b0bafdf2 · 2026-03-23T15:26:28.000+01:00
fix(samples): [Cache Tracing 22] Fix cache evict system test to match actual span op
diff --git a/sentry-samples/sentry-samples-spring-boot-4-opentelemetry/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt b/sentry-samples/sentry-samples-spring-boot-4-opentelemetry/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt
@@ -45,7 +45,7 @@ class CacheSystemTest {
     restClient.deleteCachedTodo(1L)
 
     testHelper.ensureTransactionReceived { transaction, _ ->
-      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.remove")
+      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.evict")
     }
   }
 }
diff --git a/sentry-samples/sentry-samples-spring-boot-4-otlp/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt b/sentry-samples/sentry-samples-spring-boot-4-otlp/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt
@@ -45,7 +45,7 @@ class CacheSystemTest {
     restClient.deleteCachedTodo(1L)
 
     testHelper.ensureTransactionReceived { transaction, _ ->
-      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.remove")
+      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.evict")
     }
   }
 }
diff --git a/sentry-samples/sentry-samples-spring-boot-4-webflux/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt b/sentry-samples/sentry-samples-spring-boot-4-webflux/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt
@@ -45,7 +45,7 @@ class CacheSystemTest {
     restClient.deleteCachedTodo(1L)
 
     testHelper.ensureTransactionReceived { transaction, _ ->
-      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.remove")
+      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.evict")
     }
   }
 }
diff --git a/sentry-samples/sentry-samples-spring-boot-4/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt b/sentry-samples/sentry-samples-spring-boot-4/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt
@@ -45,7 +45,7 @@ class CacheSystemTest {
     restClient.deleteCachedTodo(1L)
 
     testHelper.ensureTransactionReceived { transaction, _ ->
-      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.remove")
+      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.evict")
     }
   }
 }
diff --git a/sentry-samples/sentry-samples-spring-boot-jakarta-opentelemetry-noagent/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt b/sentry-samples/sentry-samples-spring-boot-jakarta-opentelemetry-noagent/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt
@@ -45,7 +45,7 @@ class CacheSystemTest {
     restClient.deleteCachedTodo(1L)
 
     testHelper.ensureTransactionReceived { transaction, _ ->
-      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.remove")
+      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.evict")
     }
   }
 }
diff --git a/sentry-samples/sentry-samples-spring-boot-jakarta-opentelemetry/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt b/sentry-samples/sentry-samples-spring-boot-jakarta-opentelemetry/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt
@@ -45,7 +45,7 @@ class CacheSystemTest {
     restClient.deleteCachedTodo(1L)
 
     testHelper.ensureTransactionReceived { transaction, _ ->
-      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.remove")
+      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.evict")
     }
   }
 }
diff --git a/sentry-samples/sentry-samples-spring-boot-jakarta/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt b/sentry-samples/sentry-samples-spring-boot-jakarta/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt
@@ -45,7 +45,7 @@ class CacheSystemTest {
     restClient.deleteCachedTodo(1L)
 
     testHelper.ensureTransactionReceived { transaction, _ ->
-      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.remove")
+      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.evict")
     }
   }
 }
diff --git a/sentry-samples/sentry-samples-spring-boot-webflux-jakarta/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt b/sentry-samples/sentry-samples-spring-boot-webflux-jakarta/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt
@@ -45,7 +45,7 @@ class CacheSystemTest {
     restClient.deleteCachedTodo(1L)
 
     testHelper.ensureTransactionReceived { transaction, _ ->
-      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.remove")
+      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.evict")
     }
   }
 }
diff --git a/sentry-samples/sentry-samples-spring-boot/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt b/sentry-samples/sentry-samples-spring-boot/src/test/kotlin/io/sentry/systemtest/CacheSystemTest.kt
@@ -45,7 +45,7 @@ class CacheSystemTest {
     restClient.deleteCachedTodo(1L)
 
     testHelper.ensureTransactionReceived { transaction, _ ->
-      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.remove")
+      testHelper.doesTransactionContainSpanWithOp(transaction, "cache.evict")
     }
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ class CacheSystemTest {`
`45`	`45`	`restClient.deleteCachedTodo(1L)`
`46`	`46`
`47`	`47`	`testHelper.ensureTransactionReceived { transaction, _ ->`
`48`		`- testHelper.doesTransactionContainSpanWithOp(transaction, "cache.remove")`
	`48`	`+ testHelper.doesTransactionContainSpanWithOp(transaction, "cache.evict")`
`49`	`49`	`}`
`50`	`50`	`}`
`51`	`51`	`}`