Merge pull request #554 from Instabug/chore/increase-apm-test-coverage

Increase apm test coverage

Author: AndrewAminInstabug

test/utils/screen_loading/instabug_capture_screen_loading_test.dart

@@ -0,0 +1,41 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:instabug_flutter/instabug_flutter.dart';
+import 'package:instabug_flutter/src/utils/screen_loading/screen_loading_manager.dart';
+import 'package:mockito/mockito.dart';
+import '../instabug_navigator_observer_test.mocks.dart';
+
+void main() {
+  late MockScreenLoadingManager mockScreenLoadingManager;
+
+  setUp(() {
+    mockScreenLoadingManager = MockScreenLoadingManager();
+    ScreenLoadingManager.setInstance(mockScreenLoadingManager);
+  });
+
+  testWidgets(
+      'InstabugCaptureScreenLoading starts and reports screen loading trace',
+      (WidgetTester tester) async {
+    const screenName = "/TestScreen";
+
+    when(mockScreenLoadingManager.sanitizeScreenName(screenName))
+        .thenReturn(screenName);
+    when(mockScreenLoadingManager.startScreenLoadingTrace(any))
+        .thenAnswer((_) async {});
+    when(mockScreenLoadingManager.reportScreenLoading(any))
+        .thenAnswer((_) async {});
+
+    await tester.pumpWidget(
+      MaterialApp(
+        home: InstabugCaptureScreenLoading(
+          screenName: screenName,
+          child: Container(),
+        ),
+      ),
+    );
+
+    verify(mockScreenLoadingManager.startScreenLoadingTrace(any)).called(1);
+    await tester.pumpAndSettle();
+    verify(mockScreenLoadingManager.reportScreenLoading(any)).called(1);
+  });
+}

test/utils/screen_loading/screen_loading_trace_test.dart

@@ -0,0 +1,62 @@
+import 'package:flutter_test/flutter_test.dart';
+import 'package:instabug_flutter/src/utils/screen_loading/screen_loading_trace.dart';
+
+void main() {
+  test(
+      'ScreenLoadingTrace copyWith method should keep original values when no override happens',
+      () {
+    final trace = ScreenLoadingTrace(
+      'TestScreen',
+      startTimeInMicroseconds: 1000,
+      startMonotonicTimeInMicroseconds: 2000,
+      endTimeInMicroseconds: 3000,
+      duration: 4000,
+    );
+
+    final updatedTrace = trace.copyWith();
+
+    expect(updatedTrace.screenName, 'TestScreen');
+    expect(updatedTrace.startTimeInMicroseconds, 1000);
+    expect(updatedTrace.startMonotonicTimeInMicroseconds, 2000);
+    expect(updatedTrace.endTimeInMicroseconds, 3000);
+    expect(updatedTrace.duration, 4000);
+  });
+
+  test('ScreenLoadingTrace copyWith method updates fields correctly', () {
+    final trace = ScreenLoadingTrace(
+      'TestScreen',
+      startTimeInMicroseconds: 1000,
+      startMonotonicTimeInMicroseconds: 2000,
+      endTimeInMicroseconds: 3000,
+      duration: 4000,
+    );
+
+    final updatedTrace = trace.copyWith(
+      startTimeInMicroseconds: 1500,
+      startMonotonicTimeInMicroseconds: 2500,
+      endTimeInMicroseconds: 3500,
+      duration: 4500,
+    );
+
+    expect(updatedTrace.screenName, 'TestScreen');
+    expect(updatedTrace.startTimeInMicroseconds, 1500);
+    expect(updatedTrace.startMonotonicTimeInMicroseconds, 2500);
+    expect(updatedTrace.endTimeInMicroseconds, 3500);
+    expect(updatedTrace.duration, 4500);
+  });
+
+  test('ScreenLoadingTrace toString method returns correct format', () {
+    final trace = ScreenLoadingTrace(
+      'TestScreen',
+      startTimeInMicroseconds: 1000,
+      startMonotonicTimeInMicroseconds: 2000,
+      endTimeInMicroseconds: 3000,
+      duration: 4000,
+    );
+
+    expect(
+      trace.toString(),
+      'ScreenLoadingTrace{screenName: TestScreen, startTimeInMicroseconds: 1000, startMonotonicTimeInMicroseconds: 2000, endTimeInMicroseconds: 3000, duration: 4000}',
+    );
+  });
+}

test/utils/screen_loading/ui_trace_test.dart

@@ -0,0 +1,58 @@
+import 'package:flutter_test/flutter_test.dart';
+import 'package:instabug_flutter/instabug_flutter.dart';
+import 'package:instabug_flutter/src/utils/screen_loading/ui_trace.dart';
+import 'package:mockito/annotations.dart';
+import 'package:mockito/mockito.dart';
+
+import 'screen_loading_manager_test.mocks.dart';
+
+@GenerateMocks([RouteMatcher])
+void main() {
+  test('UiTrace copyWith method updates fields correctly', () {
+    final trace = UiTrace(
+      screenName: 'TestScreen',
+      traceId: 123,
+      matchingScreenName: 'MatchingScreen',
+    );
+
+    final updatedTrace = trace.copyWith(
+      screenName: 'UpdatedScreen',
+      traceId: 456,
+    );
+
+    expect(updatedTrace.screenName, 'UpdatedScreen');
+    expect(updatedTrace.traceId, 456);
+  });
+
+  test('UiTrace matches method returns correct result', () {
+    final mockRouteMatcher = MockRouteMatcher();
+    RouteMatcher.setInstance(mockRouteMatcher);
+    when(
+      mockRouteMatcher.match(
+        routePath: 'test/path',
+        actualPath: 'MatchingScreen',
+      ),
+    ).thenReturn(true);
+
+    final trace = UiTrace(
+      screenName: 'TestScreen',
+      traceId: 123,
+      matchingScreenName: 'MatchingScreen',
+    );
+
+    expect(trace.matches('test/path'), isTrue);
+  });
+
+  test('UiTrace toString method returns correct format', () {
+    final trace = UiTrace(
+      screenName: 'TestScreen',
+      traceId: 123,
+      matchingScreenName: 'MatchingScreen',
+    );
+
+    expect(
+      trace.toString(),
+      'UiTrace{screenName: TestScreen, traceId: 123, isFirstScreenLoadingReported: false, isFirstScreenLoading: false}',
+    );
+  });
+}