Skip to content

Commit

Permalink
-
Browse files Browse the repository at this point in the history
  • Loading branch information
polina-c committed Dec 12, 2023
1 parent 93aef79 commit 299a505
Show file tree
Hide file tree
Showing 13 changed files with 24 additions and 16 deletions.
1 change: 0 additions & 1 deletion pkgs/leak_tracker/lib/leak_tracker.dart
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
// BSD-style license that can be found in the LICENSE file.

export 'src/leak_tracking/helpers.dart';
export 'src/leak_tracking/leak_testing.dart';
export 'src/leak_tracking/leak_tracking.dart';
export 'src/leak_tracking/primitives/model.dart';
export 'src/shared/shared_model.dart';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.

import 'package:leak_tracker/src/leak_tracking/leak_testing.dart';
import '../../../../leak_tracker_testing/lib/src/leak_testing.dart';
import 'package:leak_tracker/src/leak_tracking/primitives/model.dart';
import 'package:test/test.dart';

Expand Down
13 changes: 1 addition & 12 deletions pkgs/leak_tracker_flutter_testing/lib/src/testing.dart
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
import 'package:flutter/foundation.dart';
import 'package:leak_tracker/leak_tracker.dart';
import 'package:leak_tracker_testing/leak_tracker_testing.dart';
import 'package:matcher/expect.dart';

/// Makes sure leak tracking is set up for a test.
///
Expand Down Expand Up @@ -58,19 +57,9 @@ Future<void> maybeTearDownLeakTrackingForAll() async {
final Leaks leaks = await LeakTracking.collectLeaks();
LeakTracking.stop();

collectedLeaksReporter(leaks);
LeakTesting.collectedLeaksReporter(leaks);
}

/// Handler for memory leaks found in tests.
///
/// Set it to analyse the leaks programmatically.
/// The handler is invoked on tear down of the test run.
/// The default reporter fails in case of found leaks.
///
/// Used to test leak tracking functionality.
LeaksCallback collectedLeaksReporter =
(Leaks leaks) => expect(leaks, isLeakFree);

void _dispatchFlutterEventToLeakTracker(ObjectEvent event) {
return LeakTracking.dispatchObjectEvent(event.toMap());
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import 'package:flutter/material.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:leak_tracker/leak_tracker.dart';
import 'package:leak_tracker_flutter_testing/leak_tracker_flutter_testing.dart';
import 'package:leak_tracker_testing/leak_tracker_testing.dart';

void main() {
testWidgetsWithLeakTracking(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import 'dart:async';

import 'package:flutter_test/flutter_test.dart';
import 'package:leak_tracker_flutter_testing/leak_tracker_flutter_testing.dart';
import 'package:leak_tracker_testing/leak_tracker_testing.dart';

import 'failure_test.dart';

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

import 'package:flutter_test/flutter_test.dart';
import 'package:leak_tracker_flutter_testing/leak_tracker_flutter_testing.dart';
import 'package:leak_tracker_testing/leak_tracker_testing.dart';

import '../../test_infra/flutter_classes.dart';

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import 'dart:async';

import 'package:flutter_test/flutter_test.dart';
import 'package:leak_tracker_flutter_testing/leak_tracker_flutter_testing.dart';
import 'package:leak_tracker_testing/leak_tracker_testing.dart';

import '../../../test_infra/dart_classes.dart';
import 'per_test_config_test.dart';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
import 'package:flutter/widgets.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:leak_tracker_flutter_testing/leak_tracker_flutter_testing.dart';
import 'package:leak_tracker_testing/leak_tracker_testing.dart';

import '../../../test_infra/flutter_classes.dart';

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import 'dart:async';

import 'package:flutter_test/flutter_test.dart';
import 'package:leak_tracker_flutter_testing/leak_tracker_flutter_testing.dart';
import 'package:leak_tracker_testing/leak_tracker_testing.dart';

/// Test configuration for each test library in this directory.
///
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import 'dart:async';

import 'package:flutter_test/flutter_test.dart';
import 'package:leak_tracker_flutter_testing/leak_tracker_flutter_testing.dart';
import 'package:leak_tracker_testing/leak_tracker_testing.dart';

/// Test configuration for each test library in this directory.
///
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

import 'package:flutter_test/flutter_test.dart';
import 'package:leak_tracker/leak_tracker.dart';
import 'package:leak_tracker_testing/leak_tracker_testing.dart';

class _Classes {
static const anyLeak1 = 'anyLeak1';
Expand Down
1 change: 1 addition & 0 deletions pkgs/leak_tracker_testing/lib/leak_tracker_testing.dart
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@
// BSD-style license that can be found in the LICENSE file.

export 'src/matchers.dart';
export 'src/leak_testing.dart';
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,11 @@
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.

import 'package:leak_tracker/leak_tracker.dart';
import 'package:matcher/expect.dart';
import 'package:meta/meta.dart';

import '../shared/shared_model.dart';
import 'primitives/model.dart';
import 'matchers.dart';

void _emptyLeakHandler(Leaks leaks) {}

Expand Down Expand Up @@ -43,6 +44,16 @@ class LeakTesting {
this.baselining = const MemoryBaselining.none(),
});

/// Handler for memory leaks found in tests.
///
/// Set it to analyse the leaks programmatically.
/// The handler is invoked on tear down of the test run.
/// The default reporter fails in case of found leaks.
///
/// Used to test leak tracking functionality.
static LeaksCallback collectedLeaksReporter =
(Leaks leaks) => expect(leaks, isLeakFree);

/// Current configuration for leak tracking.
///
/// Is used by `testWidgets` if configuration is not provided for a test.
Expand Down

0 comments on commit 299a505

Please sign in to comment.