chore: remove unused code (#1252)

getsentry · Feb 6, 2023 · 3637a22 · 3637a22
1 parent e893df5
commit 3637a22
Show file tree

Hide file tree

Showing 8 changed files with 4 additions and 48 deletions.
diff --git a/dart/test/http_client/tracing_client_test.dart b/dart/test/http_client/tracing_client_test.dart
@@ -1,6 +1,5 @@
 import 'package:http/http.dart';
 import 'package:http/testing.dart';
-import 'package:mockito/mockito.dart';
 import 'package:sentry/sentry.dart';
 import 'package:sentry/src/http_client/tracing_client.dart';
 import 'package:sentry/src/sentry_tracer.dart';
@@ -194,8 +193,6 @@ MockClient createThrowingClient() {
   );
 }
 
-class CloseableMockClient extends Mock implements BaseClient {}
-
 class Fixture {
   final _options = SentryOptions(dsn: fakeDsn);
   late Hub _hub;

diff --git a/dart/test/mocks.dart b/dart/test/mocks.dart
@@ -99,14 +99,6 @@ final fakeEvent = SentryEvent(
   ),
 );
 
-/// Doesn't do anything with the events
-class NoOpEventProcessor extends EventProcessor {
-  @override
-  FutureOr<SentryEvent?> apply(SentryEvent event, {hint}) {
-    return event;
-  }
-}
-
 /// Always returns null and thus drops all events
 class DropAllEventProcessor extends EventProcessor {
   @override

diff --git a/dart/test/sentry_envelope_test.dart b/dart/test/sentry_envelope_test.dart
@@ -186,6 +186,4 @@ void main() {
   });
 }
 
-class NonEncodable {
-  final String message = 'Hello World';
-}
+class NonEncodable {}
diff --git a/dart/test/sentry_user_feedback_test.dart b/dart/test/sentry_user_feedback_test.dart
@@ -171,15 +171,6 @@ class SentryUserFeedbackWithoutAssert implements SentryUserFeedback {
     this.comments,
   });
 
-  factory SentryUserFeedbackWithoutAssert.fromJson(Map<String, dynamic> json) {
-    return SentryUserFeedbackWithoutAssert(
-      eventId: SentryId.fromId(json['event_id']),
-      name: json['name'],
-      email: json['email'],
-      comments: json['comments'],
-    );
-  }
-
   @override
   final SentryId eventId;
 

diff --git a/dart/test/transport/http_transport_test.dart b/dart/test/transport/http_transport_test.dart
@@ -7,15 +7,13 @@ import 'package:sentry/src/client_reports/discard_reason.dart';
 import 'package:sentry/src/sentry_envelope_header.dart';
 import 'package:sentry/src/sentry_envelope_item_header.dart';
 import 'package:sentry/src/sentry_item_type.dart';
-import 'package:sentry/src/sentry_tracer.dart';
 import 'package:sentry/src/transport/data_category.dart';
 import 'package:sentry/src/transport/http_transport.dart';
 import 'package:sentry/src/transport/rate_limiter.dart';
 import 'package:test/test.dart';
 
 import '../mocks.dart';
 import '../mocks/mock_client_report_recorder.dart';
-import '../mocks/mock_hub.dart';
 
 void main() {
   SentryEnvelope givenEnvelope() {
@@ -236,15 +234,4 @@ class Fixture {
     options.recorder = clientReportRecorder;
     return HttpTransport(options, rateLimiter);
   }
-
-  SentryTracer createTracer({
-    bool? sampled = true,
-  }) {
-    final context = SentryTransactionContext(
-      'name',
-      'op',
-      samplingDecision: SentryTracesSamplingDecision(sampled!),
-    );
-    return SentryTracer(context, MockHub());
-  }
 }
diff --git a/flutter/example/lib/main.dart b/flutter/example/lib/main.dart
@@ -112,13 +112,13 @@ class MainScaffold extends StatelessWidget {
           ),
           IconButton(
             onPressed: () {
-              themeProvider.updatePrimatryColor(Colors.orange);
+              themeProvider.updatePrimaryColor(Colors.orange);
             },
             icon: const Icon(Icons.circle, color: Colors.orange),
           ),
           IconButton(
             onPressed: () {
-              themeProvider.updatePrimatryColor(Colors.green);
+              themeProvider.updatePrimaryColor(Colors.green);
             },
             icon: const Icon(Icons.circle, color: Colors.lime),
           ),
@@ -677,7 +677,7 @@ class ThemeProvider extends ChangeNotifier {
     notifyListeners();
   }
 
-  void updatePrimatryColor(MaterialColor color) {
+  void updatePrimaryColor(MaterialColor color) {
     if (theme.brightness == Brightness.light) {
       theme = ThemeData(primarySwatch: color, brightness: theme.brightness);
     } else {

diff --git a/flutter/lib/src/jvm/jvm_exception.dart b/flutter/lib/src/jvm/jvm_exception.dart
@@ -19,14 +19,6 @@ class JvmException {
     return _parse(exception);
   }
 
-  static JvmException? tryParse(String exception) {
-    try {
-      return JvmException.parse(exception);
-    } catch (_) {
-      return null;
-    }
-  }
-
   final String? thread;
   final String? type;
   final String? description;

diff --git a/flutter/test/integrations/load_release_integration_test.dart b/flutter/test/integrations/load_release_integration_test.dart
@@ -138,7 +138,6 @@ void main() {
 }
 
 class Fixture {
-  final hub = MockHub();
   final options = SentryFlutterOptions(dsn: fakeDsn);
 
   LoadReleaseIntegration getIntegration({PackageLoader? loader}) {