rename ctor params

Author: denrase

supabase/lib/src/sentry_supabase_client.dart

@@ -27,9 +27,9 @@ import 'sentry_supabase_error_client.dart';
 /// var supabase = SupabaseClient(
 ///   'https://example.com',
 ///   SentrySupabaseClient(
-///     breadcrumbs: false,
-///     tracing: false,
-///     errors: true,
+///     enableBreadcrumbs: false,
+///     enableTracing: false,
+///     enableErrors: true,
 ///   ),
 /// );
 /// ```
@@ -47,34 +47,34 @@ import 'sentry_supabase_error_client.dart';
 /// Body data will not be sent by default. You can enable it by setting the
 /// `sendDefaultPii` option in the [SentryOptions].
 class SentrySupabaseClient extends BaseClient {
-  final bool _breadcrumbs;
-  final bool _tracing;
-  final bool _errors;
+  final bool _enableBreadcrumbs;
+  final bool _enableTracing;
+  final bool _enableErrors;
 
   Client _innerClient;
   final Hub _hub;
 
   SentrySupabaseClient({
-    bool breadcrumbs = true,
-    bool tracing = true,
-    bool errors = true,
+    bool enableBreadcrumbs = true,
+    bool enableTracing = true,
+    bool enableErrors = true,
     Client? client,
     Hub? hub,
-  })  : _breadcrumbs = breadcrumbs,
-        _tracing = tracing,
-        _errors = errors,
+  })  : _enableBreadcrumbs = enableBreadcrumbs,
+        _enableTracing = enableTracing,
+        _enableErrors = enableErrors,
         _innerClient = client ?? Client(),
         _hub = hub ?? HubAdapter();
 
   @override
   Future<StreamedResponse> send(BaseRequest request) async {
-    if (_breadcrumbs) {
+    if (_enableBreadcrumbs) {
       _innerClient = SentrySupabaseBreadcrumbClient(_innerClient, _hub);
     }
-    if (_tracing) {
+    if (_enableTracing) {
       _innerClient = SentrySupabaseTracingClient(_innerClient, _hub);
     }
-    if (_errors) {
+    if (_enableErrors) {
       _innerClient = SentrySupabaseErrorClient(_innerClient, _hub);
     }
     return _innerClient.send(request);

supabase/test/sentry_supabase_client_test.dart

@@ -16,9 +16,9 @@ void main() {
   group('Inner Client', () {
     test('send called on send', () async {
       final sut = fixture.getSut(
-        breadcrumbs: true,
-        tracing: true,
-        errors: true,
+        enableBreadcrumbs: true,
+        enableTracing: true,
+        enableErrors: true,
       );
 
       final request = Request('GET', Uri.parse('https://example.com/123'));
@@ -31,9 +31,9 @@ void main() {
 
     test('close called on close', () async {
       final sut = fixture.getSut(
-        breadcrumbs: true,
-        tracing: true,
-        errors: true,
+        enableBreadcrumbs: true,
+        enableTracing: true,
+        enableErrors: true,
       );
 
       sut.close();
@@ -45,9 +45,9 @@ void main() {
   group('Inner Sentry Supabase Clients', () {
     test('breadcrumb client', () async {
       final sut = fixture.getSut(
-        breadcrumbs: true,
-        tracing: false,
-        errors: false,
+        enableBreadcrumbs: true,
+        enableTracing: false,
+        enableErrors: false,
       );
 
       final request = Request('GET', Uri.parse('https://example.com/123'));
@@ -58,9 +58,9 @@ void main() {
 
     test('tracing client', () async {
       final sut = fixture.getSut(
-        breadcrumbs: false,
-        tracing: true,
-        errors: false,
+        enableBreadcrumbs: false,
+        enableTracing: true,
+        enableErrors: false,
       );
 
       final request = Request('GET', Uri.parse('https://example.com/123'));
@@ -71,9 +71,9 @@ void main() {
 
     test('error client', () async {
       final sut = fixture.getSut(
-        breadcrumbs: false,
-        tracing: false,
-        errors: true,
+        enableBreadcrumbs: false,
+        enableTracing: false,
+        enableErrors: true,
       );
 
       fixture.mockClient.statusCode = 404;
@@ -86,9 +86,9 @@ void main() {
 
     test('all clients', () async {
       final sut = fixture.getSut(
-        breadcrumbs: true,
-        tracing: true,
-        errors: true,
+        enableBreadcrumbs: true,
+        enableTracing: true,
+        enableErrors: true,
       );
 
       fixture.mockClient.statusCode = 404;
@@ -113,14 +113,14 @@ class Fixture {
   late final mockHub = MockHub(options);
 
   SentrySupabaseClient getSut({
-    required bool breadcrumbs,
-    required bool tracing,
-    required bool errors,
+    required bool enableBreadcrumbs,
+    required bool enableTracing,
+    required bool enableErrors,
   }) {
     return SentrySupabaseClient(
-      breadcrumbs: breadcrumbs,
-      tracing: tracing,
-      errors: errors,
+      enableBreadcrumbs: enableBreadcrumbs,
+      enableTracing: enableTracing,
+      enableErrors: enableErrors,
       client: mockClient,
       hub: mockHub,
     );