merged master

Author: stefanosiano

sentry/src/test/java/io/sentry/ScopeTest.kt

@@ -333,65 +333,68 @@ class ScopeTest {
     assertEquals(1, scope.breadcrumbs.count())
   }
 
-    @Test
-    fun `when adding breadcrumb and maxBreadcrumb is 0, beforeBreadcrumb is not executed`() {
-        var called = false
-        val options = SentryOptions().apply {
-            maxBreadcrumbs = 0
-            beforeBreadcrumb = SentryOptions.BeforeBreadcrumbCallback { breadcrumb, _ ->
-                called = true
-                breadcrumb
-            }
-        }
-
-        val scope = Scope(options)
-        scope.addBreadcrumb(Breadcrumb())
-        assertEquals(0, scope.breadcrumbs.count())
-        assertFalse(called)
-    }
+  @Test
+  fun `when adding breadcrumb and maxBreadcrumb is 0, beforeBreadcrumb is not executed`() {
+    var called = false
+    val options =
+      SentryOptions().apply {
+        maxBreadcrumbs = 0
+        beforeBreadcrumb =
+          SentryOptions.BeforeBreadcrumbCallback { breadcrumb, _ ->
+            called = true
+            breadcrumb
+          }
+      }
 
-    @Test
-    fun `when adding breadcrumb and maxBreadcrumb is not 0, beforeBreadcrumb is executed`() {
-        var called = false
-        val options = SentryOptions().apply {
-            beforeBreadcrumb = SentryOptions.BeforeBreadcrumbCallback { breadcrumb, _ ->
-                called = true
-                breadcrumb
-            }
-        }
-
-        val scope = Scope(options)
-        scope.addBreadcrumb(Breadcrumb())
-        assertEquals(1, scope.breadcrumbs.count())
-        assertTrue(called)
-    }
+    val scope = Scope(options)
+    scope.addBreadcrumb(Breadcrumb())
+    assertEquals(0, scope.breadcrumbs.count())
+    assertFalse(called)
+  }
 
-    @Test
-    fun `when adding breadcrumb and maxBreadcrumb is 0, scopesObservers are not called`() {
-        val observer = mock<IScopeObserver>()
-        val options = SentryOptions().apply {
-            maxBreadcrumbs = 0
-            addScopeObserver(observer)
-        }
-
-        val scope = Scope(options)
-        scope.addBreadcrumb(Breadcrumb())
-        assertEquals(0, scope.breadcrumbs.count())
-        verifyNoInteractions(observer)
-    }
+  @Test
+  fun `when adding breadcrumb and maxBreadcrumb is not 0, beforeBreadcrumb is executed`() {
+    var called = false
+    val options =
+      SentryOptions().apply {
+        beforeBreadcrumb =
+          SentryOptions.BeforeBreadcrumbCallback { breadcrumb, _ ->
+            called = true
+            breadcrumb
+          }
+      }
 
-    @Test
-    fun `when adding breadcrumb and maxBreadcrumb is not 0, scopesObservers are called`() {
-        val observer = mock<IScopeObserver>()
-        val options = SentryOptions().apply {
-            addScopeObserver(observer)
-        }
-
-        val scope = Scope(options)
-        scope.addBreadcrumb(Breadcrumb())
-        assertEquals(1, scope.breadcrumbs.count())
-        verify(observer).addBreadcrumb(any())
-    }
+    val scope = Scope(options)
+    scope.addBreadcrumb(Breadcrumb())
+    assertEquals(1, scope.breadcrumbs.count())
+    assertTrue(called)
+  }
+
+  @Test
+  fun `when adding breadcrumb and maxBreadcrumb is 0, scopesObservers are not called`() {
+    val observer = mock<IScopeObserver>()
+    val options =
+      SentryOptions().apply {
+        maxBreadcrumbs = 0
+        addScopeObserver(observer)
+      }
+
+    val scope = Scope(options)
+    scope.addBreadcrumb(Breadcrumb())
+    assertEquals(0, scope.breadcrumbs.count())
+    verifyNoInteractions(observer)
+  }
+
+  @Test
+  fun `when adding breadcrumb and maxBreadcrumb is not 0, scopesObservers are called`() {
+    val observer = mock<IScopeObserver>()
+    val options = SentryOptions().apply { addScopeObserver(observer) }
+
+    val scope = Scope(options)
+    scope.addBreadcrumb(Breadcrumb())
+    assertEquals(1, scope.breadcrumbs.count())
+    verify(observer).addBreadcrumb(any())
+  }
 
   @Test
   fun `when adding eventProcessor, eventProcessor should be in the list`() {