rename to beforeScreenshotCapture

denrase · denrase · commit b157a358b388 · 2024-11-18T17:59:40.000+01:00
diff --git a/flutter/lib/src/event_processor/screenshot_event_processor.dart b/flutter/lib/src/event_processor/screenshot_event_processor.dart
@@ -46,7 +46,7 @@ class ScreenshotEventProcessor implements EventProcessor {
 
     // ignore: deprecated_member_use_from_same_package
     final beforeScreenshot = _options.beforeScreenshot;
-    final beforeCapture = _options.beforeScreenshotCapture;
+    final beforeCapture = _options.beforeCaptureScreenshot;
 
     try {
       FutureOr<bool>? result;
diff --git a/flutter/lib/src/sentry_flutter_options.dart b/flutter/lib/src/sentry_flutter_options.dart
@@ -196,13 +196,13 @@ class SentryFlutterOptions extends SentryOptions {
   bool attachScreenshotOnlyWhenResumed = false;
 
   @Deprecated(
-      'Will be removed in a future version. Use [beforeScreenshotCapture] instead')
+      'Will be removed in a future version. Use [beforeCaptureScreenshot] instead')
   BeforeScreenshotCallback? beforeScreenshot;
 
   /// Sets a callback which is executed before capturing screenshots. Only
   /// relevant if `attachScreenshot` is set to true. When false is returned
   /// from the function, no screenshot will be attached.
-  BeforeCaptureCallback? beforeScreenshotCapture;
+  BeforeCaptureCallback? beforeCaptureScreenshot;
 
   /// Enable or disable automatic breadcrumbs for User interactions Using [Listener]
   ///
diff --git a/flutter/test/event_processor/screenshot_event_processor_test.dart b/flutter/test/event_processor/screenshot_event_processor_test.dart
@@ -216,10 +216,10 @@ void main() {
     });
   });
 
-  group('beforeScreenshotCapture', () {
+  group('beforeCaptureScreenshot', () {
     testWidgets('does add screenshot if beforeCapture returns true',
         (tester) async {
-      fixture.options.beforeScreenshotCapture =
+      fixture.options.beforeCaptureScreenshot =
           (SentryEvent event, Hint hint, bool shouldDebounce) {
         return true;
       };
@@ -229,7 +229,7 @@ void main() {
 
     testWidgets('does add screenshot if async beforeCapture returns true',
         (tester) async {
-      fixture.options.beforeScreenshotCapture =
+      fixture.options.beforeCaptureScreenshot =
           (SentryEvent event, Hint hint, bool shouldDebounce) async {
         await Future<void>.delayed(Duration(milliseconds: 1));
         return true;
@@ -240,7 +240,7 @@ void main() {
 
     testWidgets('does not add screenshot if beforeCapture returns false',
         (tester) async {
-      fixture.options.beforeScreenshotCapture =
+      fixture.options.beforeCaptureScreenshot =
           (SentryEvent event, Hint hint, bool shouldDebounce) {
         return false;
       };
@@ -250,7 +250,7 @@ void main() {
 
     testWidgets('does not add screenshot if async beforeCapture returns false',
         (tester) async {
-      fixture.options.beforeScreenshotCapture =
+      fixture.options.beforeCaptureScreenshot =
           (SentryEvent event, Hint hint, bool shouldDebounce) async {
         await Future<void>.delayed(Duration(milliseconds: 1));
         return false;
@@ -261,7 +261,7 @@ void main() {
 
     testWidgets('does add screenshot if beforeCapture throws', (tester) async {
       fixture.options.automatedTestMode = false;
-      fixture.options.beforeScreenshotCapture =
+      fixture.options.beforeCaptureScreenshot =
           (SentryEvent event, Hint hint, bool shouldDebounce) {
         throw Error();
       };
@@ -272,7 +272,7 @@ void main() {
     testWidgets('does add screenshot if async beforeCapture throws',
         (tester) async {
       fixture.options.automatedTestMode = false;
-      fixture.options.beforeScreenshotCapture =
+      fixture.options.beforeCaptureScreenshot =
           (SentryEvent event, Hint hint, bool shouldDebounce) async {
         await Future<void>.delayed(Duration(milliseconds: 1));
         throw Error();
@@ -286,7 +286,7 @@ void main() {
       await tester.runAsync(() async {
         var shouldDebounceValues = <bool>[];
 
-        fixture.options.beforeScreenshotCapture =
+        fixture.options.beforeCaptureScreenshot =
             (SentryEvent event, Hint hint, bool shouldDebounce) {
           shouldDebounceValues.add(shouldDebounce);
           return true;
@@ -323,7 +323,7 @@ void main() {
       SentryEvent? beforeScreenshotEvent;
       Hint? beforeScreenshotHint;
 
-      fixture.options.beforeScreenshotCapture =
+      fixture.options.beforeCaptureScreenshot =
           (SentryEvent event, Hint hint, bool shouldDebounce) {
         beforeScreenshotEvent = event;
         beforeScreenshotHint = hint;
